ErichStyger / mcuoneclipse

McuOnEclipse Processor Expert components and example projects
Other
731 stars 1.29k forks source link

Fixed App_Callback() to be interrupt safe for asynchronous TX/RX streams... #18

Closed kevinuskali closed 10 years ago

kevinuskali commented 10 years ago

It looks like the original code was written for a sequential TX => RX => TX... chain. On the other hand, if the MCU (CDC_DEVICE) and host communicate with asynchronous TX/RX streams, the RX endpoint FSM on the MCU can become corrupted. This corruption stalls the incoming RX stream. The changes resolve this issue.

ErichStyger commented 10 years ago

Many thanks for that contribution. I have not thought about this being an issue, and it seems that I have not seen it on my side. Anyway, good fix, thank you!

mcu101 commented 9 years ago

Thanks, this could solve the issue I'm experiencing regarding incoming CDC messages not being acknowledged by the MCU (missing USB ACK byte). I'll let you guys know.

Carlos

mcu101 commented 9 years ago

It works!

Thanks a lot!