Skip to content

Sync sdk 25.06.00 to hal nxp #576

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 26 commits into from
Aug 14, 2025
Merged

Sync sdk 25.06.00 to hal nxp #576

merged 26 commits into from
Aug 14, 2025

Conversation

ZhaoxiangJin
Copy link
Collaborator

Sync sdk 25.06.00 to hal nxp

@github-actions github-actions bot mentioned this pull request Jul 18, 2025
Open
This was referenced Jul 18, 2025
Open
Merged
@ZhaoxiangJin ZhaoxiangJin force-pushed the feature/sync-sdk-25.06.00-to-hal-nxp branch 6 times, most recently from 1e1c238 to 24d3075 Compare July 21, 2025 13:05
@mmahadevan108
Copy link
Collaborator

@ZhaoxiangJin , please take a look at this PR #574 that is requiring update to newer version of the mipi-dsi driver.

@ZhaoxiangJin ZhaoxiangJin force-pushed the feature/sync-sdk-25.06.00-to-hal-nxp branch from 578fb8c to 5c335ba Compare July 24, 2025 06:58
@mmahadevan108
Copy link
Collaborator

@ZhaoxiangJin , another PR that is updating SDK drivers. Is this covered as part of this update?
#581

@ZhaoxiangJin
Copy link
Collaborator Author

@ZhaoxiangJin , another PR that is updating SDK drivers. Is this covered as part of this update? #581

Not covered, PR can merge first, then i will drop irqsteer_1 driver in my PR.

@ZhaoxiangJin ZhaoxiangJin mentioned this pull request Jul 28, 2025
Merged
@zejiang0jason
Copy link
Contributor

Can the mcux-sdk-ng/board folder be moved to Zephyr repo?

@zejiang0jason
Copy link
Contributor

Are the RT1180 board level jlink script needed?

@zejiang0jason
Copy link
Contributor

should the binary files in mcux-sdk-ng/components/conn_fwloader/fw_bin/ be placed in zephyr/modules/hal/nxp/zephyr/blobs?

@ZhaoxiangJin
Copy link
Collaborator Author

Can the mcux-sdk-ng/board folder be moved to Zephyr repo?

Why move sdk files to zephyr repo,

Are the RT1180 board level jlink script needed?

yes, see https://github.com/zephyrproject-rtos/zephyr/blob/main/boards/nxp/mimxrt1180_evk/board.cmake

@ZhaoxiangJin
Copy link
Collaborator Author

should the binary files in mcux-sdk-ng/components/conn_fwloader/fw_bin/ be placed in zephyr/modules/hal/nxp/zephyr/blobs?

The conn_fwloader CMakeList.txt needs these binary files, move these binary files requires modify conn_fwloader CMakeList.txt

@ZhaoxiangJin
Copy link
Collaborator Author

Can the mcux-sdk-ng/board folder be moved to Zephyr repo?

Why move SDK files to zephyr repo? Only glue code should be placed in zephyr repo.

@zejiang0jason
Copy link
Contributor

Can the mcux-sdk-ng/board folder be moved to Zephyr repo?

Why move SDK files to zephyr repo? Only glue code should be placed in zephyr repo.

Move to the Zephyr repo board folder, they are board specific files.

@zejiang0jason
Copy link
Contributor

Hi @ZhaoxiangJin , please note don't miss this code during migration: #577. thanks.

@ZhaoxiangJin ZhaoxiangJin force-pushed the feature/sync-sdk-25.06.00-to-hal-nxp branch from b77ba07 to 905aff8 Compare July 30, 2025 06:23
@ZhaoxiangJin ZhaoxiangJin marked this pull request as ready for review July 30, 2025 06:24
@ZhaoxiangJin
hal_nxp: Clean up mcux-sdk-ng.
0b683b6 
Signed-off-by: Zhaoxiang Jin <Zhaoxiang.Jin_1@nxp.com>
@ZhaoxiangJin
hal_nxp: Update sync_sdk_ng.py
609fcb8 
Signed-off-by: Zhaoxiang Jin <Zhaoxiang.Jin_1@nxp.com>
@ZhaoxiangJin
hal_nxp: Sync 25.06 arch to hal_nxp
ce21612 
Signed-off-by: Zhaoxiang Jin <Zhaoxiang.Jin_1@nxp.com>
@ZhaoxiangJin
hal_nxp: Sync 25.06 MCX to hal_nxp
e283bbf 
Signed-off-by: Zhaoxiang Jin <Zhaoxiang.Jin_1@nxp.com>
@ZhaoxiangJin
hal_nxp: Sync 25.06 RT to hal_nxp
623f462 
Signed-off-by: Zhaoxiang Jin <Zhaoxiang.Jin_1@nxp.com>
@ZhaoxiangJin
hal_nxp: Sync 25.06 wireless to hal_nxp
aa97faa 
Signed-off-by: Zhaoxiang Jin <Zhaoxiang.Jin_1@nxp.com>
@ZhaoxiangJin
hal_nxp: Remove SDK specific cmake code.
db2e50b 
This cmake code is not needed for zephyr, and is specific for
MCUX SDK. SDK also applied this patch.

Signed-off-by: Zhaoxiang Jin <Zhaoxiang.Jin_1@nxp.com>
@ZhaoxiangJin
hal_nxp: Remove MCXA166/A276, add MCXA266/366
2d3970f 
1. Remove MCXA166 and MCXA276 as they are renamed to
MCXA346 and MCXA266.
2. Add MCXA266 and MCXA366 header and pinctrl
3. Rename MCXA166 to MCXA346.

Signed-off-by: Peter Wang <chaoyi.wang@nxp.com>
Signed-off-by: Zhaoxiang Jin <Zhaoxiang.Jin_1@nxp.com>
@ZhaoxiangJin
hal_nxp: Add code for A core compatibility
43f59d8 
Signed-off-by: Zhaoxiang Jin <Zhaoxiang.Jin_1@nxp.com>
@ZhaoxiangJin
hal_nxp: Update dmamux, sai, flexcan driver to the SDK latest
ce080db 
Signed-off-by: Zhaoxiang Jin <Zhaoxiang.Jin_1@nxp.com>
@ZhaoxiangJin
hal_nxp: Update i.mx943 driver to the SDK latest
211a330 
Signed-off-by: Zhaoxiang Jin <Zhaoxiang.Jin_1@nxp.com>
@ZhaoxiangJin
hal_nxp: Replace the caam driver with an older version
53fea4f 
The latest version of the caam driver causes run
issues on the RT1170/RT1160 platform, using an older
version for now, and have contacted the SDK driver
owner to resolve the issue.

Signed-off-by: Zhaoxiang Jin <Zhaoxiang.Jin_1@nxp.com>
@ZhaoxiangJin ZhaoxiangJin force-pushed the feature/sync-sdk-25.06.00-to-hal-nxp branch from 6c0d41d to 53fea4f Compare August 11, 2025 05:33
@ZhaoxiangJin
Copy link
Collaborator Author

Update:
removed boards folder, moved board specific files to zephyr repo.

@mmahadevan108
Copy link
Collaborator

@ZhaoxiangJin , please include #571 as well in your update

@mmahadevan108
Copy link
Collaborator

@ZhaoxiangJin , please include #575 in your update

@dleach02 dleach02 mentioned this pull request Aug 13, 2025
Open
@github-actions github-actions bot mentioned this pull request Aug 13, 2025
Closed
@mmahadevan108
Copy link
Collaborator

Another PR that is changing mcux-sdk-ng #578

@mmahadevan108
Copy link
Collaborator

Another PR changing mcux-sdk-ng: #573

Zhiqiang-Hou and others added 5 commits August 14, 2025 09:09
@Zhiqiang-Hou @ZhaoxiangJin
fsl_common_arm: Fix comment sign issue on different architectures
b62ae5a 
The comment sign character is '//' on ARMv8 AARCH64, however it's
'@' on other ARM architectures.
So the current definitions of AT_NONCACHEABLE_SECTION_ALIGN
and AT_NONCACHEABLE_SECTION will result in the following error on
AARCH64.

  /tmp/cc96AQ7j.s: Assembler messages:
  /tmp/cc96AQ7j.s:1020: Error: junk at end of line, first unrecognized character is `@'

Signed-off-by: Hou Zhiqiang <Zhiqiang.Hou@nxp.com>
Signed-off-by: Stephane Viau <stephane.viau@nxp.com>
Signed-off-by: Jiafei Pan <Jiafei.Pan@nxp.com>
@JiafeiPan @ZhaoxiangJin
mcux-sdk: MIMX9596_ca55: fix A-Core header file
eb11068 
NETC driver is not compatible with struct definition, for example will
report the following build error like:
modules/hal/nxp/mcux/mcux-sdk-ng/drivers/netc/fsl_netc_timer.h:369:20:
error: 'ENETC_PF_TMR_Type' has no member named 'TMR_TEVENT'

Signed-off-by: Jiafei Pan <Jiafei.Pan@nxp.com>
@JiafeiPan @ZhaoxiangJin
mcux-sdk: MIMX9596_ca55: disable FSL_FEATURE_NETC_HAS_ERRATA_051711
8fc3f54 
This Errata is not available on i.MX 95.

Signed-off-by: Jiafei Pan <Jiafei.Pan@nxp.com>
@JiafeiPan @ZhaoxiangJin
drivers: netc: make the driver to be aarch64 compatible
2ad8f70 
Signed-off-by: Jiafei Pan <Jiafei.Pan@nxp.com>
@yangbolu1991 @ZhaoxiangJin
mcux-sdk-ng: update netc driver to the latest
b5688b6 
Updated netc driver to the latest SDK commit.

5309b12e [MCUX-80906] drivers: netc: enable ring0 for EP supporting tag

Signed-off-by: Yangbo Lu <yangbo.lu@nxp.com>
@ZhaoxiangJin
Copy link
Collaborator Author

Hello @mmahadevan108, #581, #577, #575, #571 have picked to my PR. #573, #574, #578 are pending in review.

@mmahadevan108
Copy link
Collaborator

@ZhaoxiangJin I do not see any change mcux-sdk-ng /drivers/. Can you please double check?

@mmahadevan108
Copy link
Collaborator

Hello @mmahadevan108, #581, #577, #575, #571 have picked to my PR. #573, #574, #578 are pending in review.

If #573, #574, #578 look fine to the SDK team, then please include in your PR and we can close them.

@ZhaoxiangJin
Copy link
Collaborator Author

@ZhaoxiangJin I do not see any change mcux-sdk-ng /drivers/. Can you please double check?

This commit update drivers?

@ZhaoxiangJin
Copy link
Collaborator Author

Hello @mmahadevan108, #581, #577, #575, #571 have picked to my PR. #573, #574, #578 are pending in review.

If #573, #574, #578 look fine to the SDK team, then please include in your PR and we can close them.

I am ok with these PRs, you can close them, then i pick these changes to this PR

@dleach02
Copy link
Member

we will merge this one and let the smaller ones deal with any merge conflicts (if any)

@dleach02 dleach02 merged commit 54ebe9b into zephyrproject-rtos:master Aug 14, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@dleach02 dleach02 dleach02 approved these changes

@mmahadevan108 mmahadevan108 Awaiting requested review from mmahadevan108

@decsny decsny Awaiting requested review from decsny

Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
7 participants
@ZhaoxiangJin @mmahadevan108 @zejiang0jason @dleach02 @Zhiqiang-Hou @JiafeiPan @yangbolu1991