Fixed USBtoSerial and XPLAINBridge demos discarding data from the PC if the send...
authorDean Camera <dean@fourwalledcubicle.com>
Wed, 3 Nov 2010 11:24:34 +0000 (11:24 +0000)
committerDean Camera <dean@fourwalledcubicle.com>
Wed, 3 Nov 2010 11:24:34 +0000 (11:24 +0000)
LUFA/Drivers/USB/HighLevel/EndpointStream.h
LUFA/Drivers/USB/HighLevel/PipeStream.h
LUFA/ManPages/ChangeLog.txt
LUFA/ManPages/LibraryResources.txt
Projects/USBtoSerial/USBtoSerial.c
Projects/XPLAINBridge/XPLAINBridge.c

index 5c3c957..a8b9021 100644 (file)
@@ -51,7 +51,6 @@
 #define __ENDPOINT_STREAM_H__\r
 \r
        /* Includes: */\r
-               #include <avr/io.h>\r
                #include <avr/pgmspace.h>\r
                #include <avr/eeprom.h>\r
                #include <stdbool.h>\r
index 1279a71..730a338 100644 (file)
@@ -51,7 +51,6 @@
 #define __PIPE_STREAM_H__\r
 \r
        /* Includes: */\r
-               #include <avr/io.h>\r
                #include <avr/pgmspace.h>\r
                #include <avr/eeprom.h>\r
                #include <stdbool.h>\r
index 69b129d..86aac5c 100644 (file)
@@ -93,6 +93,7 @@
   *  - Fixed random enumeration failure while in device mode due to interrupts causing the Set Address request to exceed maximum timings
   *  - Fixed MIDI_Host_Flush() not aborting early when the specified MIDI host interface was not configured
   *  - Fixed MIDI class driver send routines silently discarding packets if the endpoint or pipe is busy (thanks to Robin Green)
+  *  - Fixed USBtoSerial and XPLAINBridge demos discarding data from the PC if the send buffer becomes full
   *
   *  \section Sec_ChangeLog100807 Version 100807
   *  <b>New:</b>
index 062702b..7f67739 100644 (file)
@@ -20,6 +20,7 @@
  *
  *  \section Sec_InDevelopment Latest In-Development Source Code
  *  Issue Tracker: http://www.lufa-lib.org/tracker \n
+ *  Bazaar Access: http://www.lufa-lib.org/bzr \n
  *  SVN Access: http://www.lufa-lib.org/svn \n
  *  Git Access: http://www.lufa-lib.org/git \n
  *  Mercurial Access: http://www.lufa-lib.org/hg \n
index 3f06acd..c26667d 100644 (file)
@@ -81,15 +81,21 @@ int main(void)
 
        for (;;)
        {
-               /* Read bytes from the USB OUT endpoint into the USART transmit buffer */
-               int16_t ReceivedByte = CDC_Device_ReceiveByte(&VirtualSerial_CDC_Interface);
-               if (!(ReceivedByte < 0) && !(RingBuffer_IsFull(&USBtoUSART_Buffer)))
-                 RingBuffer_Insert(&USBtoUSART_Buffer, ReceivedByte);
+               /* Only try to read in bytes from the CDC interface if the transmit buffer is not full */
+               if (!(RingBuffer_IsFull(&USBtoUSART_Buffer)))
+               {
+                       int16_t ReceivedByte = CDC_Device_ReceiveByte(&VirtualSerial_CDC_Interface);
 
+                       /* Read bytes from the USB OUT endpoint into the USART transmit buffer */
+                       if (!(ReceivedByte < 0))
+                         RingBuffer_Insert(&USBtoUSART_Buffer, ReceivedByte);
+               }
+               
                /* Check if the UART receive buffer flush timer has expired or the buffer is nearly full */
                RingBuff_Count_t BufferCount = RingBuffer_GetCount(&USARTtoUSB_Buffer);
                if ((TIFR0 & (1 << TOV0)) || (BufferCount > 200))
                {
+                       /* Clear flush timer expiry flag */
                        TIFR0 |= (1 << TOV0);
 
                        /* Read bytes from the USART receive buffer into the USB IN endpoint */
index 8cd1b2d..d645ebe 100644 (file)
@@ -119,15 +119,21 @@ void UARTBridge_Task(void)
        if (USB_DeviceState != DEVICE_STATE_Configured)
          return;
 
-       /* Read bytes from the USB OUT endpoint into the UART transmit buffer */
-       int16_t ReceivedByte = CDC_Device_ReceiveByte(&VirtualSerial_CDC_Interface);
-       if (!(ReceivedByte < 0) && !(RingBuffer_IsFull(&USBtoUART_Buffer)))
-         RingBuffer_Insert(&USBtoUART_Buffer, ReceivedByte);
+       /* Only try to read in bytes from the CDC interface if the transmit buffer is not full */
+       if (!(RingBuffer_IsFull(&USBtoUART_Buffer)))
+       {
+               int16_t ReceivedByte = CDC_Device_ReceiveByte(&VirtualSerial_CDC_Interface);
 
+               /* Read bytes from the USB OUT endpoint into the UART transmit buffer */
+               if (!(ReceivedByte < 0))
+                 RingBuffer_Insert(&USBtoUART_Buffer, ReceivedByte);
+       }
+       
        /* Check if the UART receive buffer flush timer has expired or buffer is nearly full */
        RingBuff_Count_t BufferCount = RingBuffer_GetCount(&UARTtoUSB_Buffer);
        if ((TIFR0 & (1 << TOV0)) || (BufferCount > 200))
        {
+               /* Clear flush timer expiry flag */
                TIFR0 |= (1 << TOV0);
 
                /* Read bytes from the UART receive buffer into the USB IN endpoint */