Better fix for LowLevel CDC demo issue where sending data before the line encoding...

[pub/USBasp.git] / Demos / Device / LowLevel / USBtoSerial / USBtoSerial.c

diff --git a/Demos/Device/LowLevel/USBtoSerial/USBtoSerial.c b/Demos/Device/LowLevel/USBtoSerial/USBtoSerial.c
index af963b1..2765064 100644 (file)
--- a/Demos/Device/LowLevel/USBtoSerial/USBtoSerial.c
+++ b/Demos/Device/LowLevel/USBtoSerial/USBtoSerial.c
@@ -36,10 +36,10 @@
  *  These values are set by the host via a class-specific request, and the physical USART should be reconfigured to match the\r
  *  new settings each time they are changed by the host.\r
  */\r
  *  These values are set by the host via a class-specific request, and the physical USART should be reconfigured to match the\r
  *  new settings each time they are changed by the host.\r
  */\r

-CDC_Line_Coding_t LineCoding = { .BaudRateBPS = 9600,\r
-                                 .CharFormat  = OneStopBit,\r
-                                 .ParityType  = Parity_None,\r
-                                 .DataBits    = 8            };\r
+CDC_Line_Coding_t LineEncoding = { .BaudRateBPS = 0,\r
+                                   .CharFormat  = OneStopBit,\r
+                                   .ParityType  = Parity_None,\r
+                                   .DataBits    = 8            };\r

 \r
 /** Ring (circular) buffer to hold the RX data - data from the host to the attached device on the serial port. */\r
 RingBuff_t Rx_Buffer;\r
 \r
 /** Ring (circular) buffer to hold the RX data - data from the host to the attached device on the serial port. */\r
 RingBuff_t Rx_Buffer;\r


@@ -145,8 +145,6 @@ void EVENT_USB_ConfigurationChanged(void)
  */\r
 void EVENT_USB_UnhandledControlPacket(void)\r
 {\r
  */\r
 void EVENT_USB_UnhandledControlPacket(void)\r
 {\r

-       uint8_t* LineCodingData = (uint8_t*)&LineCoding;\r
-\r

        /* Process CDC specific control requests */\r
        switch (USB_ControlRequest.bRequest)\r
        {\r
        /* Process CDC specific control requests */\r
        switch (USB_ControlRequest.bRequest)\r
        {\r


@@ -157,7 +155,7 @@ void EVENT_USB_UnhandledControlPacket(void)
                                Endpoint_ClearSETUP();\r
 \r
                                /* Write the line coding data to the control endpoint */\r
                                Endpoint_ClearSETUP();\r
 \r
                                /* Write the line coding data to the control endpoint */\r

-                               Endpoint_Write_Control_Stream_LE(LineCodingData, sizeof(LineCoding));\r
+                               Endpoint_Write_Control_Stream_LE(&LineEncoding, sizeof(LineEncoding));\r

                                \r
                                /* Finalize the stream transfer to send the last packet or clear the host abort */\r
                                Endpoint_ClearOUT();\r
                                \r
                                /* Finalize the stream transfer to send the last packet or clear the host abort */\r
                                Endpoint_ClearOUT();\r


@@ -171,7 +169,7 @@ void EVENT_USB_UnhandledControlPacket(void)
                                Endpoint_ClearSETUP();\r
 \r
                                /* Read the line coding data in from the host into the global struct */\r
                                Endpoint_ClearSETUP();\r
 \r
                                /* Read the line coding data in from the host into the global struct */\r

-                               Endpoint_Read_Control_Stream_LE(LineCodingData, sizeof(LineCoding));\r
+                               Endpoint_Read_Control_Stream_LE(&LineEncoding, sizeof(LineEncoding));\r

 \r
                                /* Finalize the stream transfer to clear the last packet from the host */\r
                                Endpoint_ClearIN();\r
 \r
                                /* Finalize the stream transfer to clear the last packet from the host */\r
                                Endpoint_ClearIN();\r


@@ -259,7 +257,7 @@ void CDC_Task(void)
        Endpoint_SelectEndpoint(CDC_TX_EPNUM);\r
 \r
        /* Check if the Tx buffer contains anything to be sent to the host */\r
        Endpoint_SelectEndpoint(CDC_TX_EPNUM);\r
 \r
        /* Check if the Tx buffer contains anything to be sent to the host */\r

-       if (Tx_Buffer.Elements)\r
+       if ((Tx_Buffer.Elements) && LineEncoding.BaudRateBPS)\r

        {\r
                /* Wait until Serial Tx Endpoint Ready for Read/Write */\r
                while (!(Endpoint_IsReadWriteAllowed()))\r
        {\r
                /* Wait until Serial Tx Endpoint Ready for Read/Write */\r
                while (!(Endpoint_IsReadWriteAllowed()))\r


@@ -304,7 +302,7 @@ void CDC_Task(void)
 ISR(USART1_RX_vect, ISR_BLOCK)\r
 {\r
        /* Only store received characters if the USB interface is connected */\r
 ISR(USART1_RX_vect, ISR_BLOCK)\r
 {\r
        /* Only store received characters if the USB interface is connected */\r

-       if (USB_DeviceState != DEVICE_STATE_Configured)\r
+       if ((USB_DeviceState != DEVICE_STATE_Configured) && LineEncoding.BaudRateBPS)\r

          Buffer_StoreElement(&Tx_Buffer, UDR1);\r
 }\r
 \r
          Buffer_StoreElement(&Tx_Buffer, UDR1);\r
 }\r
 \r


@@ -314,21 +312,21 @@ void ReconfigureUSART(void)
        uint8_t ConfigMask = 0;\r
 \r
        /* Determine parity - non odd/even parity mode defaults to no parity */\r
        uint8_t ConfigMask = 0;\r
 \r
        /* Determine parity - non odd/even parity mode defaults to no parity */\r

-       if (LineCoding.ParityType == Parity_Odd)\r
+       if (LineEncoding.ParityType == Parity_Odd)\r

          ConfigMask = ((1 << UPM11) | (1 << UPM10));\r
          ConfigMask = ((1 << UPM11) | (1 << UPM10));\r

-       else if (LineCoding.ParityType == Parity_Even)\r
+       else if (LineEncoding.ParityType == Parity_Even)\r

          ConfigMask = (1 << UPM11);\r
 \r
        /* Determine stop bits - 1.5 stop bits is set as 1 stop bit due to hardware limitations */\r
          ConfigMask = (1 << UPM11);\r
 \r
        /* Determine stop bits - 1.5 stop bits is set as 1 stop bit due to hardware limitations */\r

-       if (LineCoding.CharFormat == TwoStopBits)\r
+       if (LineEncoding.CharFormat == TwoStopBits)\r

          ConfigMask |= (1 << USBS1);\r
 \r
        /* Determine data size - 5, 6, 7, or 8 bits are supported */\r
          ConfigMask |= (1 << USBS1);\r
 \r
        /* Determine data size - 5, 6, 7, or 8 bits are supported */\r

-       if (LineCoding.DataBits == 6)\r
+       if (LineEncoding.DataBits == 6)\r

          ConfigMask |= (1 << UCSZ10);\r
          ConfigMask |= (1 << UCSZ10);\r

-       else if (LineCoding.DataBits == 7)\r
+       else if (LineEncoding.DataBits == 7)\r

          ConfigMask |= (1 << UCSZ11);\r
          ConfigMask |= (1 << UCSZ11);\r

-       else if (LineCoding.DataBits == 8)\r
+       else if (LineEncoding.DataBits == 8)\r

          ConfigMask |= ((1 << UCSZ11) | (1 << UCSZ10));\r
        \r
        /* Enable double speed, gives better error percentages at 8MHz */\r
          ConfigMask |= ((1 << UCSZ11) | (1 << UCSZ10));\r
        \r
        /* Enable double speed, gives better error percentages at 8MHz */\r


@@ -341,5 +339,5 @@ void ReconfigureUSART(void)
        UCSR1C = ConfigMask;\r
        \r
        /* Set the USART baud rate register to the desired baud rate value */\r
        UCSR1C = ConfigMask;\r
        \r
        /* Set the USART baud rate register to the desired baud rate value */\r

-       UBRR1  = SERIAL_2X_UBBRVAL((uint16_t)LineCoding.BaudRateBPS);\r
+       UBRR1  = SERIAL_2X_UBBRVAL((uint16_t)LineEncoding.BaudRateBPS);\r

 }\r
 }\r