Lufa CDC Echo

[Bülent C. (mirki)]

Hallo zusammen,

anbei habe ich folgenden Code,mit dem ich über eine Terminalsoftware was 
an den AT90USB162 (Lufa Implementierung für USB CDC) sende und er mir 
das gleiche wieder zurücksenden soll.

Es klappt auch, nur halt sehr langsam, also ich kann nicht frei drauf 
lostippen und muss zwischen den einzelnen Eingaben warten bis diese 
akzeptiert werden.
Mache ich hier was grundlegendes falsch? Und vorallem, müsste ich bei 
einem empfangen Byte nicht in eine Interrupt Routine abspringen können?

int main(void)
{
  
  SetupHardware();
  CDC_Device_CreateStream(&VirtualSerial_CDC_Interface, &USBSerialStream);
  sei();

  for (;;)
  {
    CDC_Device_ReceiveByte(&VirtualSerial_CDC_Interface);
    
    int16_t ReceivedByte = CDC_Device_ReceiveByte(&VirtualSerial_CDC_Interface);
       if (!(ReceivedByte < 0))
         CDC_Device_SendByte(&VirtualSerial_CDC_Interface, (uint8_t)ReceivedByte);


     CDC_Device_USBTask(&VirtualSerial_CDC_Interface);
     
     USB_USBTask();
    
  }
}

[Stephan B. (matrixstorm)]

Hi

Fuer soetwas kennt LUFA meines Wissens nach Ringpuffer.
Ueber deren Benutzung solltest du ggf. einmal nachdenken.

(Dadurch wuerden sich dann deine verschiedenen Peripherien nicht 
gegenseitig ausbremsen.)

Nachtrag:
In meinen Beispielen ( http://matrixstorm.com/avr/avrstick/#bideavr ) 
wird intern auch LUFA (mit Puffern) verwendet - laeuft ohne Probleme.

MfG

[Bülent C. (mirki)]

Hi,

Wo ist jetzt der große Unterschied zu meinem code?

int main(void) {
    /* Init the USB (and clock + interrupts) */
    USBInit();

    for (;;) {
        /* Must throw away unused bytes from the host, or it will lock up while waiting for the device */
        int c = fgetc(USBtty0);

        /* simple demo: implement an uppercase echo */
        if (!(c<0)) {
            fputc(toupper(c), USBtty0);
        }

        USBPoll();
    }
}

[Stephan B. (matrixstorm)]

Ja hast recht, ich habe nochmal genauer in die lib gesehen - am Puffer 
duerfte es nicht liegen.

Wohl aber an:

    CDC_Device_ReceiveByte(&VirtualSerial_CDC_Interface);
    
    int16_t ReceivedByte = CDC_Device_ReceiveByte(&VirtualSerial_CDC_Interface);

[...]

Was zu lockups/loss fuehren koennte, wenn du tatsaechlich mal was 
empfangen willst?

Nachtrag: Loech einfach mal bitte das erste "CDC_Device_ReceiveByte" 
raus.

[Bülent C. (mirki)]

Ja, daran lag es und es funktioniert jetzt gut..Danke!