Use USART TX state consistently

git-svn-id: svn://svn.code.sf.net/p/nuttx/code/trunk@3720 42af7a65-404d-4744-a932-0658087f49c3

3 files changed, 81 insertions, 24 deletions

diff --git a/nuttx/Documentation/NuttX.html b/nuttx/Documentation/NuttX.html
index 28e97d01b..e386b6bdb 100644
--- a/nuttx/Documentation/NuttX.html
+++ b/nuttx/Documentation/NuttX.html
@@ -1518,9 +1518,9 @@
   <td>
     <p>
       <b>SoC Robotics ATMega128</b>.
-      This port of NuttX to the Amber Web Server from SoC Robotics
-      (http://www.soc-robotics.com/index.htm) partially complete.  The Amber Web Server is
-      based on an Atmel ATMega128.
+      This port of NuttX to the Amber Web Server from <a href="http://www.soc-robotics.com/index.htm">SoC Robotics</a>
+      is partially completed.
+      The Amber Web Server is based on an Atmel ATMega128.
     </p>
     <ul>
       <p>
@@ -1550,10 +1550,7 @@
        <b>STATUS:</b>
          The basic port was release in NuttX-6.5.  This basic port consists only of
          a "Hello, World!!" example that demonstrates initialization of the OS,
-         creation of a simple task, and serial console output.  A complete OS
-         test configuration is also available, but this would have to be scaled
-         down quite a bit before it could be executed in the tiny AT90USB SRAM
-         memory.
+         creation of a simple task, and serial console output.
       </p>
     </ul>
   </td>
@@ -1568,15 +1565,17 @@
     <p>
       <b>PJRC Teensy++ 2.0 AT9USB1286</b>.
       This is a port of NuttX to the PJRC Teensy++ 2.0 board.
-      This board is developed by http://pjrc.com/teensy/. 
+      This board is developed by <a href="http://pjrc.com/teensy/">PJRC</a>.
       The Teensy++ 2.0 is based on an Atmel AT90USB1286 MCU.
     </p>
     <ul>
       <p>
        <b>STATUS:</b>
-         Work on the Teensy board is just beginning as of this writing.
-         However, because of the great similarity to the Micropendous board,
-         this port should be completed with only a small additional effort.
+         The basic port was release in NuttX-6.5.  This basic port consists of
+         a "Hello, World!!" example that demonstrates initialization of the OS,
+         creation of a simple task, and serial console output as well as a
+         simplified NuttShell (NSH) configuration (see the
+         <a href="http://www.nuttx.org/NuttShell.html">NSH User Guide</a>).
       </p>
     </ul>
   </td>
diff --git a/nuttx/arch/avr/src/at90usb/at90usb_serial.c b/nuttx/arch/avr/src/at90usb/at90usb_serial.c
index 35c6d691a..88769bc0b 100644
--- a/nuttx/arch/avr/src/at90usb/at90usb_serial.c
+++ b/nuttx/arch/avr/src/at90usb/at90usb_serial.c
@@ -100,6 +100,7 @@ static bool usart1_rxavailable(struct uart_dev_s *dev);
 static void usart1_send(struct uart_dev_s *dev, int ch);
 static void usart1_txint(struct uart_dev_s *dev, bool enable);
 static bool usart1_txready(struct uart_dev_s *dev);
+static bool usart1_txempty(struct uart_dev_s *dev);
 
 /****************************************************************************
  * Private Variables
@@ -118,7 +119,7 @@ struct uart_ops_s g_uart1_ops =
   .send           = usart1_send,
   .txint          = usart1_txint,
   .txready        = usart1_txready,
-  .txempty        = usart1_txready,
+  .txempty        = usart1_txempty,
 };
 
 /* I/O buffers */
@@ -168,9 +169,12 @@ static void usart1_restoreusartint(uint8_t imr)
 
 static inline void usart1_disableusartint(uint8_t *imr)
 {
-  uint8_t regval = UCSR1B;
+  uint8_t regval;
+
+  regval  = UCSR1B;
   *imr    = regval;
   regval &= ~((1 << RXCIE1) | (1 << TXCIE1) | (1 << UDRIE1));
+  UCSR1B  = regval;
 }
 
 /****************************************************************************
@@ -308,9 +312,11 @@ static int usart1_txinterrupt(int irq, void *context)
 {
   uint8_t ucsr1a = UCSR1A;
 
-  /* Handle outgoing, transmit bytes */
+  /* Handle outgoing, transmit bytes when the transmit data buffer is empty.
+   * (There may still be data in the shift register).
+   */
 
-  if ((ucsr1a & (1 << TXC1)) != 0)
+  if ((ucsr1a & (1 << UDRE1)) != 0)
     {
        /* Transmit data regiser empty ... process outgoing bytes */
 
@@ -361,7 +367,7 @@ static int usart1_ioctl(struct file *filep, int cmd, unsigned long arg)
 
 static int usart1_receive(struct uart_dev_s *dev, FAR unsigned int *status)
 {
-  /* Return status information */
+  /* Return status information (error bits will be cleared after reading UDR1) */
 
   if (status)
     {
@@ -471,6 +477,7 @@ static void usart1_txint(struct uart_dev_s *dev, bool enable)
 
       UCSR1B &= ~((1 << UDRIE1) | (1 << TXCIE1));
     }
+
   irqrestore(flags);
 }
 
@@ -488,6 +495,20 @@ static bool usart1_txready(struct uart_dev_s *dev)
 }
 
 /****************************************************************************
+ * Name: usart1_txempty
+ *
+ * Description:
+ *   Return true if the tranmsit data register and shift register are both
+ *   empty
+ *
+ ****************************************************************************/
+
+static bool usart1_txempty(struct uart_dev_s *dev)
+{
+  return (UCSR1A & (1 << TXC1)) != 0;
+}
+
+/****************************************************************************
  * Public Functions
  ****************************************************************************/
 
diff --git a/nuttx/arch/avr/src/atmega/atmega_serial.c b/nuttx/arch/avr/src/atmega/atmega_serial.c
index 696cd5e9a..813fa9e8f 100644
--- a/nuttx/arch/avr/src/atmega/atmega_serial.c
+++ b/nuttx/arch/avr/src/atmega/atmega_serial.c
@@ -121,6 +121,7 @@ static bool usart0_rxavailable(struct uart_dev_s *dev);
 static void usart0_send(struct uart_dev_s *dev, int ch);
 static void usart0_txint(struct uart_dev_s *dev, bool enable);
 static bool usart0_txready(struct uart_dev_s *dev);
+static bool usart0_txempty(struct uart_dev_s *dev);
 #endif
 
 #ifdef CONFIG_AVR_USART1
@@ -137,6 +138,7 @@ static bool usart1_rxavailable(struct uart_dev_s *dev);
 static void usart1_send(struct uart_dev_s *dev, int ch);
 static void usart1_txint(struct uart_dev_s *dev, bool enable);
 static bool usart1_txready(struct uart_dev_s *dev);
+static bool usart1_txempty(struct uart_dev_s *dev);
 #endif
 
 /****************************************************************************
@@ -159,7 +161,7 @@ struct uart_ops_s g_usart0_ops =
   .send           = usart0_send,
   .txint          = usart0_txint,
   .txready        = usart0_txready,
-  .txempty        = usart0_txready,
+  .txempty        = usart0_txempty,
 };
 
 /* USART0 I/O buffers */
@@ -201,7 +203,7 @@ struct uart_ops_s g_usart1_ops =
   .send           = usart1_send,
   .txint          = usart1_txint,
   .txready        = usart1_txready,
-  .txempty        = usart1_txready,
+  .txempty        = usart1_txempty,
 };
 
 /* USART 1 I/O buffers */
@@ -268,18 +270,24 @@ static void usart1_restoreusartint(uint8_t imr)
 #ifdef CONFIG_AVR_USART0
 static inline void usart0_disableusartint(uint8_t *imr)
 {
-  uint8_t regval = UCSR0B;
+  uint8_t regval;
+
+  regval  = UCSR0B;
   *imr    = regval;
   regval &= ~((1 << RXCIE0) | (1 << TXCIE0) | (1 << UDRIE0));
+  UCSR0B  = regval;
 }
 #endif
 
 #ifdef CONFIG_AVR_USART1
 static inline void usart1_disableusartint(uint8_t *imr)
 {
-  uint8_t regval = UCSR1B;
+  uint8_t regval;
+
+  regval  = UCSR1B;
   *imr    = regval;
   regval &= ~((1 << RXCIE1) | (1 << TXCIE1) | (1 << UDRIE1));
+  UCSR0B  = regval;
 }
 #endif
 
@@ -505,9 +513,11 @@ static int usart0_txinterrupt(int irq, void *context)
 {
   uint8_t ucsr0a = UCSR0A;
 
-  /* Handle outgoing, transmit bytes */
+  /* Handle outgoing, transmit bytes when the transmit data buffer is empty.
+   * (There may still be data in the shift register).
+   */
 
-  if ((ucsr0a & (1 << TXC0)) != 0)
+  if ((ucsr0a & (1 << UDRE0)) != 0)
     {
        /* Transmit data regiser empty ... process outgoing bytes */
 
@@ -523,9 +533,11 @@ static int usart1_txinterrupt(int irq, void *context)
 {
   uint8_t ucsr1a = UCSR1A;
 
-  /* Handle outgoing, transmit bytes */
+  /* Handle outgoing, transmit bytes when the transmit data buffer is empty.
+   * (There may still be data in the shift register).
+   */
 
-  if ((ucsr1a & (1 << TXC1)) != 0)
+  if ((ucsr1a & (1 << UDRE1)) != 0)
     {
        /* Transmit data regiser empty ... process outgoing bytes */
 
@@ -773,6 +785,7 @@ static void usart0_txint(struct uart_dev_s *dev, bool enable)
 
       UCSR0B &= ~((1 << UDRIE0) | (1 << TXCIE0));
     }
+
   irqrestore(flags);
 }
 #endif
@@ -814,6 +827,7 @@ static void usart1_txint(struct uart_dev_s *dev, bool enable)
 
       UCSR1B &= ~((1 << UDRIE1) | (1 << TXCIE1));
     }
+
   irqrestore(flags);
 }
 #endif
@@ -841,6 +855,29 @@ static bool usart1_txready(struct uart_dev_s *dev)
 #endif
 
 /****************************************************************************
+ * Name: usart0/1_txempty
+ *
+ * Description:
+ *   Return true if the tranmsit data register and shift reqister are both
+ *   empty
+ *
+ ****************************************************************************/
+
+#ifdef CONFIG_AVR_USART0
+static bool usart0_txempty(struct uart_dev_s *dev)
+{
+  return (UCSR0A & (1 << TXC0)) != 0;
+}
+#endif
+
+#ifdef CONFIG_AVR_USART1
+static bool usart1_txempty(struct uart_dev_s *dev)
+{
+  return (UCSR1A & (1 << TXC1)) != 0;
+}
+#endif
+
+/****************************************************************************
  * Public Functions
  ****************************************************************************/