-
-
Notifications
You must be signed in to change notification settings - Fork 8.2k
Portenta C33 USB regression. #15651
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
Comments
Hi @iabdalkader I wanted to capture this regression here in its own separate issue for tracking, I feel it really should be resolved one way or another before the next release. I don't personally feel confident that just a crystal/clock issue could cause the fault in combination with 2d33071 One of the primary goals of my set of stdio consolidate commits was to enable the runtime USB / I don't have any renesas high speed devices to test & fix with however. Did you get any feedback on whether the newer builds do work on production boards though? |
I think it really might just be the bad crystal there, it also had something to do with USB maybe it derived some clock from it. I wouldn't revert anything until we can test it.
Not yet, everyone is on vacation including me and I don't have a second C33 to test with, but will check again. |
Everything is working on a production board, USB and WiFi:
|
I can test this when I get back in two weeks: #15550 |
That all sounds great thanks! I still find it curious it worked before the change on your board but not after, but I'll take any positive test results! Thanks for the feedback and enjoy the rest of your time off! |
Port, board and/or hardware
Renesas port, Portenta C33
MicroPython version
Micropython nightly builds after v1.23
Reproduction
Use DFU to load micropython firmware.
Expected behaviour
USB serial / repl should be available.
Observed behaviour
USB does not enumerate.
Additional Information
As reported in #14462 (comment)
Code of Conduct
Yes, I agree
The text was updated successfully, but these errors were encountered: