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

SDK 2.1.0 tinyusb panics on composite usb audio and cdc acm device #2236

Open
not1337 opened this issue Feb 4, 2025 · 5 comments
Open

SDK 2.1.0 tinyusb panics on composite usb audio and cdc acm device #2236

not1337 opened this issue Feb 4, 2025 · 5 comments
Assignees
Labels
Milestone

Comments

@not1337
Copy link

not1337 commented Feb 4, 2025

The tinyusb version pulled by SDK 2.1.0 is missing a fix causing panic when using a composite usb device (audio and cdc/acm). The problem seems to be well known and fixed in current versions of tinyusb. See: hathach/tinyusb#1802
A future SDK release needs an updated version of tinyusb.

As a temporary fix for everyone affected, replace the dcd_rp2040.c of the pulled tinyusb library with:

https://raw.githubusercontent.com/howard0su/tinyusb/cd7ee9639ee5ceddcdaecde5842c2ce97a56ce63/src/portable/raspberrypi/rp2040/dcd_rp2040.c

The above fixes the problem for me and the link is included for convenience to reduce search time for everyone affected.

@kilograham
Copy link
Contributor

Note that we don't want that particular fix, but we want the change that is being made in TinyUSB upstream (which may or may not be this one hathach/tinyusb#2937 )

@kilograham kilograham added this to the 2.1.1 milestone Feb 4, 2025
@kilograham
Copy link
Contributor

OP: is this a regression from earlier SDK, or was that also broken due to TinyUSB/rp2040 issues?

@not1337
Copy link
Author

not1337 commented Feb 5, 2025

Sorry, don't know. SDK 2.1.0 is the first version I'm using and I never used rp2040, only rp2350.

@kilograham kilograham modified the milestones: 2.1.1, 2.2.0 Feb 6, 2025
@Slion
Copy link

Slion commented Feb 14, 2025

Is this bug specific to rp2040 or does it affect rp2350 too?

@not1337
Copy link
Author

not1337 commented Feb 14, 2025

In my case rp2350a, I don't have any rp2040.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

No branches or pull requests

4 participants