STM32CubeL0 icon indicating copy to clipboard operation
STM32CubeL0 copied to clipboard

Duplicate volatile reads in interrupt handler for hal_timer

Open jrahlf opened this issue 4 years ago • 2 comments

Similar to #17 , there are unnecessary duplicate volatile reads in the interrupt handler for the timer peripheral. __HAL_TIM_GET_FLAG is called several times inside the handler, each time the actual value is fetched due to volatile qualifier.

https://github.com/STMicroelectronics/STM32CubeL0/blob/a7b74aed35ecb7baeadeb16107aa8fddb6823589/Drivers/STM32L0xx_HAL_Driver/Src/stm32l0xx_hal_tim.c#L3560

As in #17, this likely affects all other device variants (F1/F3/F4...) as well.

jrahlf avatar Apr 26 '21 21:04 jrahlf

ST Internal Reference: 162350

ASELSTM avatar Jun 08 '21 16:06 ASELSTM

Hi @jrahlf,

Thank you for your contribution. This has been communicated to the development team. We will come back to you as soon as we get their feedback.

Thank you again for your contribution.

With regards,

ASELSTM avatar Jun 09 '21 09:06 ASELSTM

Fixed in 9d7402e

TOUNSTM avatar Jan 25 '24 16:01 TOUNSTM