Added idle interupt to callback to UART driver (#482)

Added uart interrupt callback to the serial driver which is required for Spektrum SRXL2 protocol
emuflight · Mar 19, 2021 · 0e89169 · 0e89169
1 parent eeb396b
commit 0e89169
Show file tree

Hide file tree

Showing 6 changed files with 42 additions and 3 deletions.
diff --git a/src/main/drivers/serial_uart_hal.c b/src/main/drivers/serial_uart_hal.c
@@ -125,6 +125,8 @@ void uartReconfigure(uartPort_t *uartPort) {
             SET_BIT(uartPort->USARTx->CR3, USART_CR3_EIE);
             /* Enable the UART Data Register not empty Interrupt */
             SET_BIT(uartPort->USARTx->CR1, USART_CR1_RXNEIE);
+            /* Enable Idle Line detection */
+            SET_BIT(uartPort->USARTx->CR1, USART_CR1_IDLEIE);
         }
     }
     // Transmit DMA or IRQ

diff --git a/src/main/drivers/serial_uart_init.c b/src/main/drivers/serial_uart_init.c
@@ -167,6 +167,7 @@ serialPort_t *uartOpen(UARTDevice_e device, serialReceiveCallbackPtr rxCallback,
         } else {
             USART_ClearITPendingBit(s->USARTx, USART_IT_RXNE);
             USART_ITConfig(s->USARTx, USART_IT_RXNE, ENABLE);
+            USART_ITConfig(s->USARTx, USART_IT_IDLE, ENABLE);
         }
     }
     // Transmit DMA or IRQ

diff --git a/src/main/drivers/serial_uart_stm32f10x.c b/src/main/drivers/serial_uart_stm32f10x.c
@@ -194,5 +194,13 @@ void uartIrqHandler(uartPort_t *s) {
             USART_ITConfig(s->USARTx, USART_IT_TXE, DISABLE);
         }
     }
+    if (SR & USART_FLAG_IDLE) {
+        if (s->port.idleCallback) {
+            s->port.idleCallback();
+        }
+
+        const uint32_t read_to_clear = s->USARTx->DR;
+        (void) read_to_clear;
+      }
 }
 #endif // USE_UART
diff --git a/src/main/drivers/serial_uart_stm32f30x.c b/src/main/drivers/serial_uart_stm32f30x.c
@@ -261,8 +261,17 @@ void uartIrqHandler(uartPort_t *s) {
             USART_ITConfig(s->USARTx, USART_IT_TXE, DISABLE);
         }
     }
-    if (ISR & USART_FLAG_ORE) {
-        USART_ClearITPendingBit (s->USARTx, USART_IT_ORE);
+    if (ISR & USART_FLAG_ORE)
+    {
+      USART_ClearITPendingBit(s->USARTx, USART_IT_ORE);
+    }
+
+    if (ISR & USART_FLAG_IDLE) {
+        if (s->port.idleCallback) {
+            s->port.idleCallback();
+          }
+
+          USART_ClearITPendingBit(s->USARTx, USART_IT_IDLE);
     }
 }
 #endif // USE_UART
diff --git a/src/main/drivers/serial_uart_stm32f4xx.c b/src/main/drivers/serial_uart_stm32f4xx.c
@@ -264,8 +264,19 @@ void uartIrqHandler(uartPort_t *s) {
             USART_ITConfig(s->USARTx, USART_IT_TXE, DISABLE);
         }
     }
+
     if (USART_GetITStatus(s->USARTx, USART_IT_ORE) == SET) {
-        USART_ClearITPendingBit (s->USARTx, USART_IT_ORE);
+        USART_ClearITPendingBit(s->USARTx, USART_IT_ORE);
+    }
+
+    if (USART_GetITStatus(s->USARTx, USART_IT_IDLE) == SET) {
+        if (s->port.idleCallback) {
+            s->port.idleCallback();
+        }
+
+        // clear
+        (void) s->USARTx->SR;
+        (void) s->USARTx->DR;
     }
 }
 #endif
diff --git a/src/main/drivers/serial_uart_stm32f7xx.c b/src/main/drivers/serial_uart_stm32f7xx.c
@@ -300,6 +300,14 @@ void uartIrqHandler(uartPort_t *s) {
             handleUsartTxDma(s);
         }
     }
+
+        if (__HAL_UART_GET_IT(huart, UART_IT_IDLE)) {
+            if (s->port.idleCallback) {
+                s->port.idleCallback();
+            }
+
+            __HAL_UART_CLEAR_IDLEFLAG(huart);
+        }    
 }
 
 static void handleUsartTxDma(uartPort_t *s) {