Bug in HAL_IWDG_DEFAULT_TIMEOUT

[Kira-sempai]

STM32CubeF2/Drivers/STM32F2xx_HAL_Driver/Src/stm32f2xx_hal_iwdg.c

Line 124 in cfa4386

#define HAL_IWDG_DEFAULT_TIMEOUT (((6UL * 256UL * 1000UL) / LSI_VALUE) + ((LSI_STARTUP_TIME / 1000UL) + 1UL))

Probably should be
#define HAL_IWDG_DEFAULT_TIMEOUT (((6UL * 256UL * 1000UL) / LSI_VALUE) + ((LSI_STARTUP_TIMEOUT / 1000UL) + 1UL))

[Kira-sempai]

BTW, can't find it's definition in stm32f2xx_hal_conf_template.h

[RKOUSTM]

Hi @Kira-sempai,

Thank you for this report. It has been transferred to our development teams. We will be back to you as soon as we get an answer.

With regards,

[RKOUSTM]

Hi @Kira-sempai,

Thank you for your contribution. The point you reported is related to a typo in the comment description. Indeed, a fix in the comment below :

STM32CubeF2/Drivers/STM32F2xx_HAL_Driver/Src/stm32f2xx_hal_iwdg.c

Lines 121 to 123 in cfa4386

	The timeout value is multiplied by 1000 to be converted in milliseconds.
	LSI startup time is also considered here by adding LSI_STARTUP_TIMEOUT
	converted in milliseconds. */

will be updated and made available in future release looks like the following:

  The timeout value is multiplied by 1000 to be converted in milliseconds.
- LSI startup time is also considered here by adding LSI_STARTUP_TIMEOUT
+ LSI startup time is also considered here by adding LSI_STARTUP_TIME
   converted in milliseconds. 

Concerning the define of the LSI_STARTUP_TIME, you can find it in the include files of the CMSIS device.

Thank you once again for your contribution.

With regards,

[RKOUSTM]

ST Internal Reference: 111928

[TOUNSTM]

Hello @Kira-sempai,

The fix has been implemented and is now available in the frame of the latest STM32CubeF2 package V1.9.4 release. This issue can be closed now. Thank you again for your contribution.

With Regards,