Skip to content

Add mcxw23 power management support #98986

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
Open
wants to merge 11 commits into
base: main
Choose a base branch
from
Open

Add mcxw23 power management support #98986

wants to merge 11 commits into from
+532 −19

Conversation

@zejiang0jason
Copy link
Contributor

@zejiang0jason zejiang0jason commented Nov 6, 2025 

Enabled modes:
  idle: SLEEP
  suspend: DEEP-SLEEP
  standby: POWER-DOWN with CPU retention

OS Time Base: OSTIMER with fro1m clock source

@zejiang0jason zejiang0jason changed the title Feature/add mcxw23 power Add mcxw23 power management support Nov 6, 2025
@zejiang0jason zejiang0jason changed the title Add mcxw23 power management support [DRAFT]Add mcxw23 power management support Nov 6, 2025
@zejiang0jason zejiang0jason marked this pull request as draft November 6, 2025 09:26
@zejiang0jason zejiang0jason force-pushed the feature/add_mcxw23_power branch from 4d0b6b2 to 0bd9b0f Compare November 17, 2025 09:58
@zejiang0jason zejiang0jason force-pushed the feature/add_mcxw23_power branch from f8f5741 to 465082f Compare November 18, 2025 08:34
@yeaissa

This comment was marked as resolved.

Sign in to view
@zejiang0jason zejiang0jason force-pushed the feature/add_mcxw23_power branch from 465082f to 7920277 Compare November 24, 2025 14:33
@zejiang0jason

This comment was marked as resolved.

Sign in to view
ZhaoxiangJin
ZhaoxiangJin reviewed Nov 25, 2025
View reviewed changes
yeaissa
yeaissa reviewed Nov 27, 2025
View reviewed changes
boards/nxp/frdm_mcxw23/doc/index.rst
*** Booting Zephyr OS build v4.2.0-2105-g48f2ffda26de ***
Hello World! frdm_mcxw23/mcxw236
Power Management
Copy link
Contributor

@yeaissa yeaissa Nov 27, 2025 •
edited
Loading

Choose a reason for hiding this comment

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

I'm not ok with this, since it will require adding the overlay on each app that will run with PM. I expect to have everything I need once I enable CONFIG_PM.
To avoid this lets set the os_timer status to okay by default in the nxp_mcxw23x_common.dtsi file, and change the MCUX_OS_TIMER redefinition in Kconfig.defconfig to:

 config MCUX_OS_TIMER
-       default y if PM
+       default n if !PM

This will allow to run on the CORTEX_M_SYSTICK when PM is not defined and switch to OS_TIMER when PM is enabled.

Copy link
Contributor Author

@zejiang0jason zejiang0jason Dec 1, 2025

Choose a reason for hiding this comment

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

updated.

@zejiang0jason
dts: bindings: os-timer: Add resets array
7de9fcb 
Add to support reset OS_TIMER in driver

Signed-off-by: Jason Yu <[email protected]>
@zejiang0jason
drivers: timer: ostimer: Change to use reset API
b1c73a2 
Use reset API to reset OSTIMER for better portability.

Signed-off-by: Jason Yu <[email protected]>
@zejiang0jason
drivers: timer: ostimer: Fix run fail when no deep_sleep_counter
e90532c 
Only access deep_sleep_counter when it is available.

Signed-off-by: Jason Yu <[email protected]>
@zejiang0jason
drivers: dma: dma_mcux_lpc: Support power device constraint
b6b43f7 
Call pm_policy_device_power_lock_put/pm_policy_device_power_lock_get
to coordinate with system level power modes.

Signed-off-by: Jason Yu <[email protected]>
@zejiang0jason
drivers: dma: dma_mcux_lpc: Add register backup and restore
50089c4 
Add for the case that the registers are not kept in some
low power modes.

Signed-off-by: Jason Yu <[email protected]>
@zejiang0jason
drivers: counter: nxp_mrt: Support power device constraint
075aae2 
Call pm_policy_device_power_lock_put/pm_policy_device_power_lock_get
to coordinate with system level power modes.

Signed-off-by: Jason Yu <[email protected]>
@zejiang0jason
drivers: interrupt: pint: Add API to get pin used IRQ slot
fa4a894 
PINT connects GPIO pin to seperate IRQ slot.
Add new API to get which IRQ slot is connected to, based on
pin index.

Signed-off-by: Jason Yu <[email protected]>
@zejiang0jason
soc: nxp: mcxw2xx: Improve OS tick timer selection
566d83c 
When os_timer is enabled in dts, then os_timer will be used as
OS tick timer.

To make systick as the default OS tick timer, currently os_timer
is not enabled in dts. When users want to use os_timer as
OS tick timer, they need to override the dts.

Improve the method, enable the os_timer is dts, but not enable
in Kconfig by default. If need to use os_timer as OS tick, just pass
CONFIG_MCUX_OS_TIMER=y

Signed-off-by: Jason Yu <[email protected]>
@zejiang0jason
soc: nxp: mcxw2xx: Enable the power management
e90e7a5 
Enabled modes:
  idle: SLEEP
  suspend: DEEP-SLEEP
  standby: POWER-DOWN with CPU retention

OS Time Base: OSTIMER with 32K clock source

Signed-off-by: Jason Yu <[email protected]>
@zejiang0jason
soc: nxp: mcxw2xx: Add poweroff support
8ca7b4b 
Add poweroff MCXW2xx SoCs, support btn_wk pin wakeup.

Signed-off-by: Jason Yu <[email protected]>
@zejiang0jason
boards: nxp: mcxw23: Add power management MCXW23 board
2e2eba2 
Add mcxw23_evk and frdm_mcxw23 platforms to the power management
test suite to validate power management functionality on MCXW23
hardware.

Signed-off-by: Jason Yu <[email protected]>
@zejiang0jason zejiang0jason force-pushed the feature/add_mcxw23_power branch from 7920277 to 2e2eba2 Compare December 1, 2025 03:55
@sonarqubecloud
Copy link

sonarqubecloud bot commented Dec 1, 2025

Quality Gate Passed Quality Gate passed

Issues
0 New issues
0 Accepted issues

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

See analysis details on SonarQube Cloud

@dleach02 dleach02 removed their assignment Dec 3, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

@ZhaoxiangJin ZhaoxiangJin ZhaoxiangJin left review comments

@decsny decsny Awaiting requested review from decsny

@mmahadevan108 mmahadevan108 Awaiting requested review from mmahadevan108

@axelnxp axelnxp Awaiting requested review from axelnxp

@bjarki-andreasen bjarki-andreasen Awaiting requested review from bjarki-andreasen

@ceolin ceolin Awaiting requested review from ceolin

@dbaluta dbaluta Awaiting requested review from dbaluta

@dleach02 dleach02 Awaiting requested review from dleach02

@George-Stefan George-Stefan Awaiting requested review from George-Stefan

@manuargue manuargue Awaiting requested review from manuargue

@MaochenWang1 MaochenWang1 Awaiting requested review from MaochenWang1

@nashif nashif Awaiting requested review from nashif

@andyross andyross Awaiting requested review from andyross

@nordic-krch nordic-krch Awaiting requested review from nordic-krch

@teburd teburd Awaiting requested review from teburd

+2 more reviewers

@Albort12138 Albort12138 Albort12138 left review comments

@yeaissa yeaissa yeaissa left review comments

Reviewers whose approvals may not affect merge requirements

At least 2 approving reviews are required to merge this pull request.

Assignees

@mmahadevan108 mmahadevan108

Labels

area: Boards/SoCs area: DMA Direct Memory Access area: Timer Timer platform: NXP MCU platform: NXP NXP

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

7 participants

@zejiang0jason @ZhaoxiangJin @yeaissa @Albort12138 @mmahadevan108 @zephyrbot @dleach02