Added Error flag to show if a UART overflow has occurred

Author: gfwilliams

ChangeLog

@@ -22,6 +22,7 @@
             ESP8266: Wifi library doesn't handle {password: null} (fix #753)
             ESP8266: make topstrings and topreadonly work on Mac OS X (fix #1210)
             Change order of execution for init - E.on('init',...) now executed before onInit
+            Added Error flag to show if a UART overflow has occurred
 
      1v93 : Ensure that specifying too many advertising UUIDs causes an exception, not a reboot
             nRF5x: Fix for time jump caused by reentrancy in jshGetSystemTime

src/jsutils.h

@@ -404,6 +404,7 @@ typedef enum {
   JSERR_LOW_MEMORY = 8, ///< Memory is running low - Espruino had to run a garbage collection pass or remove some of the command history
   JSERR_MEMORY = 16, ///< Espruino ran out of memory and was unable to allocate some data that it needed.
   JSERR_MEMORY_BUSY = 32, ///< Espruino was busy doing something with memory (eg. garbage collection) so an IRQ couldn't allocate memory
+  JSERR_UART_OVERFLOW = 64 ///< A UART received data but it was not read in time and was lost
 } PACKED_FLAGS JsErrorFlags;
 
 /** Error flags for things that we don't really want to report on the console,

src/jswrap_espruino.c

@@ -644,6 +644,8 @@ Get and reset the error flags. Returns an array that can contain:
 `'LOW_MEMORY'`: Memory is running low - Espruino had to run a garbage collection pass or remove some of the command history
 
 `'MEMORY'`: Espruino ran out of memory and was unable to allocate some data that it needed.
+
+`'JSERR_UART_OVERFLOW'` : A UART received data but it was not read in time and was lost
  */
 JsVar *jswrap_espruino_getErrorFlags() {
   JsVar *arr = jsvNewEmptyArray();
@@ -653,7 +655,8 @@ JsVar *jswrap_espruino_getErrorFlags() {
   if (jsErrorFlags&JSERR_CALLBACK) jsvArrayPushAndUnLock(arr, jsvNewFromString("CALLBACK"));
   if (jsErrorFlags&JSERR_LOW_MEMORY) jsvArrayPushAndUnLock(arr, jsvNewFromString("LOW_MEMORY"));
   if (jsErrorFlags&JSERR_MEMORY) jsvArrayPushAndUnLock(arr, jsvNewFromString("MEMORY"));
-  if (jsErrorFlags&JSERR_MEMORY_BUSY) jsvArrayPushAndUnLock(arr, jsvNewFromString("JSERR_MEMORY_BUSY"));
+  if (jsErrorFlags&JSERR_MEMORY_BUSY) jsvArrayPushAndUnLock(arr, jsvNewFromString("MEMORY_BUSY"));
+  if (jsErrorFlags&JSERR_UART_OVERFLOW) jsvArrayPushAndUnLock(arr, jsvNewFromString("UART_OVERFLOW"));
   jsErrorFlags = JSERR_NONE;
   return arr;
 }

targets/nrf5x/jshardware.c

@@ -838,6 +838,8 @@ void uart0_event_handle(app_uart_evt_t * p_event) {
       jshPushIOEvent(IOEVENTFLAGS_SERIAL_TO_SERIAL_STATUS(EV_SERIAL1) | EV_SERIAL_STATUS_FRAMING_ERR, 0);
     if (p_event->data.error_communication & (UART_ERRORSRC_PARITY_Msk) && jshGetErrorHandlingEnabled(EV_SERIAL1))
       jshPushIOEvent(IOEVENTFLAGS_SERIAL_TO_SERIAL_STATUS(EV_SERIAL1) | EV_SERIAL_STATUS_PARITY_ERR, 0);
+    if (p_event->data.error_communication & (UART_ERRORSRC_OVERRUN_Msk))
+      jsErrorFlags |= JSERR_UART_OVERFLOW;
   } else if (p_event->evt_type == APP_UART_TX_EMPTY) {
     int ch = jshGetCharToTransmit(EV_SERIAL1);
     if (ch >= 0) {

targets/stm32/stm32_it.c

@@ -300,8 +300,8 @@ NO_INLINE static void USART_IRQHandler(USART_TypeDef *USART, IOEventFlags device
     jshPushIOCharEvent(device, (char)USART_ReceiveData(USART));
   }
   /* If overrun condition occurs, clear the ORE flag and recover communication */
-  if (USART_GetFlagStatus(USART, USART_FLAG_ORE) != RESET)
-  {
+  if (USART_GetFlagStatus(USART, USART_FLAG_ORE) != RESET) {
+    jsErrorFlags |= JSERR_UART_OVERFLOW;
     (void)USART_ReceiveData(USART);
   }
   if(USART_GetITStatus(USART, USART_IT_TXE) != RESET) {