Skip to content

Add mcxa153 usb support #90577

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.

Sign up for GitHub

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

Jump to bottom
Merged
merged 3 commits into from
May 29, 2025
Merged

Add mcxa153 usb support #90577

merged 3 commits into from
May 29, 2025

Conversation

NeilChen93
Copy link
Collaborator

Add mcxa153 usb support

@NeilChen93
dts: arm/nxp: Add usb nodes to NXP MCXA153 dtsi file
9ee7dce 
Add usb nodes to NXP MCXA153 dtsi file

Signed-off-by: Neil Chen <[email protected]>
@github-actions github-actions bot added platform: NXP NXP area: Samples Samples area: USB Universal Serial Bus labels May 26, 2025
tmon-nordic
tmon-nordic reviewed May 26, 2025
View reviewed changes
@NeilChen93 NeilChen93 requested a review from hakehuang May 26, 2025 10:19
@kartben kartben assigned mmahadevan108 and dleach02 and unassigned kartben May 26, 2025
hakehuang
hakehuang reviewed May 27, 2025
View reviewed changes
boards/nxp/frdm_mcxa153/frdm_mcxa153.yaml
@@ -24,5 +24,7 @@ supported:
- pwm
- spi
- uart
- usb_device
Copy link
Collaborator

Choose a reason for hiding this comment

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

support both new and old usb stack?

Copy link
Collaborator Author

Choose a reason for hiding this comment

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

thanks for reminder, only left usb_device.

NeilChen93 added 2 commits May 27, 2025 11:29
@NeilChen93
boards: nxp: frdm_mcxa153: Support usb for NXP frdm_mcxa153 board
aad36ba 
Support usb for NXP frdm_mcxa153 board.
Test it using samples/subsys/usb/console.

Signed-off-by: Neil Chen <[email protected]>
@NeilChen93
samples: usb: Exclude frdm_mcxa153 platform from dfu sample
e90aca1 
Exclude the platform frdm_mcxa153 from dfu sample.

Signed-off-by: Neil Chen <[email protected]>
@NeilChen93 NeilChen93 force-pushed the add_mcxa153_usb_support branch from 36d636d to e90aca1 Compare May 27, 2025 03:30
@hakehuang
Copy link
Collaborator

board testing pass on v4.1.0-4636-g36d636d1c9f3

INFO    - JOBS: 8
INFO    - Adding tasks to the queue...
INFO    - Added initial list of jobs to queue
INFO    - 1/8 frdm_mcxa153/mcxa153      tests/subsys/usb/bos/usb.bos                       PASSED (device: D4ECYWWUMFIGV, 39.253s <zephyr>)
INFO    -                                    usb.bos.osdesc_bos.osdesc                                                   PASSED
INFO    - 2/8 frdm_mcxa153/mcxa153      tests/subsys/usb/desc_sections/usb.desc_sections   PASSED (device: D4ECYWWUMFIGV, 10.327s <zephyr>)
INFO    -                                    usb.desc_sections.desc_sections.desc_sections                               PASSED
INFO    - 3/8 frdm_mcxa153/mcxa153      samples/subsys/usb/audio/headphones_microphone/sample.usb.audio.headphones_microphone PASSED (device: D4ECYWWUMFIGV, 10.396s <zephyr>)
INFO    -                                    sample.usb.audio.headphones_microphone                                      PASSED
INFO    - 4/8 frdm_mcxa153/mcxa153      samples/subsys/usb/midi/sample.usb_device_next.midi PASSED (device: D4ECYWWUMFIGV, 10.410s <zephyr>)
INFO    -                                    sample.usb_device_next.midi                                                 PASSED
INFO    - 5/8 frdm_mcxa153/mcxa153      samples/subsys/usb/audio/headset/sample.usb.audio.headset PASSED (device: D4ECYWWUMFIGV, 10.353s <zephyr>)
INFO    -                                    sample.usb.audio.headset                                                    PASSED
INFO    - 6/8 frdm_mcxa153/mcxa153      tests/subsys/usb/device/usb.device                 PASSED (device: D4ECYWWUMFIGV, 10.423s <zephyr>)
INFO    -                                    usb.device.device_usb.usb_dc_api                                            PASSED
INFO    -                                    usb.device.device_usb.usb_disable                                           PASSED
INFO    -                                    usb.device.device_usb.usb_deconfig                                          PASSED
INFO    -                                    usb.device.device_usb.usb_dc_api_invalid                                    PASSED
INFO    -                                    usb.device.device_usb.usb_dc_api_read_write                                 PASSED
INFO    - 7/8 frdm_mcxa153/mcxa153      samples/subsys/usb/cdc_acm/sample.usb_device_next.cdc-acm PASSED (device: D4ECYWWUMFIGV, 10.416s <zephyr>)
INFO    -                                    sample.usb_device_next.cdc-acm                                              PASSED
INFO    - 8/8 frdm_mcxa153/mcxa153      samples/subsys/usb/cdc_acm/sample.usb.cdc-acm      PASSED (device: D4ECYWWUMFIGV, 10.355s <zephyr>)
INFO    -                                    sample.usb.cdc-acm                                                          PASSED

@sonarqubecloud SonarQubeCloud
Copy link

Quality Gate Passed Quality Gate passed

Issues
0 New issues
0 Accepted issues

Measures
0 Security Hotspots
0.0% Coverage on New Code
0.0% Duplication on New Code

See analysis details on SonarQube Cloud

decsny
decsny approved these changes May 27, 2025
View reviewed changes
Copy link
Member

@decsny decsny left a comment

Choose a reason for hiding this comment

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

please do whole board enablement on one PR, these tiny additions to overlays is really not worth it's own PR adding noise. I have over 100 new PRs to review every day and these type of ones really are a big culprit for that, a waste of time for reviewers compared to this whole overlay just being added on one PR instead of like one line at a time

@kartben kartben merged commit 13d8ca5 into zephyrproject-rtos:main May 29, 2025
27 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@hakehuang hakehuang hakehuang approved these changes

@dleach02 dleach02 dleach02 approved these changes

@decsny decsny decsny approved these changes

@tmon-nordic tmon-nordic tmon-nordic approved these changes

@butok butok Awaiting requested review from butok

@DerekSnell DerekSnell Awaiting requested review from DerekSnell

@EmilioCBen EmilioCBen Awaiting requested review from EmilioCBen

@jfischer-no jfischer-no Awaiting requested review from jfischer-no

@kartben kartben Awaiting requested review from kartben

@mmahadevan108 mmahadevan108 Awaiting requested review from mmahadevan108

@nashif nashif Awaiting requested review from nashif

Assignees

@mmahadevan108 mmahadevan108

@dleach02 dleach02

Labels
area: Samples Samples area: USB Universal Serial Bus platform: NXP NXP
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
7 participants
@NeilChen93 @hakehuang @dleach02 @decsny @tmon-nordic @kartben @mmahadevan108