MCXW23 zephyr base enablement #95290

AllenZhang-NXP · 2025-09-01T14:08:13Z

soc: mcxw235,mcxw236: add SOC support for MCXW235 and MCXW236
dts: mcxw235,mcxw236: add dts for MCXW235 and MCXW236
boards: frdm_mcxw23: add frdm_mcxw23 board
boards: mcxw23_evk: add mcxw23_evk board

github-actions · 2025-09-01T14:08:52Z

Hello @AllenZhang-NXP, and thank you very much for your first pull request to the Zephyr project!
Our Continuous Integration pipeline will execute a series of checks on your Pull Request commit messages and code, and you are expected to address any failures by updating the PR. Please take a look at our commit message guidelines to find out how to format your commit messages, and at our contribution workflow to understand how to update your Pull Request. If you haven't already, please make sure to review the project's Contributor Expectations and update (by amending and force-pushing the commits) your pull request if necessary.
If you are stuck or need help please join us on Discord and ask your question there. Additionally, you can escalate the review when applicable. 😊

nordicjm · 2025-09-01T14:25:48Z

soc/nxp/mcx/mcxw2x/CMakeLists.txt

+# CMSIS SystemInit allows us to skip enabling clock to SRAM banks via
+# this compiler definition
+if(NOT DEFINED CONFIG_MCXW23X_SRAM_CLOCKS)
+zephyr_compile_definitions(DONT_ENABLE_DISABLED_RAMBANKS=1)


2 space indent

nordicjm · 2025-09-01T14:26:12Z

soc/nxp/mcx/mcxw2x/Kconfig.defconfig

+	default 60
+
+config SYS_CLOCK_HW_CYCLES_PER_SEC
+	default 32000000


get from dts value using dt functions

You can reference this line for an example.
https://github.com/zephyrproject-rtos/zephyr/blob/main/soc/nxp/imxrt/Kconfig.defconfig#L27
You would need to add clock-frequency = 32000000 for the sysclk node in the soc dts file

nordicjm · 2025-09-01T14:26:56Z

dts/arm/nxp/nxp_mcxw23x_common.dtsi

+		compatible = "nxp,lpc-syscon";
+		reg = <0x0 0x4000>;
+		#clock-cells = <1>;
+		reset: reset {


newline line 79, fix in whole PR

nordicjm · 2025-09-01T14:27:44Z

boards/nxp/frdm_mcxw23/Kconfig.defconfig

+if BOOTLOADER_MCUBOOT
+choice MCUBOOT_BOOTLOADER_MODE
+	# Board only supports MCUBoot via "upgrade only" method:
+	default MCUBOOT_BOOTLOADER_MODE_OVERWRITE_ONLY
+endchoice
+endif #BOOTLOADER_MCUBOOT


Suggested change

if BOOTLOADER_MCUBOOT

choice MCUBOOT_BOOTLOADER_MODE

# Board only supports MCUBoot via "upgrade only" method:

default MCUBOOT_BOOTLOADER_MODE_OVERWRITE_ONLY

endchoice

endif #BOOTLOADER_MCUBOOT

if BOOTLOADER_MCUBOOT

choice MCUBOOT_BOOTLOADER_MODE

# Board only supports MCUBoot via "upgrade only" method:

default MCUBOOT_BOOTLOADER_MODE_OVERWRITE_ONLY

endchoice

endif # BOOTLOADER_MCUBOOT

nordicjm · 2025-09-01T14:28:32Z

boards/nxp/frdm_mcxw23/doc/frdm_mcxw23.webp

needs vastly reducing in size/resolution

nordicjm · 2025-09-01T14:29:01Z

boards/nxp/frdm_mcxw23/frdm_mcxw23_common.dtsi

+		#gpio-cells = <2>;
+		gpio-map-mask = <0xffffffff 0xffffffc0>;
+		gpio-map-pass-thru = <0 0x3f>;
+		gpio-map =	<0 0 &gpio0 16 0>,	/* A0 */


space after =, not tab

nordicjm · 2025-09-01T14:29:21Z

boards/nxp/mcxw23_evk/Kconfig.defconfig

+	depends on FXLS8974_TRIGGER
+
+if BOOTLOADER_MCUBOOT
+choice MCUBOOT_BOOTLOADER_MODE


apply changes from other commit

github-actions · 2025-09-04T03:51:06Z

The following west manifest projects have changed revision in this Pull Request:

Name	Old Revision	New Revision	Diff

✅ All manifest checks OK

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

AllenZhang-NXP · 2025-09-04T03:54:39Z

Fixed the issues mentioned by @nordicjm & the Github CI.
Rebased the main branch and synced the latest hal_nxp (has the updated MCXW23 header files) in west.yml.
Force updated the feature branch.

mmahadevan108 · 2025-09-05T16:21:37Z

boards/nxp/frdm_mcxw23/Kconfig.defconfig

+choice MCUBOOT_BOOTLOADER_MODE
+	# Board only supports MCUBoot via "upgrade only" method:
+	default MCUBOOT_BOOTLOADER_MODE_OVERWRITE_ONLY
+endchoice


@butok , do we need this defined here and Kconfig.sysbuild

Yes, both (with & without sysbuild)
The flash write-block-size = <512>

mmahadevan108 · 2025-09-05T16:31:13Z

boards/nxp/frdm_mcxw23/frdm_mcxw23.yaml

+  - gpio
+  - uart
+  - counter
+  - flash


mmahadevan108 · 2025-09-05T17:45:47Z

boards/nxp/frdm_mcxw23/frdm_mcxw23_defconfig

+CONFIG_UART_INTERRUPT_DRIVEN=y
+CONFIG_GPIO=y
+CONFIG_ARM_MPU=y
+CONFIG_RUNTIME_NMI=y


Why was this added to the defconfig. I don't see it in a number of the other NXP boards, hence curious why this was added.

hakehuang · 2025-09-09T09:17:02Z

boards/nxp/frdm_mcxw23/frdm_mcxw23.yaml

+  - gnuarmemb
+supported:
+  - gpio
+  - uart


can you help to enable the tests/driver/gpio with arduino header?
and enable tests/drivers/uart/uart_basic_api with lpuart loopback as well?

add soc MCX235 and MCXW236 for board frdm_mcxw23 Signed-off-by: Allen Zhang <[email protected]>

add dts for device MCXW235 and MCXW236 Signed-off-by: Allen Zhang <[email protected]>

enable board support for frdm_mcxw23 Signed-off-by: Allen Zhang <[email protected]>

enable board support for mcxw23_evk Signed-off-by: Allen Zhang <[email protected]>

…mcxw23_evk Support arduino_gpio example for NXP frdm_mcxw23 and mcxw23_evk boards. Test using tests/drivers/gpio/gpio_basic_api. Signed-off-by: Allen Zhang <[email protected]>

sonarqubecloud · 2025-09-11T17:42:30Z

Quality Gate passed

Issues
1 New issue
0 Accepted issues

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

See analysis details on SonarQube Cloud

zephyrbot added platform: NXP Drivers NXP Semiconductors, drivers platform: NXP NXP labels Sep 1, 2025

zephyrbot requested review from axelnxp, bperseghetti, butok, decsny, DerekSnell, dleach02, EmilioCBen, George-Stefan, JiafeiPan, manuargue, MaochenWang1, mmahadevan108, PetervdPerk-NXP and ZhaoxiangJin September 1, 2025 14:09

zephyrbot assigned dleach02, mmahadevan108 and decsny Sep 1, 2025

nordicjm requested changes Sep 1, 2025

View reviewed changes

nordicjm requested a review from kartben September 1, 2025 14:29

AllenZhang-NXP force-pushed the mcxw23_zephyr_base_enablement branch from 3a9127e to 8810b1d Compare September 4, 2025 03:50

github-actions bot added manifest manifest-hal_nxp labels Sep 4, 2025

mmahadevan108 reviewed Sep 5, 2025

View reviewed changes

boards/nxp/frdm_mcxw23/frdm_mcxw23.yaml

- gpio

- uart

- counter

- flash

Copy link

Contributor

mmahadevan108 Sep 5, 2025

Choose a reason for hiding this comment

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

i2c?

mmahadevan108 reviewed Sep 5, 2025

View reviewed changes

hakehuang reviewed Sep 9, 2025

View reviewed changes

AllenZhang-NXP added 5 commits September 12, 2025 01:21

soc: mcxw235,mcxw236: add SOC support for MCXW235 and MCXW236

fd846a2

add soc MCX235 and MCXW236 for board frdm_mcxw23 Signed-off-by: Allen Zhang <[email protected]>

dts: mcxw235,mcxw236: add dts for MCXW235 and MCXW236

1d23951

add dts for device MCXW235 and MCXW236 Signed-off-by: Allen Zhang <[email protected]>

boards: frdm_mcxw23: add frdm_mcxw23 board

bbb6559

enable board support for frdm_mcxw23 Signed-off-by: Allen Zhang <[email protected]>

boards: mcxw23_evk: add mcxw23_evk board

3e581f5

enable board support for mcxw23_evk Signed-off-by: Allen Zhang <[email protected]>

tests: drivers: gpio: add overlay to run test on NXP frdm_mcxw23 and …

3f88f52

…mcxw23_evk Support arduino_gpio example for NXP frdm_mcxw23 and mcxw23_evk boards. Test using tests/drivers/gpio/gpio_basic_api. Signed-off-by: Allen Zhang <[email protected]>

AllenZhang-NXP force-pushed the mcxw23_zephyr_base_enablement branch from 8810b1d to 3f88f52 Compare September 11, 2025 17:26

zephyrbot added the area: GPIO label Sep 11, 2025

github-actions bot removed manifest manifest-hal_nxp labels Sep 11, 2025

MCXW23 zephyr base enablement #95290

Are you sure you want to change the base?

MCXW23 zephyr base enablement #95290

Conversation

AllenZhang-NXP commented Sep 1, 2025

Uh oh!

github-actions bot commented Sep 1, 2025

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

github-actions bot commented Sep 4, 2025 • edited Loading Uh oh! There was an error while loading. Please reload this page.

Uh oh!

Uh oh!

AllenZhang-NXP commented Sep 4, 2025

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

sonarqubecloud bot commented Sep 11, 2025

Quality Gate passed

Uh oh!

Uh oh!

github-actions bot commented Sep 4, 2025 •

edited

Loading