Skip to content

Conversation

acassis
Copy link
Contributor

@acassis acassis commented Aug 30, 2025

Summary

Some iMXRT chips like iMXRT-1050 has two USB controllers, but until
now only USB1 was supported. This patch modify the driver to support
both USBs.

Impact

Users will be able to use both USBs

Testing

arcx-socket-grid board

@github-actions github-actions bot added Area: Documentation Improvements or additions to documentation Arch: arm Issues related to ARM (32-bit) architecture Board: arm Size: XL The size of the change in this PR is very large. Consider breaking down the PR into smaller pieces. labels Aug 30, 2025
Some iMXRT chips like iMXRT-1050 has two USB controllers, but until
now only USB1 was supported. This patch modify the driver to support
both USBs.

Signed-off-by: Alan C. Assis <acassis@gmail.com>
This patch adds support to both USB controllers.

Signed-off-by: Alan C. Assis <acassis@gmail.com>
Signed-off-by: Alan C. Assis <acassis@gmail.com>
@acassis
Copy link
Contributor Author

acassis commented Sep 1, 2025

@xiaoxiang781216 @hartmannathan please re-review/approve

Copy link
Contributor

@linguini1 linguini1 left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Can you elaborate in the testing section about what tests you performed? I.e. tested each USB interface enabled individually and then tested with both configured at the same time, etc?

@xiaoxiang781216 xiaoxiang781216 merged commit a503fc6 into apache:master Sep 1, 2025
26 checks passed
@acassis
Copy link
Contributor Author

acassis commented Sep 1, 2025

@linguini1 that is correct, I only focused on usbhost, the usbdev I kept same way. I will work on it later

@linguini1
Copy link
Contributor

Can you elaborate in the testing section about what tests you performed? I.e. tested each USB interface enabled individually and then tested with both configured at the same time, etc?

I trust that Alan did test this as a long-time contributor, but I think in the future PRs should be held to merge until some proof of testing or explanation of what was tested is included further. I personally don't think the testing information in this PR meets what I'd expect from other contributors.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Arch: arm Issues related to ARM (32-bit) architecture Area: Documentation Improvements or additions to documentation Board: arm Size: XL The size of the change in this PR is very large. Consider breaking down the PR into smaller pieces.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants