Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Update USB library #2737

Merged
merged 4 commits into from
Mar 19, 2024
Merged

Update USB library #2737

merged 4 commits into from
Mar 19, 2024

Conversation

mikee47
Copy link
Contributor

@mikee47 mikee47 commented Mar 19, 2024

This PR fixes USB support for ESP IDF 5.2. Note that host mode is still not yet supported for esp32sx.

Improvements:

  • Update to tinyusb master
  • Move initialisation code into separate Arch source file, patch to support ESP32 IDF 5.2 (not yet in tinyusb upstream)
  • Add CI support (speeds up library testing)

Fixes:

  • Lazy-initialise flush timer: Causes problems with static Device objects and esp32sx as timer's aren't yet initialised
  • Enable python UTF8 mode for usbconfig tool (Windows)
  • UsbSerial conditional on CDC
  • Don't implement descriptor callbacks unless building for device
  • Ensure HID device is ready when sendReport called: get div-by-zero exception on esp32s2 otherwise
  • Fix Basic_IFS sample with ENABLE_USB_STORAGE: only for rp2040, no host support yet for esp32sx

Tested with rp2040, esp32s2.

@mikee47 mikee47 force-pushed the feature/USB-upgrade branch from c18c0ef to 2c994ef Compare March 19, 2024 11:16
@SmingHub SmingHub deleted a comment from what-the-diff bot Mar 19, 2024
@slaff slaff added this to the 5.2.0 milestone Mar 19, 2024
@slaff slaff merged commit 5352490 into SmingHub:develop Mar 19, 2024
46 checks passed
@mikee47 mikee47 deleted the feature/USB-upgrade branch March 19, 2024 15:15
@slaff slaff mentioned this pull request May 8, 2024
5 tasks
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants