CRITICAL: stm32h7xx_hal_tim_ex.h wrong defines

[Max Rest]

Dear STM,

please fix the defines in stm32h7xx_hal_tim_ex.h as at least TIM_TIMx_ETR are either empty or wrong, according to the datasheet. Here is a short example, _NOT_ stating all incorrect positions:

NOW, IN THE FILE stm32h7xx_hal_tim_ex.h in STM32Cube_FW_H7_V1.3.2 :

#define TIM_TIM5_ETR_GPIO                          (0x00000000) /* !< TIM5_ETR is connected to GPIO */
#define TIM_TIM5_ETR_SAI2_FSA                      (0x00000000) /* !< TIM5_ETR is connected to SAI2 FS_A */
#define TIM_TIM5_ETR_SAI2_FSB                      (0x00000000) /* !< TIM5_ETR is connected to SAI2 FS_B */

CORRECT, according to RM0433 Rev 5, p. 1686/3247

#define TIM_TIM5_ETR_GPIO                          (0x00000000) /* !< TIM5_ETR is connected to GPIO */
#define TIM_TIM5_ETR_SAI2_FSA                      (0x00001000) /* !< TIM5_ETR is connected to SAI2 FS_A */
#define TIM_TIM5_ETR_SAI2_FSB                      (0x00002000) /* !< TIM5_ETR is connected to SAI2 FS_B */

This is a critical bug.

Thanks,

Maximilian Rest

[Mike_ST · Accepted solution]

Hello Max,

thank you for reporting,

This bug is already fixed internally. It should be in a next release.

[Max Rest]

Hi Mike,

thanks for your quick reply, sounds good. I hope you meant in _the_ next release though, because fixing it manually and overwriting it each time cube regenerates the files is quite ... annoying.

Best,

Max