riot-ci/RIOT
1
0
Fork 0
Code Issues Pull Requests Projects Releases Wiki Activity

Merge pull request #15899 from OTAkeys/pr/stm32-fix-exti

Browse Source 
cpu/stm32/gpio: fix EXTI flag clearing
This commit is contained in:
benpicco 2021-02-01 18:22:25 +01:00 committed by GitHub
commit efd8afd3ab
No known key found for this signature in database
GPG Key ID: 4AEE18F83AFDEB23
1 changed files with 4 additions and 3 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

7
cpu/stm32/periph/gpio_all.c
View File

@ -36,6 +36,7 @@
* @brief The STM32F0 family has 16 external interrupt lines * @brief The STM32F0 family has 16 external interrupt lines
*/ */
#define EXTI_NUMOF (16U) #define EXTI_NUMOF (16U)
#define EXTI_MASK (0xFFFF)
/** /**
* @brief Allocate memory for one callback and argument per EXTI channel * @brief Allocate memory for one callback and argument per EXTI channel
@ -338,8 +339,8 @@ void isr_exti(void)
#if defined(CPU_FAM_STM32G0) || defined(CPU_FAM_STM32L5) || \ #if defined(CPU_FAM_STM32G0) || defined(CPU_FAM_STM32L5) || \
defined(CPU_FAM_STM32MP1) defined(CPU_FAM_STM32MP1)
/* only generate interrupts against lines which have their IMR set */ /* only generate interrupts against lines which have their IMR set */
uint32_t pending_rising_isr = (EXTI->RPR1 & EXTI_REG_IMR); uint32_t pending_rising_isr = (EXTI->RPR1 & EXTI_REG_IMR & EXTI_MASK);
uint32_t pending_falling_isr = (EXTI->FPR1 & EXTI_REG_IMR); uint32_t pending_falling_isr = (EXTI->FPR1 & EXTI_REG_IMR & EXTI_MASK);
/* clear by writing a 1 */ /* clear by writing a 1 */
EXTI->RPR1 = pending_rising_isr; EXTI->RPR1 = pending_rising_isr;
@ -348,7 +349,7 @@ void isr_exti(void)
uint32_t pending_isr = pending_rising_isr | pending_falling_isr; uint32_t pending_isr = pending_rising_isr | pending_falling_isr;
#else #else
/* only generate interrupts against lines which have their IMR set */ /* only generate interrupts against lines which have their IMR set */
uint32_t pending_isr = (EXTI_REG_PR & EXTI_REG_IMR); uint32_t pending_isr = (EXTI_REG_PR & EXTI_REG_IMR & EXTI_MASK);
/* clear by writing a 1 */ /* clear by writing a 1 */
EXTI_REG_PR = pending_isr; EXTI_REG_PR = pending_isr;