Skip to content

nxp: usb: zephyr support on imx93 frdm Cortex-A Core #97320

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
Open
wants to merge 7 commits into
base: main
Choose a base branch
from
Open

nxp: usb: zephyr support on imx93 frdm Cortex-A Core #97320

wants to merge 7 commits into from
+271 −19

Conversation

JonyZhang7
Copy link

@JonyZhang7 JonyZhang7 commented Oct 10, 2025 •
edited by JiafeiPan
Loading

nxp: add usb support on frdm-imx93

Depends on zephyrproject-rtos/hal_nxp#615

@github-actions GitHub Actions
Copy link

Hello @JonyZhang7, and thank you very much for your first pull request to the Zephyr project!
Our Continuous Integration pipeline will execute a series of checks on your Pull Request commit messages and code, and you are expected to address any failures by updating the PR. Please take a look at our commit message guidelines to find out how to format your commit messages, and at our contribution workflow to understand how to update your Pull Request. If you haven't already, please make sure to review the project's Contributor Expectations and update (by amending and force-pushing the commits) your pull request if necessary.
If you are stuck or need help please join us on Discord and ask your question there. Additionally, you can escalate the review when applicable. 😊

tmon-nordic
tmon-nordic reviewed Oct 10, 2025
View reviewed changes
subsys/usb/device_next/usbd_ch9.c
}

LOG_DBG("wLength %u, bLength %u, tailroom %u",
LOG_DBG("wLength %u, bLength %u, tailroom %zu",
Copy link
Contributor

Choose a reason for hiding this comment

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

I think this commit should go in its own PR. Can you also provide better PR title so it is immediately clear that this adds support for new target?

@JonyZhang7 JonyZhang7 changed the title Jony usb upstream nxp: usb: add frdm-imx93 support Oct 10, 2025
@JonyZhang7 JonyZhang7 changed the title nxp: usb: add frdm-imx93 support nxp: usb: zephyr support on imx93 frdm Cortex-A Core Oct 10, 2025
@decsny decsny removed their request for review October 10, 2025 13:58
@github-actions GitHub Actions
Copy link

github-actions bot commented Oct 11, 2025 •
edited
Loading

The following west manifest projects have changed revision in this Pull Request:

Name Old Revision New Revision Diff
hal_nxp zephyrproject-rtos/hal_nxp@4377ecf zephyrproject-rtos/hal_nxp#615 zephyrproject-rtos/hal_nxp#615/files

⛔ DNM label due to: 1 project with PR revision

Note: This message is automatically posted and updated by the Manifest GitHub Action.

@github-actions github-actions bot added manifest manifest-hal_nxp DNM (manifest) This PR should not be merged (controlled by action-manifest) labels Oct 11, 2025
@zephyrbot zephyrbot requested a review from decsny October 11, 2025 09:12
@JonyZhang7
west.yml: depend on hal_nxp pr 615
35f8f79 
Depend on hal_nxp PR 615.

Signed-off-by: qizhang <[email protected]>
@JonyZhang7 JonyZhang7 force-pushed the jony_usb_upstream branch 2 times, most recently from b94da40 to 0c880ca Compare October 13, 2025 10:36
@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.9% Duplication on New Code

See analysis details on SonarQube Cloud

WinteriWang and others added 6 commits October 13, 2025 21:57
@WinteriWang @JonyZhang7
drivers: clock: ccm_rev2: add imx93 common clocks set support
676e539 
Setup most clocks with common_clock_set_freq().
PLL and mux are preset.

Signed-off-by: Winteri Wang <[email protected]>
Signed-off-by: qizhang <[email protected]>
@JasonHe-nxp @JonyZhang7
drivers: clock: mcux_ccm: add usb clock support
8e0e906 
Add usb clock support.

Signed-off-by: Jason He <[email protected]>
Signed-off-by: Jiafei Pan <[email protected]>
Signed-off-by: qizhang <[email protected]>
@JiafeiPan @JonyZhang7
modules: hal_nxp: fix camke for sdk-ng middleware
bed5287 
As middleware.cmake could be included by mcux-sdk's CMakeLists.txt,
so use CMAKE_CURRENT_LIST_DIR to make sure the directory is correct.

Signed-off-by: Jiafei Pan <[email protected]>
Signed-off-by: qizhang <[email protected]>
@JiafeiPan @JonyZhang7
usb: fix build warning for printing format
61bafec 
Use zu% format for size_t type.

Signed-off-by: Jiafei Pan <[email protected]>
Signed-off-by: qizhang <[email protected]>
@JasonHe-nxp @JonyZhang7
drivers: usb: mcux_ehci: enable support to imx93
08ecea3 
Enable support on i.MX93.

Signed-off-by: Jason He <[email protected]>
Signed-off-by: Jiafei Pan <[email protected]>
Signed-off-by: qizhang <[email protected]>
@JasonHe-nxp @JonyZhang7
boards: frdm_imx93: enable usb support
0c880ca 
Enable USB support on i.MX93 FRDM.

Signed-off-by: Jason He <[email protected]>
Signed-off-by: Jiafei Pan <[email protected]>
Signed-off-by: qizhang <[email protected]>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

@tmon-nordic tmon-nordic tmon-nordic left review comments

@bperseghetti bperseghetti Awaiting requested review from bperseghetti

@dbaluta dbaluta Awaiting requested review from dbaluta

@dleach02 dleach02 Awaiting requested review from dleach02

@iuliana-prodan iuliana-prodan Awaiting requested review from iuliana-prodan

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

@JiafeiPan JiafeiPan Awaiting requested review from JiafeiPan

@manuargue manuargue Awaiting requested review from manuargue

@MarkWangChinese MarkWangChinese Awaiting requested review from MarkWangChinese

@mmahadevan108 mmahadevan108 Awaiting requested review from mmahadevan108

@nordic-krch nordic-krch Awaiting requested review from nordic-krch

@PetervdPerk-NXP PetervdPerk-NXP Awaiting requested review from PetervdPerk-NXP

@yangbolu1991 yangbolu1991 Awaiting requested review from yangbolu1991

@Zhiqiang-Hou Zhiqiang-Hou Awaiting requested review from Zhiqiang-Hou

@ZhaoxiangJin ZhaoxiangJin Awaiting requested review from ZhaoxiangJin

@decsny decsny Awaiting requested review from decsny

At least 2 approving reviews are required to merge this pull request.

Assignees

@nordic-krch nordic-krch

Labels

area: ARM64 ARM (64-bit) Architecture area: Boards/SoCs area: Clock Control area: Devicetree Bindings area: USB Universal Serial Bus DNM (manifest) This PR should not be merged (controlled by action-manifest) manifest manifest-hal_nxp platform: NXP MPU platform: NXP NXP

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

7 participants

@JonyZhang7 @tmon-nordic @nordic-krch @zephyrbot @WinteriWang @JasonHe-nxp @JiafeiPan