-
Notifications
You must be signed in to change notification settings - Fork 348
Implementing async feature for USB Serial/JTAG peripheral.
#632
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
Conversation
|
Solving #633 will allow to get this merged |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
This looks quite good to me, thanks for your work here! I've left a couple small comments, but honestly nothing worth blocking merging on. If you get a chance to update them that'd be great, if not no worries. This PR will need to be rebased, though.
I think if we're blocking this PR only on the ESP32-S3 issues, that there is no real need to do so and we can merge it and then we already have an issue to remind ourselves.
|
#633 has been resolved, so after a rebase this PR should (hopefully) be in good shape! |
Fix: Interrupt name
PS. : Yet only for esp32c3, we need to update PACs Temporary example fix
+ Update examples according to upstream + Code cleaning Rust format
352c5c4 to
048ad13
Compare
942109c to
99c48bb
Compare
Typo fix Fix fmt
99c48bb to
683a44f
Compare
|
FYI I've patched the USB serial jtag interrupts to all be |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Seems to be working based on my testing, and code looks good, so LGTM!
|
…s#632) * Implementing ASYNC features for USB_SERIAL_JTAG (work in progress) Fix: Interrupt name * Async works for USB Serial/JTAG PS. : Yet only for esp32c3, we need to update PACs Temporary example fix * Adjustment for different chips + Update examples according to upstream + Code cleaning Rust format * Add record to the CHANGELOG * Code cleaning, getting rid of useless generic argument Typo fix Fix fmt
At this point we have some issues connected with
esp32s3chip, corresponding issue will be created.