cpu/atmega_common/periph_timer: fix spurious IRQs

maribu · Nov 25, 2022 · 787884a · 787884a
1 parent f1a8e1f
commit 787884a
Showing 1 changed file with 9 additions and 2 deletions.
diff --git a/cpu/atmega_common/periph/timer.c b/cpu/atmega_common/periph/timer.c
@@ -158,7 +158,9 @@ int timer_set_absolute(tim_t tim, int channel, unsigned int value)
     unsigned state = irq_disable();
 
     ctx[tim].dev->OCR[channel] = (uint16_t)value;
-    *ctx[tim].flag &= ~(1 << (channel + OCF1A));
+    /* clear spurious IRQs, if any */
+    *ctx[tim].flag = (1 << (channel + OCF1A));
+    /* unmask IRQ */
     *ctx[tim].mask |= (1 << (channel + OCIE1A));
     set_oneshot(tim, channel);
 
@@ -177,6 +179,9 @@ int timer_set(tim_t tim, int channel, unsigned int timeout)
     unsigned absolute = ctx[tim].dev->CNT + timeout;
 
     ctx[tim].dev->OCR[channel] = absolute;
+    /* clear spurious IRQs, if any */
+    *ctx[tim].flag = (1 << (channel + OCF1A));
+    /* unmask IRQ */
     *ctx[tim].mask |= (1 << (channel + OCIE1A));
     set_oneshot(tim, channel);
 
@@ -210,7 +215,9 @@ int timer_set_periodic(tim_t tim, int channel, unsigned int value, uint8_t flags
 
     ctx[tim].dev->OCR[channel] = (uint16_t)value;
 
-    *ctx[tim].flag &= ~(1 << (channel + OCF1A));
+    /* clear spurious IRQs, if any */
+    *ctx[tim].flag = (1 << (channel + OCF1A));
+    /* unmask IRQ */
     *ctx[tim].mask |=  (1 << (channel + OCIE1A));
 
     clear_oneshot(tim, channel);