Skip to content

Conversation

@nxf58150
Copy link
Contributor

Added new config CONFIG_NXP_WIFI_WAKE_TIMER_ENABLE and enable it for NXP wifi chips.

@github-actions
Copy link

github-actions bot commented Nov 12, 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@b374adf zephyrproject-rtos/hal_nxp@a7f64ac zephyrproject-rtos/hal_nxp@b374adf9..a7f64ac2

All manifest checks OK

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

@decsny
Copy link
Member

decsny commented Nov 12, 2025

I am not sure if it's smart to rely on zephyr configs in external modules. You should probably have an independent definition in the module and set it based on the config in zephyr repo in respective modules/ folder.

MaochenWang1
MaochenWang1 previously approved these changes Nov 13, 2025
@nxf58150
Copy link
Contributor Author

Hi @mmahadevan108, could you please help review this PR? What's the synchronization that Declan mentioned? Thanks!

MaochenWang1
MaochenWang1 previously approved these changes Nov 20, 2025
@mmahadevan108
Copy link
Contributor

@nxf58150 Declan is pointing about a design decision where the WiFi module is tied to the Zephyr Kconfig name.

@mmahadevan108 mmahadevan108 removed the DNM (manifest) This PR should not be merged (controlled by action-manifest) label Nov 20, 2025
Added new config CONFIG_NXP_WIFI_WAKE_TIMER_ENABLE and enable it for NXP
wifi chips.

Signed-off-by: Hui Bai <hui.bai@nxp.com>
Sync hal_nxp repo

Signed-off-by: Hui Bai <hui.bai@nxp.com>
Update wakeup source dump function to DBG level. This function will
print and clear wakeup source registers of NXP wifi chips with
CONFIG_WIFI_LOG_LEVEL_DBG enabled. By changing debug level of this file,
user can avoid wakeup source clear.

Signed-off-by: Hui Bai <hui.bai@nxp.com>
@sonarqubecloud
Copy link

@mmahadevan108
Copy link
Contributor

@jukkar , can you help take a look at this PR.

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.

6 participants