STM32CubeG0 icon indicating copy to clipboard operation
STM32CubeG0 copied to clipboard

LL Syscfg comments incorrectly identify COMP EXTI lines

Open grantg012 opened this issue 3 years ago • 2 comments

The comments in https://github.com/STMicroelectronics/STM32CubeG0/blob/master/Drivers/STM32G0xx_HAL_Driver/Inc/stm32g0xx_ll_system.h#L1021 incorrectly label COMP1 and COMP2 to use EXTI 21 and 22 respectively. As per the reference manual, they use EXTI 17 and 18.

Additional context If you have a first analysis or a patch proposal, thank you to share your proposal.

Screenshots image

grantg012 avatar Aug 17 '22 14:08 grantg012

Hi @grantg012,

The comment are indeed incorrect. The EXTI lines should be updated as you have mentioned.

/**
-  * @brief  Check if Comparator 1 interrupt occurred or not (EXTI line 21).
+  * @brief  Check if Comparator 1 interrupt occurred or not (EXTI line 17).
  * @rmtoll SYSCFG_ITLINE12 SR_COMP1      LL_SYSCFG_IsActiveFlag_COMP1
  * @retval State of bit (1 or 0).
  */
/**
-  * @brief  Check if Comparator 2 interrupt occurred or not (EXTI line 22).
+  * @brief  Check if Comparator 2 interrupt occurred or not (EXTI line 18).
  * @rmtoll SYSCFG_ITLINE12 SR_COMP2      LL_SYSCFG_IsActiveFlag_COMP2
  * @retval State of bit (1 or 0).
  */
  */

With regards,

ASELSTM avatar Sep 01 '22 10:09 ASELSTM

ST Internal Reference: 133892

ASELSTM avatar Sep 01 '22 10:09 ASELSTM

fixed in commit 9f4c4a1c1ee2a497eb94defee1525f8cf7100bd5

TOUNSTM avatar Nov 23 '23 17:11 TOUNSTM