Skip to content

Conversation

MaochenWang1
Copy link
Contributor

manifest: sync hal_nxp to upgrade NXP wifi driver
samples: net: wifi: enable NXP_WIFI_FW_DEBUG for RW612 and IW610
modules: hal_nxp: build flexspi and mflash for FW dump case
drivers: wifi: nxp: update Kconfig to support FW dump

Copy link

github-actions bot commented Oct 10, 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@4377ecf zephyrproject-rtos/hal_nxp@41d5acc zephyrproject-rtos/[email protected]

All manifest checks OK

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

@ZhaoxiangJin
Copy link
Contributor

manifest change needs to be first?

@MaochenWang1 MaochenWang1 force-pushed the main_mc_support_fw_dump_for_wifi branch from c0611d1 to 8fb06c6 Compare October 10, 2025 08:55
@MaochenWang1
Copy link
Contributor Author

manifest change needs to be first?

updated

@zephyrbot zephyrbot requested a review from kartben October 10, 2025 08:57
@MaochenWang1 MaochenWang1 force-pushed the main_mc_support_fw_dump_for_wifi branch from ec2c707 to b07fd99 Compare October 14, 2025 02:45
@github-actions github-actions bot removed the DNM (manifest) This PR should not be merged (controlled by action-manifest) label Oct 14, 2025
@MaochenWang1
Copy link
Contributor Author

rebase on TOT and update the hal_nxp SHA.

ZhaoxiangJin
ZhaoxiangJin previously approved these changes Oct 14, 2025
config NXP_FW_DUMP_FLASH_START_ADDR
hex "Start address in flash to store FW dump"
default 0x0B000000 if NXP_RW610
default 0x60500000 if BOARD_MIMXRT1060_EVK
Copy link
Contributor

@mmahadevan108 mmahadevan108 Oct 14, 2025

Choose a reason for hiding this comment

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

This should be set at the board level

Copy link
Member

Choose a reason for hiding this comment

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

@MaochenWang1 what @mmahadevan108 is saying is that you should go to the BOARD_MIMXRT1060_EVK and set this value. Don't put it down in the driver file. Look at board\nxp\mimxrt1060_evk\Kconfig.defconfig. There you would put:
config NXP_FW_DUMP_FLASH_START_ADDR default 0x60500000
In the case of the NXP_RW610 there are comparable Kconfig.defconfig files in the soc folder.

Copy link
Contributor Author

Choose a reason for hiding this comment

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

updated, please review again.

upgrade wifi driver version to r52.p11.
fix several bugs and support fw dump.

Signed-off-by: Maochen Wang <[email protected]>
Remove the redundant dependency of NXP_WIFI_EXTRA_DEBUG in Kconfig.
Add NXP_FW_DUMP_FLASH_START_ADDR for the start address in flash to
store FW dump.
Add NXP_FW_DUMP_FLASH_ERASE_LENGTH for the erased length in flash
to store FW dump.

Signed-off-by: Maochen Wang <[email protected]>
Build flexspi and mflash to support save the FW dump into flash for
debug purpose.

Signed-off-by: Maochen Wang <[email protected]>
Enable NXP_WIFI_FW_DEBUG for RW612 and IW610 to support save FW dump
into flash for debug purpose.

Signed-off-by: Maochen Wang <[email protected]>
Set different default value for NXP_FW_DUMP_FLASH_START_ADDR, based on
mimxrt1060_evk board and rw612 soc.

Signed-off-by: Maochen Wang <[email protected]>
@mmahadevan108 mmahadevan108 force-pushed the main_mc_support_fw_dump_for_wifi branch from 1fe65e8 to e759adb Compare October 15, 2025 14:39
@mmahadevan108 mmahadevan108 self-requested a review October 15, 2025 14:40
Copy link

@cfriedt cfriedt merged commit 1bb4e5d into zephyrproject-rtos:main Oct 15, 2025
34 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

8 participants