Skip to content
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

Move hal_nxp glue layer to zephyr repo #84366

Draft
wants to merge 6 commits into
base: main
Choose a base branch
from

Conversation

ZhaoxiangJin
Copy link
Contributor

@ZhaoxiangJin ZhaoxiangJin commented Jan 22, 2025

Version with leveraging MCUX SDK NEXT kconfig tree.

@ZhaoxiangJin ZhaoxiangJin changed the title Move mcux glue code to zephyr repo Move hal_nxp glue layer to zephyr repo Jan 22, 2025
@ZhaoxiangJin
Copy link
Contributor Author

Hello @hakehuang, please help test, thanks.

@ZhaoxiangJin ZhaoxiangJin force-pushed the move-mcux-glue-code-to-zephyr-repo branch from f00f5d5 to 1c5f6f0 Compare January 22, 2025 10:02
@zephyrbot
Copy link
Collaborator

zephyrbot commented Jan 22, 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@49ff7e3 (master) zephyrproject-rtos/hal_nxp#500 zephyrproject-rtos/hal_nxp#500/files

DNM label due to: 1 project with PR revision

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

@zephyrbot zephyrbot added manifest manifest-hal_nxp DNM This PR should not be merged (Do Not Merge) labels Jan 22, 2025
@carlescufi
Copy link
Member

+1 from me, glue layers belong in the Zephyr tree, not the module repo.

@hakehuang
Copy link
Collaborator

Hello @hakehuang, please help test, thanks.

I kick a full build first, and let's see

@ZhaoxiangJin ZhaoxiangJin force-pushed the move-mcux-glue-code-to-zephyr-repo branch 8 times, most recently from 5f3ac19 to 31fbc34 Compare January 26, 2025 05:21
The cntMode in gpc_tran_step_config_t is not used, removed it.

Signed-off-by: Zhaoxiang Jin <[email protected]>
@ZhaoxiangJin ZhaoxiangJin force-pushed the move-mcux-glue-code-to-zephyr-repo branch 2 times, most recently from 092e470 to f5f5517 Compare January 31, 2025 02:01
Add cortex-m core suffix item for imx devices

Signed-off-by: Zhaoxiang Jin <[email protected]>
The XSPI features was updated in MCUXSDK, need to update
zephyr side code to avoiding build error.

Signed-off-by: Zhaoxiang Jin <[email protected]>
second core kconfig file

CONFIG_SECOND_CORE_MCUX is not set to y when building
the RT1160 and RT1170 second core synchronization
examples. It is better to set this kconfig item to y at
the board layer.

Signed-off-by: Zhaoxiang Jin <[email protected]>
Move hal_nxp glue layer to zephyr repo.

Signed-off-by: Zhaoxiang Jin <[email protected]>
Update manifest to track hal_nxp side PR

Signed-off-by: Zhaoxiang Jin <[email protected]>
@ZhaoxiangJin ZhaoxiangJin force-pushed the move-mcux-glue-code-to-zephyr-repo branch from f5f5517 to ddf8465 Compare January 31, 2025 04:53
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
DNM This PR should not be merged (Do Not Merge) manifest manifest-hal_nxp
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants