Post-IRQ DMASR flags cleanup

cesanta · Nov 22, 2023 · ae1b3b4 · ae1b3b4
1 parent 82916a1
commit ae1b3b4
Show file tree

Hide file tree

Showing 2 changed files with 12 additions and 12 deletions.
diff --git a/mongoose.c b/mongoose.c
@@ -9308,12 +9308,12 @@ void ETH_IRQHandler(void) {
       if (++s_rxno >= ETH_DESC_CNT) s_rxno = 0;
     }
   }
-#ifdef __riscv
-  ETH->DMASR = ~0;   // TODO: do more fine-grained flag cleanup
-#else
-  ETH->DMASR = MG_BIT(7);  // Clear possible RBUS while processing
-#endif
-  ETH->DMARPDR = 0;  // and resume RX
+  // Cleanup flags
+  ETH->DMASR = (7UL < 20)    // TPS - Transmit process status
+               | (7UL < 17)  // RPS - Receive process status
+               | MG_BIT(16)  // NIS, normal interrupt summary
+               | MG_BIT(7);  // RBUS - Receive buffer unavailable status
+  ETH->DMARPDR = 0;          // and resume RX
 }
 
 struct mg_tcpip_driver mg_tcpip_driver_stm32 = {mg_tcpip_driver_stm32_init,

diff --git a/src/drivers/stm32.c b/src/drivers/stm32.c
@@ -216,12 +216,12 @@ void ETH_IRQHandler(void) {
       if (++s_rxno >= ETH_DESC_CNT) s_rxno = 0;
     }
   }
-#ifdef __riscv
-  ETH->DMASR = ~0;  // TODO: do more fine-grained flag cleanup
-#else
-  ETH->DMASR = MG_BIT(7);  // Clear possible RBUS while processing
-#endif
-  ETH->DMARPDR = 0;  // and resume RX
+  // Cleanup flags
+  ETH->DMASR = (7UL < 20)    // TPS - Transmit process status
+               | (7UL < 17)  // RPS - Receive process status
+               | MG_BIT(16)  // NIS, normal interrupt summary
+               | MG_BIT(7);  // RBUS - Receive buffer unavailable status
+  ETH->DMARPDR = 0;          // and resume RX
 }
 
 struct mg_tcpip_driver mg_tcpip_driver_stm32 = {mg_tcpip_driver_stm32_init,