search

rust-embedded-community / usbd-serial

Work-in progress minimal CDC-ACM (USB serial port) class for usb-device
MIT License
116 stars 35 forks source link

Add High Speed device support #14

Closed Disasm closed 3 years ago

Disasm commented 3 years ago

I'd like to know what do you think about this approach. I don't like it, but also I don't know how to do better right now.

mvirkkunen commented 3 years ago

The main issue with this would be compliance because this would present invalid descriptors to a full-speed host. Full-speed only hosts are very rare in practice these days, but it'd be nice to stay compliant.

High-speed support is something that has to start from adding actual support to usb-device itself. That's something that would be next on the list after "endpoint-trait" is released.

If you need HS only CDC-ACM for something right now, I suggest copying and pasting the class into the project for now. It's not that big.

Disasm commented 3 years ago

Good point, thank you!