Skip to content

nxp: mcxw7x: integrate MCUXSDK 25.09 blobs #98030

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 3 commits into from
Oct 23, 2025
Merged

nxp: mcxw7x: integrate MCUXSDK 25.09 blobs #98030

merged 3 commits into from
Oct 23, 2025
+35 −15

Conversation

@mmahadevan108
Copy link
Contributor

@mmahadevan108 mmahadevan108 commented Oct 21, 2025 •
edited by George-Stefan
Loading

  1. Update MCUX 25.09 blobs for mcxw71/mcxw72
  2. Update shared memory placement for mcxw72

Created this PR as previous github PR #97814 got stuck after updating the HAL SHA
Associated HAL PR zephyrproject-rtos/hal_nxp#625

  1. Update ieee802154_mcxw driver
@github-actions
Copy link

github-actions bot commented Oct 21, 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@6adb4c5 zephyrproject-rtos/hal_nxp@7bf6f09 zephyrproject-rtos/hal_nxp@6adb4c50..7bf6f09c

Additional metadata changed:

Name URL Submodules West cmds module.yml Blobs
hal_nxp ✏ 4x ✏

⛔ DNM label due to: 1 project with metadata changes and 4 blob changes

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) Binary Blobs Modified labels Oct 21, 2025
@mmahadevan108 mmahadevan108 removed the DNM (manifest) This PR should not be merged (controlled by action-manifest) label Oct 21, 2025
@mmahadevan108
Copy link
Contributor Author

@jhedberg , #97814 has been open for 4 days. So technically this has been open for that duration as well. However it will not show up in the merge queue.
I would appreciate if you could help merge this PR once it passes CI and has the approvals.
dleach02
dleach02 previously approved these changes Oct 22, 2025
View reviewed changes
ZhaoxiangJin
ZhaoxiangJin previously approved these changes Oct 22, 2025
View reviewed changes
axelnxp
axelnxp previously approved these changes Oct 22, 2025
View reviewed changes
@jhedberg
Copy link
Member

@jhedberg , #97814 has been open for 4 days. So technically this has been open for that duration as well. However it will not show up in the merge queue. I would appreciate if you could help merge this PR once it passes CI and has the approvals.

There will be time to get this merged through the normal process before rc1, so I'm not sure such an exception is warranted (I'm also not aware of such exceptions having been made in the past). @zephyrproject-rtos/release FYI
@jhedberg
Copy link
Member

Created this PR as previous github PR #97814 got stuck after updating the HAL SHA
Associated HAL PR zephyrproject-rtos/hal_nxp#625

@mmahadevan108 what exactly do you mean by "got stuck"? Do we have some bug in our infrastructure or CI that needs fixing? Force pushing and closing/reopening didn't help?
@George-Stefan George-Stefan dismissed stale reviews from axelnxp, ZhaoxiangJin, and dleach02 via d70c237 October 22, 2025 10:19
@github-actions github-actions bot added the DNM (manifest) This PR should not be merged (controlled by action-manifest) label Oct 22, 2025
@mmahadevan108
Copy link
Contributor Author

Created this PR as previous github PR #97814 got stuck after updating the HAL SHA
Associated HAL PR zephyrproject-rtos/hal_nxp#625

@mmahadevan108 what exactly do you mean by "got stuck"? Do we have some bug in our infrastructure or CI that needs fixing? Force pushing and closing/reopening didn't help?

Might have been a github issue, there was a message at the top of the PR that kept saying updating and nothing was happening. I waited for a few hours before closing it and opening a new PR.
@mmahadevan108 mmahadevan108 removed the DNM (manifest) This PR should not be merged (controlled by action-manifest) label Oct 22, 2025
@jhedberg jhedberg added this to the v4.3.0 milestone Oct 22, 2025
@dleach02
Copy link
Member

dleach02 commented Oct 23, 2025 •
edited
Loading

@jhedberg , #97814 has been open for 4 days. So technically this has been open for that duration as well. However it will not show up in the merge queue. I would appreciate if you could help merge this PR once it passes CI and has the approvals.

There will be time to get this merged through the normal process before rc1, so I'm not sure such an exception is warranted (I'm also not aware of such exceptions having been made in the past). @zephyrproject-rtos/release FYI

@jhedberg The reason we are asking for this is because we have 3 more PRs lined up with HAL updates we are trying to coordinate

But let me do a quick check on the other PR to see if I can figure out why it was problematic
axelnxp and others added 3 commits October 23, 2025 10:04
@axelnxp @dleach02
manifest: update hal_nxp
4b246d0 
Update MCXW7x nbu_ble blobs for MCUXSDK 25.09. Signed-off-by: Axel Le Bourhis <axel.lebourhis@nxp.com>
@axelnxp @dleach02
soc: nxp: mcxw72: update shared memory placement
ccd9cfe 
mcxw72 shared memory placement has been changed with MCUXSDK 25.09 update. Moving the shared memory declaration to mcxw71 and mcxw72 specific dts since the placement is now different. Signed-off-by: Axel Le Bourhis <axel.lebourhis@nxp.com>
@George-Stefan @dleach02
drivers: ieee802154: mcxw: disable poll optimization
7104638 
The 802.15.4 PHY has the poll optimization enabled by default. Disable it until the driver state machine is updated. Signed-off-by: George Stefan <george.stefan@nxp.com>
@dleach02 dleach02 force-pushed the upstream/integrate_2509_blobs_mcxw7x branch from d70c237 to 7104638 Compare October 23, 2025 15:04
@github-actions github-actions bot added the DNM (manifest) This PR should not be merged (controlled by action-manifest) label Oct 23, 2025
@sonarqubecloud
Copy link

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
@dleach02 dleach02 removed the DNM (manifest) This PR should not be merged (controlled by action-manifest) label Oct 23, 2025
@dleach02
Copy link
Member

okay... sorry but this was confusing. This PR was opened on the same branch as the other one so I can't even open the other one... and I accidentally rebased this PR to main and force pushed so now it is going through CI again ... sigh
@jhedberg jhedberg merged commit e25a741 into zephyrproject-rtos:main Oct 23, 2025
29 of 30 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

area: Boards/SoCs area: IEEE 802.15.4 Binary Blobs Modified manifest manifest-hal_nxp platform: NXP MCU platform: NXP NXP

9 participants

@mmahadevan108 @jhedberg @dleach02 @axelnxp @George-Stefan @Cristib05 @ZhaoxiangJin @zephyrbot @decsny