Skip to content

Supports hal_nxp of MCUX SDK 25.06.00 #93306

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

Open
wants to merge 8 commits into
base: main
Choose a base branch
from

Conversation

ZhaoxiangJin
Copy link
Contributor

Update hal nxp to mcux sdk 25.06.00.

Copy link

github-actions bot commented Jul 18, 2025

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

Name Old Revision New Revision Diff
hal_nxp zephyrproject-rtos/hal_nxp@10f1faf zephyrproject-rtos/hal_nxp@54ebe9b (master) zephyrproject-rtos/hal_nxp@10f1faf4..54ebe9bf

All manifest checks OK

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 Jul 18, 2025
@ZhaoxiangJin ZhaoxiangJin reopened this Jul 18, 2025
@ZhaoxiangJin ZhaoxiangJin force-pushed the feature/test-hal-nxp-update branch from 3d12908 to 41f8800 Compare July 18, 2025 14:43
@ZhaoxiangJin ZhaoxiangJin reopened this Jul 21, 2025
@ZhaoxiangJin ZhaoxiangJin force-pushed the feature/test-hal-nxp-update branch 4 times, most recently from 9ff95b8 to 8ca95a8 Compare July 28, 2025 06:17
@ZhaoxiangJin ZhaoxiangJin force-pushed the feature/test-hal-nxp-update branch 3 times, most recently from e30e2a4 to 8a23c6a Compare July 30, 2025 06:24
@ZhaoxiangJin ZhaoxiangJin changed the title Update hal nxp to mcux sdk 25.06.00 Supports hal_nxp of MCUX SDK 25.06.00 Jul 30, 2025
@ZhaoxiangJin ZhaoxiangJin marked this pull request as ready for review July 30, 2025 07:47
@zejiang0jason
Copy link
Contributor

The changes are split into several patches, do we need to merge the necessary patches, to make every snapshot buildable?

@ZhaoxiangJin ZhaoxiangJin force-pushed the feature/test-hal-nxp-update branch from c112334 to 5c359d4 Compare August 11, 2025 05:34
@ZhaoxiangJin
Copy link
Contributor Author

Update:

  1. Move board specific files (dcd, flash_config, etc.) from nxp hal repo to zephyr repo.
  2. Removed frdm_mcxa166 and frdm_mcxa276 from release-notes-4.2.rst.

@ZhaoxiangJin ZhaoxiangJin marked this pull request as draft August 11, 2025 06:37
@ZhaoxiangJin ZhaoxiangJin force-pushed the feature/test-hal-nxp-update branch from 5c359d4 to 9e64dd7 Compare August 11, 2025 07:09
@dleach02
Copy link
Member

@ZhaoxiangJin please address the compliance failures

@ZhaoxiangJin
Copy link
Contributor Author

ZhaoxiangJin commented Aug 13, 2025

@ZhaoxiangJin please address the compliance failures

Fix is on going, but there are a lot of compliance issues that need to be resolved, I need more time to fix them...

@zejiang0jason
Copy link
Contributor

Hi @dleach02 , the compliance issue is because we are moving the board specific dcd.c, flexspi_nor_config.c to zephyr board folder. There are many coding style issue in these files.

@ZhaoxiangJin ZhaoxiangJin force-pushed the feature/test-hal-nxp-update branch 4 times, most recently from d174486 to 1af35a7 Compare August 14, 2025 00:50
@dleach02 dleach02 marked this pull request as ready for review August 14, 2025 16:43
@dleach02 dleach02 force-pushed the feature/test-hal-nxp-update branch from 1af35a7 to 1d2ba0e Compare August 14, 2025 19:56
These variables are used to make MCUX SDK NG cmake
can find the right cmake files.

Signed-off-by: Zhaoxiang Jin <Zhaoxiang.Jin_1@nxp.com>
'MCUX_HW_CORE' and 'MCUX_HW_FPU_TYPE' are used in
modules/hal_nxp/device.cmake. It is a variable used
by MCUX SDK CMake.

Signed-off-by: Zhaoxiang Jin <Zhaoxiang.Jin_1@nxp.com>
@github-actions github-actions bot removed the DNM (manifest) This PR should not be merged (controlled by action-manifest) label Aug 14, 2025
ZhaoxiangJin and others added 6 commits August 14, 2025 14:58
Update hal_nxp to mcux sdk 25.06.00.

Signed-off-by: Zhaoxiang Jin <Zhaoxiang.Jin_1@nxp.com>
Now use the 5-parameter function "PINT_PinInterruptConfig"
deprecated in MCUX SDK, need to add compile definition
'PINT_USE_LEGACY_CALBACK' to make intc_nxp_pint compatible
with updated 'fsl_pint' driver.

Signed-off-by: Zhaoxiang Jin <Zhaoxiang.Jin_1@nxp.com>
1. Add core_id for RT685 HIFI4 core, this variable is
used in MCUX SDK cmake.
2. Set cmake variables 'CONFIG_MCUX_COMPONENT_driver.smartdma_mcxn'
and 'CONFIG_MCUX_COMPONENT_driver.smartdma_rt500' to "ON" as these
variables control the loading of source files in the latest driver
CMakeList.txt.

Signed-off-by: Zhaoxiang Jin <Zhaoxiang.Jin_1@nxp.com>
1. Renamed MCXA166 to MCXA346.
2. NXP frdm_mcxa166 is renamed to frdm_mcxa346,
add this information to migration-guide-4.3.rst.

Signed-off-by: Zhaoxiang Jin <Zhaoxiang.Jin_1@nxp.com>
1. Renamed MCXA276 to MCXA266
2. NXP frdm_mcxa276 is renamed to frdm_mcxa266,
add this information to migration-guide-4.3.rst.

Signed-off-by: Peter Wang <chaoyi.wang@nxp.com>
Move board specific files (dcd, flash_config, etc.) from
nxp hal repo to zephyr repo.

Signed-off-by: Zhaoxiang Jin <Zhaoxiang.Jin_1@nxp.com>
@dleach02 dleach02 force-pushed the feature/test-hal-nxp-update branch from 1d2ba0e to 4028d40 Compare August 14, 2025 19:58
Copy link

@dleach02
Copy link
Member

@hakehuang can we run one final regression against this PR. If it passes then please apply your +1

@hakehuang
Copy link
Contributor

@hakehuang can we run one final regression against this PR. If it passes then please apply your +1

testing is in progress, will feedback once done

Copy link
Contributor

@hakehuang hakehuang left a comment

Choose a reason for hiding this comment

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

board testing and regression build pass

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

7 participants