RIOT-OS/RIOT
RIOT-OS/RIOT
1
0
Fork 0
mirror of https://github.com/RIOT-OS/RIOT.git synced 2024-12-29 04:50:03 +01:00
Code Releases Activity

cpu/sam3/periph/timer: fix trigger of cleared timer

Browse Source
This commit is contained in:
Kaspar Schleiser 2020-10-30 13:06:12 +01:00
parent e8453eaba8
commit 8455756618
2 changed files with 15 additions and 2 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

9
cpu/sam3/include/periph_cpu.h
View File

@ -66,9 +66,14 @@ typedef uint32_t gpio_t;
#define TIMER_MAX_VAL (0xffffffff)
/**
* @brief We use 3 channels for each defined timer
* @brief We use one channel for each defined timer
*
* While the peripheral provides three channels, the current interrupt
* flag handling leads to a race condition where calling timer_clear() on one
* channel can disable a pending flag for other channels.
* Until resolved, limit the peripheral to only one channel.
*/
#define TIMER_CHANNEL_NUMOF (3)
#define TIMER_CHANNEL_NUMOF (1)
/**
* @brief The RTT width is fixed to 32-bit

8
cpu/sam3/periph/timer.c
View File

@ -128,6 +128,14 @@ int timer_set_absolute(tim_t tim, int channel, unsigned int value)
return -1;
}
(&dev(tim)->TC_CHANNEL[0].TC_RA)[channel] = value;
/* read TC status register to clear any possibly pending
* ISR flag (that has not been served yet).
* timer_clear() disables the interrupt, but does not clear the flags.
* if we don't clear them here, re-enabling the interrupt below
* can trigger for the previously disabled timer. */
(void)dev(tim)->TC_CHANNEL[0].TC_SR;
dev(tim)->TC_CHANNEL[0].TC_IER = (TC_IER_CPAS << channel);
return 0;