Re: [PATCH 09/13] ARM: lpc18xx: remove entire platform
From: sashiko-bot@kernel.org
Date: 2026-09-08 15:41:54
Also in:
imx
Thank you for your contribution! Sashiko AI review found 1 potential issue(s) to consider: - [Medium] Incomplete platform removal leaves orphaned Kconfig references and dead driver code. -- commit 4fd816877e2bb7047a8b9bdb89bde42225376a14 Author: Arnd Bergmann [off-list ref] ARM: lpc18xx: remove entire platform This commit removes the architecture support for the NXP LPC18xx and LPC43xx microcontrollers since there are no known users running Linux on them. It deletes the corresponding platform directory, device tree sources, and removes the ARCH_LPC18XX Kconfig symbol.
quoted hunk ↗ jump to hunk
diff --git a/arch/arm/Kconfig b/arch/arm/Kconfig index da29afce309f6..13b38be3353a3 100644 --- a/arch/arm/Kconfig +++ b/arch/arm/Kconfig@@ -443,20 +443,6 @@ source "arch/arm/mach-zte/Kconfig" source "arch/arm/mach-zynq/Kconfig" -# ARMv7-M architecture -config ARCH_LPC18XX - bool "NXP LPC18xx/LPC43xx (DEPRECATED)" - depends on ARM_SINGLE_ARMV7M - select ARCH_HAS_RESET_CONTROLLER - select ARM_AMBA - select CLKSRC_LPC32XX - select PINCTRL - help - Support for NXP's LPC18xx Cortex-M3 and LPC43xx Cortex-M4 - high performance microcontrollers. - - This platform is scheduled for removal in early 2027 -
[Severity: Medium]
Does this removal leave several driver Kconfig entries and Makefiles with
unmet dependencies?
Removing ARCH_LPC18XX here without also cleaning up the corresponding driver
references causes Kconfig evaluation to check against a non-existent symbol.
For example, in drivers/gpio/Kconfig, GPIO_LPC18XX has:
depends on ARCH_LPC18XX || COMPILE_TEST
Similar orphaned references to ARCH_LPC18XX are left behind in:
- drivers/clk/Kconfig (COMMON_CLK_NXP)
- drivers/i2c/busses/Kconfig (I2C_LPC2K)
- drivers/dma/Kconfig (LPC18XX_DMAMUX)
- drivers/iio/adc/Kconfig (LPC18XX_ADC)
Additionally, it seems platform-specific driver code is left behind as dead
code. For instance, drivers/clk/nxp/Makefile continues to gate builds on the
deleted config:
obj-$(CONFIG_ARCH_LPC18XX) += clk-lpc18xx-cgu.o
obj-$(CONFIG_ARCH_LPC18XX) += clk-lpc18xx-ccu.o
Should the associated drivers, such as drivers/pinctrl/pinctrl-lpc18xx.c, be
removed in this patch series as well to prevent unmaintained dead code from
lingering in the tree?
config ARCH_MPS2 bool "ARM MPS2 platform (DEPRECATED)" depends on ARM_SINGLE_ARMV7M
-- Sashiko AI review · https://sashiko.dev/#/patchset/20260908152808.3928630-1-arnd@kernel.org?part=9