osEventFlagsWait with options

STMicroelectronics / STM32CubeU5

Full Firmware Package for the STM32U5 series: HAL+LL drivers, CMSIS, BSP, MW, plus a set of Projects (examples and demos) running on all boards provided by ST (Nucleo, Evaluation and Discovery Kits).

Other

116 stars 61 forks source link

osEventFlagsWait with options #14

Closed albetaCOM closed 1 year ago

albetaCOM commented 2 years ago

osEventFlagsWait with wait options osFlagsWaitAny (OR) and osFlagsWaitAll (AND)

IMPORTANT INFORMATION

Contributor License Agreement (CLA)

The Pull Request feature will be considered by STMicroelectronics after the signature of a Contributor License Agreement (CLA) by the submitter.
If you did not sign such agreement, please follow the steps mentioned in the CONTRIBUTING.md file.

ASELSTM commented 2 years ago

ST Internal Reference: 131718

matth314 commented 1 year ago

We also modified this part in our code, but I think we did clearer code with less cyclomatic complexity:

  if (options == osFlagsNoClear)
  {
    get_option = TX_OR;
  }
  else if (options == osFlagsWaitAny)
  {
    get_option = TX_OR_CLEAR;
  }
  else if (options == (osFlagsWaitAll | osFlagsNoClear) )
  {
    get_option = TX_AND;
  }
  else
  {
    get_option = TX_AND_CLEAR;
  }

ASELSTM commented 1 year ago

Hi @albetaCOM,

Thank you for your contribution. This issue has been fixed in the frame of version v1.2.0 of the STM32CubeU5. Please allow me then to close this thread.

With regards,