Skip to content

Fix drivers: ieee802154: mcxw71 echo_client and echo_server compilation #90291

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 4 commits into from
May 27, 2025
Merged

Fix drivers: ieee802154: mcxw71 echo_client and echo_server compilation #90291

merged 4 commits into from
May 27, 2025

Conversation

andrei-menzopol
Copy link
Contributor

Description

Fix #90290

  • soc: nxp: mcxw: update macro in Kconfig.defconfig
  • modules: hal_nxp: update mcux-sdk-ng compoments.cmake
  • drivers: ieee802154: fix mcxw ieee802154 driver

Testing

  • Compiled echo_server/echo_client with: west build -p always -b frdm_mcxw71 samples/net/sockets/echo_client -- -DEXTRA_CONF_FILE=overlay-ot.conf
  • Created a thread network with it and joined another thread device
  • Checked that the thread communication is working (with a 15.4 thread wireshark capture)
  • Was able to see the neighbors and ping

@github-actions GitHub Actions
Copy link

github-actions bot commented May 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@fa2b503 zephyrproject-rtos/hal_nxp@6a316c1 (master) zephyrproject-rtos/hal_nxp@fa2b503d..6a316c1b

✅ 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 May 21, 2025
@andrei-menzopol andrei-menzopol mentioned this pull request May 21, 2025
Merged
decsny
decsny previously approved these changes May 21, 2025
View reviewed changes
@andrei-menzopol @dleach02
west.yml: update hal_nxp to include middleware cmake update
fc11fe9 
Include hal_nxp to include middleware cmake update

Signed-off-by: Andrei Menzopol <andrei.menzopol@nxp.com>
dleach02
dleach02 previously approved these changes May 21, 2025
View reviewed changes
@dleach02 dleach02 closed this May 21, 2025
@dleach02 dleach02 reopened this May 21, 2025
@dleach02
Copy link
Member

closed/reopen to clear compliance meta data issue

@github-actions github-actions bot removed the DNM (manifest) This PR should not be merged (controlled by action-manifest) label May 21, 2025
jukkar
jukkar reviewed May 22, 2025
View reviewed changes
@George-Stefan George-Stefan requested review from jukkar and dleach02 May 22, 2025 09:05
jukkar
jukkar previously approved these changes May 22, 2025
View reviewed changes
cfriedt
cfriedt reviewed May 22, 2025
View reviewed changes
cfriedt
cfriedt previously approved these changes May 22, 2025
View reviewed changes
Copy link
Member

@cfriedt cfriedt left a comment

Choose a reason for hiding this comment

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

looks ok to me as well, just one observation about a misra rule violation. Fix that if you like or if e.g. @dleach02 requires it.

The CI issues seems to be unrelated.

@andrei-menzopol
drivers: ieee802154: fix mcxw ieee802154 driver
142265e 
Fix mcxw_cca function.
Remove not used / not implemented caps.
Select counter as dependency.

Signed-off-by: Andrei Menzopol <andrei.menzopol@nxp.com>
@andrei-menzopol
modules: hal_nxp: update mcux-sdk-ng compoments.cmake
83b6780 
Select driver.spc
Use CONFIG_IEEE802154

Signed-off-by: Andrei Menzopol <andrei.menzopol@nxp.com>
@andrei-menzopol
soc: nxp: mcxw: update macro in Kconfig.defconfig
6b3ece8 
Use IEEE802154 as it is set by the L2 macros too.

Signed-off-by: Andrei Menzopol <andrei.menzopol@nxp.com>
@andrei-menzopol andrei-menzopol dismissed stale reviews from cfriedt and jukkar via 6b3ece8 May 22, 2025 11:26
@sonarqubecloud 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

@kartben kartben merged commit 88a1805 into zephyrproject-rtos:main May 27, 2025
29 checks passed
@andrei-menzopol andrei-menzopol deleted the fix-issue-90290 branch May 27, 2025 07:18
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@cfriedt cfriedt

@George-Stefan George-Stefan

@decsny decsny

@ankuns ankuns

@bperseghetti bperseghetti

@butok butok

@DerekSnell DerekSnell

@EmilioCBen EmilioCBen

@manuargue manuargue

@MaochenWang1 MaochenWang1

@mmahadevan108 mmahadevan108

@PetervdPerk-NXP PetervdPerk-NXP

@rlubos rlubos

@dleach02 dleach02

@jukkar jukkar

Assignees

@dleach02 dleach02

@decsny decsny

Labels
area: IEEE 802.15.4 manifest manifest-hal_nxp platform: NXP Drivers NXP Semiconductors, drivers platform: NXP NXP
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.

drivers: ieee802154: mcxw71 echo_client and echo_server don't compile
7 participants
@andrei-menzopol @dleach02 @cfriedt @jukkar @George-Stefan @decsny @kartben