max/embassy
1
0
Fork 0
Code Issues Pull Requests Packages Projects Releases Wiki Activity

stm32/dma: fix spurious transfer complete interrupts

Browse Source 
DMA interrupts must be acknowledged by writing to the DMA_{L,H}IFCR
register.

Writing to the CR register is unnecessary as the channel (EN bit) is
disabled by hardware on completion of the transfer.
This commit is contained in:
Patrick Oppenlander
2023-02-23 10:09:09 +11:00
parent d159a6c62d
commit 4e884ee2d2
1 changed files with 2 additions and 6 deletions
Download Patch File Download Diff File Expand all files Collapse all files

8
embassy-stm32/src/dma/dma.rs
View File

@@ -411,12 +411,8 @@ mod low_level_api {
} }
if isr.tcif(channel_num % 4) && cr.read().tcie() { if isr.tcif(channel_num % 4) && cr.read().tcie() {
if cr.read().dbm() == vals::Dbm::DISABLED { /* acknowledge transfer complete interrupt */
cr.write(|_| ()); // Disable channel with the default value. dma.ifcr(channel_num / 4).write(|w| w.set_tcif(channel_num % 4, true));
} else {
// for double buffered mode, clear TCIF flag but do not stop the transfer
dma.ifcr(channel_num / 4).write(|w| w.set_tcif(channel_num % 4, true));
}
STATE.channels[state_index].waker.wake(); STATE.channels[state_index].waker.wake();
} }
} }