Fix comments.

bakerstu · Jul 8, 2023 · e4c9d95 · e4c9d95
1 parent 0496cf4
commit e4c9d95
Show file tree

Hide file tree

Showing 3 changed files with 14 additions and 13 deletions.
diff --git a/include/freertos/tc_ioctl.h b/include/freertos/tc_ioctl.h
@@ -53,7 +53,6 @@
 /// Two stop bits
 #define TCSTOPTWO   IO(TERMIOS_IOC_MAGIC, 0xF9)
 
-
 /// Argument is a Notifiable* pointer. This notifiable will be invoked when all
 /// bytes have completed transferring and the transmit engine is idle.
 #define TCDRAINNOTIFY   IOW(TERMIOS_IOC_MAGIC, 0xE0, 4)

diff --git a/src/freertos_drivers/ti/CC32xxUart.cxx b/src/freertos_drivers/ti/CC32xxUart.cxx
@@ -314,15 +314,16 @@ void CC32xxUart::interrupt_handler()
                 rxBuf->signal_condition_from_isr();
             }
         }
-        else
+        else if (data >= 0 && data <= 0xff)
         {
-            if (data >= 0 && data <= 0xff)
+            if (rxBuf->space() < 1)
+            {
+                ++overrunCount;
+            }
+            else
             {
                 unsigned char c = data;
-                if (rxBuf->put(&c, 1) == 0)
-                {
-                    ++overrunCount;
-                }
+                rxBuf->put(&c, 1);
                 rxBuf->signal_condition_from_isr();
             }
         }

diff --git a/src/freertos_drivers/ti/TivaUart.cxx b/src/freertos_drivers/ti/TivaUart.cxx
@@ -236,15 +236,16 @@ void TivaUart::interrupt_handler()
                 rxBuf->signal_condition_from_isr();
             }
         }
-        else
+        else if (data >= 0 && data <= 0xff)
         {
-            if (data >= 0 && data <= 0xff)
+            if (rxBuf->space() < 1)
+            {
+                ++overrunCount;
+            }
+            else
             {
                 unsigned char c = data;
-                if (rxBuf->put(&c, 1) == 0)
-                {
-                    overrunCount++;
-                }
+                rxBuf->put(&c, 1);
                 rxBuf->signal_condition_from_isr();
             }
         }