Re: [PATCH 10/11] clk: nxp: lpc: Remove NOMMU platform support
From: sashiko-bot@kernel.org
Date: 2026-06-19 15:58:30
Also in:
imx
Thank you for your contribution! Sashiko AI review found 2 potential issue(s) to consider: - [High] Mischaracterized platform deletion: LPC32xx is an MMU-enabled platform, not NOMMU. - [Low] Incomplete architecture removal leaves orphaned peripheral drivers as dead code. -- commit 6253e432d16195a0666c33f89bdf6509acf7cdf3 Author: Frank Li [off-list ref] clk: nxp: lpc: Remove NOMMU platform support This commit removes the NXP LPC clock drivers to reduce maintenance burden, stating that the NOMMU platform support is obsolete, unused, and unmaintained as users have migrated to other RTOS solutions.
clk: nxp: lpc: Remove NOMMU platform support Commercial users and hardware vendors migrated to Zephyr or other RTOS solutions years ago, leaving the NOMMU platform support effectively unused and unmaintained.
[ ... ]
quoted hunk ↗ jump to hunk
diff --git a/drivers/clk/nxp/Makefile b/drivers/clk/nxp/Makefile deleted file mode 100644 index 2cf6317d28531..0000000000000 --- a/drivers/clk/nxp/Makefile +++ /dev/null@@ -1,5 +0,0 @@ -# SPDX-License-Identifier: GPL-2.0-only -obj-$(CONFIG_ARCH_LPC18XX) += clk-lpc18xx-cgu.o -obj-$(CONFIG_ARCH_LPC18XX) += clk-lpc18xx-ccu.o -obj-$(CONFIG_ARCH_LPC18XX) += clk-lpc18xx-creg.o -obj-$(CONFIG_ARCH_LPC32XX) += clk-lpc32xx.o
[Severity: High] Is this justification accurate for all removed platforms? The commit message states this is removing NOMMU platform support, but the LPC32xx platform uses an ARM926EJ-S core. This core features a full Memory Management Unit (MMU) and runs standard virtual-memory Linux. Was LPC32xx swept up by mistake because it resided in the same directory as the NOMMU LPC18xx platform, or is the commit message mischaracterizing an MMU platform deletion? [ ... ]
quoted hunk ↗ jump to hunk
diff --git a/drivers/clk/Makefile b/drivers/clk/Makefile index cc108a75a9008..23b14a934635d 100644 --- a/drivers/clk/Makefile +++ b/drivers/clk/Makefile@@ -136,7 +136,6 @@ obj-y += mstar/ obj-y += mvebu/ obj-$(CONFIG_ARCH_MXS) += mxs/ obj-$(CONFIG_ARCH_MA35) += nuvoton/ -obj-$(CONFIG_COMMON_CLK_NXP) += nxp/
[Severity: Low] Does this incomplete architecture removal leave orphaned peripheral drivers as dead code elsewhere in the tree? Because the ARCH_LPC18XX and ARCH_LPC32XX Kconfig symbols have been deleted, their associated peripheral drivers (e.g., in drivers/gpio, drivers/pwm, drivers/net, drivers/iio) can now only be built when COMPILE_TEST is enabled and will never be executed. Could these peripheral drivers be removed atomically alongside the platform deletion? -- Sashiko AI review · https://sashiko.dev/#/patchset/20260619-dts_cleanup_arm_mcore-v1-0-0101795a2662@nxp.com?part=10