Move hal_nxp glue layer to zephyr repo (with leveraging MCUX SDK NEXT kconfig tree) #84366
Conversation
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
ZhaoxiangJin
changed the title
|
Hello @hakehuang, please help test, thanks. |
ZhaoxiangJin
force-pushed
the
move-mcux-glue-code-to-zephyr-repo
branch
from
f00f5d5 to
1c5f6f0
Compare
|
The following west manifest projects have changed revision in this Pull Request:
⛔ DNM label due to: 1 project with PR revision Note: This message is automatically posted and updated by the Manifest GitHub Action. |
zephyrbot
added
manifest
manifest-hal_nxp
DNM
|
+1 from me, glue layers belong in the Zephyr tree, not the module repo. |
I kick a full build first, and let's see |
ZhaoxiangJin
force-pushed
the
move-mcux-glue-code-to-zephyr-repo
branch
8 times, most recently
from
5f3ac19 to
31fbc34
Compare
The cntMode in gpc_tran_step_config_t is not used, removed it. Signed-off-by: Zhaoxiang Jin <[email protected]>
ZhaoxiangJin
force-pushed
the
move-mcux-glue-code-to-zephyr-repo
branch
2 times, most recently
from
092e470 to
f5f5517
Compare
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]>
ZhaoxiangJin
force-pushed
the
move-mcux-glue-code-to-zephyr-repo
branch
from
f5f5517 to
ddf8465
Compare
fabiobaltieri
added
DNM (manifest)
Hello @hakehuang, I have fixed all twister issues reported by CI, could you please help test again? Thanks. |
@ZhaoxiangJin what is the relation with this pr? #84423 they get the same name, also please fix below: |
ZhaoxiangJin
changed the title
Kconfig.imx, Kconfig.mcux, Kconfig.nxp_s32 have moved inside hal_nxp, need to remove them from MAINTAINERS.yml Signed-off-by: Zhaoxiang Jin <[email protected]>
Update manifest to track hal_nxp side PR Signed-off-by: Zhaoxiang Jin <[email protected]>
ZhaoxiangJin
force-pushed
the
move-mcux-glue-code-to-zephyr-repo
branch
from
ddf8465 to
23e82e9
Compare
I have updated this PR to fix this issue. PR (#84423) is a version that does not use the MCUX SDK NEXT kconfig tree, while this PR is a version that uses the MCUX SDK-NEXT kconfig tree. I have updated the titles of these two PRs to distinguish them. |
sample.display.shield v4.0.0-4307-g23e82e911eea Build failure frdm_mcxn236/mcxn236 drivers.watchdog.wdt_basic_test_suite.wdt v4.0.0-4307-g23e82e911eea Build failure frdm_ke17z/mke17z7 |
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
Version with leveraging MCUX SDK NEXT kconfig tree.