Skip to content

Update hal_nxp to MCUX SDK latest release #99715

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
kartben merged 1 commit into from
Dec 3, 2025
Merged

Update hal_nxp to MCUX SDK latest release #99715

kartben merged 1 commit into from
Dec 3, 2025

Conversation

@ZhaoxiangJin
Copy link
Contributor

@ZhaoxiangJin ZhaoxiangJin commented Nov 20, 2025

update hal_nxp to MCUX SDK latest release (25.12.00 pvw2)

@github-actions
Copy link

github-actions bot commented Nov 20, 2025 •
edited by ZhaoxiangJin
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@6d6ab91 zephyrproject-rtos/hal_nxp#638 zephyrproject-rtos/hal_nxp#630/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 Nov 20, 2025
@github-actions
Copy link

github-actions bot commented Nov 20, 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@6f58abe zephyrproject-rtos/hal_nxp@ee1ab8a (master) zephyrproject-rtos/hal_nxp@6f58abea..ee1ab8aa

✅ All manifest checks OK

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

@hakehuang hakehuang added the block: HW Test Testing on hardware required before merging label Nov 20, 2025
@hakehuang
Copy link
Contributor

hakehuang commented Nov 21, 2025 •
edited
Loading

@ZhaoxiangJin mimxrt1160_evk network cases all failed, please take a look which is ok in mainline

@ZhaoxiangJin
Copy link
Contributor Author

ZhaoxiangJin commented Nov 21, 2025

@ZhaoxiangJin mimxrt1160_evk network cases all failed, please take a look which is ok in mainline

Thanks, I will look into this.

@ZhaoxiangJin
Copy link
Contributor Author

ZhaoxiangJin commented Nov 24, 2025

@ZhaoxiangJin mimxrt1160_evk network cases all failed, please take a look which is ok in mainline

Hello @hakehuang, I have updated the hal_nxp side PR to fix this issue, could you please help to test mimxrt1160_evk network cases again? Thanks.

@hakehuang
Copy link
Contributor

hakehuang commented Nov 24, 2025

Hello @hakehuang, I have updated the hal_nxp side PR to fix this issue, could you please help to test mimxrt1160_evk network cases again? Thanks.

@ZhaoxiangJin so you change only related to mimxrt1160 or any other platform need run regression as well?

@ZhaoxiangJin
Copy link
Contributor Author

ZhaoxiangJin commented Nov 24, 2025

Hello @hakehuang, I have updated the hal_nxp side PR to fix this issue, could you please help to test mimxrt1160_evk network cases again? Thanks.

@ZhaoxiangJin so you change only related to mimxrt1160 or any other platform need run regression as well?

I changed CAAM driver, this driver is used on RT116x and RT117x, if possible, please help to run regression on these platforms.

@hakehuang hakehuang removed the block: HW Test Testing on hardware required before merging label Nov 25, 2025
@hakehuang
Copy link
Contributor

hakehuang commented Nov 25, 2025

I changed CAAM driver, this driver is used on RT116x and RT117x, if possible, please help to run regression on these platforms.

@ZhaoxiangJin I regression RT1160 and rt1170, no issue found. the net tests are fixed in mimxrt1160_evk. v4.3.0-935-gf09927530eb9

@hakehuang hakehuang added the block: HW Test Testing on hardware required before merging label Nov 26, 2025
@hakehuang
Copy link
Contributor

hakehuang commented Nov 26, 2025 •
edited
Loading

start another round of regression. no new issues found. regression on v4.3.0-1113-g388c4d8db0d3

@hakehuang hakehuang removed the block: HW Test Testing on hardware required before merging label Nov 27, 2025
@ZhaoxiangJin ZhaoxiangJin marked this pull request as ready for review November 27, 2025 07:26
@zephyrbot zephyrbot added platform: NXP NXP size: XS A PR changing only a single line of code labels Nov 27, 2025
@mmahadevan108
Copy link
Contributor

mmahadevan108 commented Dec 1, 2025

@hakehuang, please approve if this PR passes your testing

@ZhaoxiangJin
Copy link
Contributor Author

ZhaoxiangJin commented Dec 2, 2025

Hello @JiafeiPan, we have also passed the MPU side regression test, please apply your approval. Thanks.

JiafeiPan
JiafeiPan previously approved these changes Dec 2, 2025
View reviewed changes
Holt-Sun
Holt-Sun previously approved these changes Dec 2, 2025
View reviewed changes
hakehuang
hakehuang previously approved these changes Dec 2, 2025
View reviewed changes
@hakehuang
Copy link
Contributor

hakehuang commented Dec 2, 2025

@hakehuang, please approve if this PR passes your testing

sure. approved

@ZhaoxiangJin
west.yml: update hal_nxp to MCUX SDK latest release
9f00897 
update hal_nxp to MCUX SDK latest release (25.12.00 pvw2)

Signed-off-by: Zhaoxiang Jin <Zhaoxiang.Jin_1@nxp.com>
@ZhaoxiangJin ZhaoxiangJin dismissed stale reviews from hakehuang, Holt-Sun, and JiafeiPan via 9f00897 December 3, 2025 06:22
@github-actions github-actions bot removed the DNM (manifest) This PR should not be merged (controlled by action-manifest) label Dec 3, 2025
@sonarqubecloud
Copy link

sonarqubecloud bot commented Dec 3, 2025

Quality Gate Passed Quality Gate passed

Issues
0 New issues
0 Accepted issues

Measures
0 Security Hotspots
0.0% Coverage on New Code
0.0% Duplication on New Code

See analysis details on SonarQube Cloud

@ZhaoxiangJin
Copy link
Contributor Author

ZhaoxiangJin commented Dec 3, 2025

Hello @hakehuang, @JiafeiPan, @Holt-Sun, I just updated the west.yml, please help to apply your approval again. Thanks.

@Holt-Sun Holt-Sun self-requested a review December 3, 2025 07:19
@kartben kartben merged commit 10b5297 into zephyrproject-rtos:main Dec 3, 2025
28 checks passed
@ZhaoxiangJin ZhaoxiangJin deleted the update-hal-nxp branch December 3, 2025 11:59
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

@hakehuang hakehuang hakehuang approved these changes

@Holt-Sun Holt-Sun Holt-Sun approved these changes

@zejiang0jason zejiang0jason zejiang0jason approved these changes

@JiafeiPan JiafeiPan JiafeiPan left review comments

@axelnxp axelnxp Awaiting requested review from axelnxp

@bperseghetti bperseghetti Awaiting requested review from bperseghetti

@decsny decsny Awaiting requested review from decsny

@dleach02 dleach02 Awaiting requested review from dleach02

@EmilioCBen EmilioCBen Awaiting requested review from EmilioCBen

@manuargue manuargue Awaiting requested review from manuargue

@MaochenWang1 MaochenWang1 Awaiting requested review from MaochenWang1

@mmahadevan108 mmahadevan108 Awaiting requested review from mmahadevan108

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

Assignees

@mmahadevan108 mmahadevan108

@dleach02 dleach02

@decsny decsny

@ZhaoxiangJin ZhaoxiangJin

Labels

manifest manifest-hal_nxp platform: NXP NXP size: XS A PR changing only a single line of code

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

10 participants

@ZhaoxiangJin @hakehuang @mmahadevan108 @Holt-Sun @JiafeiPan @zejiang0jason @kartben @zephyrbot @dleach02 @decsny