Thread (12 messages) 12 messages, 11 authors, 16d ago

Re: [PATCH 05/13] ARM: orion/dove/mv78xx0: remove all board files

From: "Arnd Bergmann" <arnd@kernel.org>
Date: 2026-09-08 18:22:05
Also in: imx, linux-devicetree, linux-ide, linux-pci, linux-rtc, linux-watchdog, sashiko-reviews

On Tue, Sep 8, 2026, at 18:11, sashiko-bot@kernel.org wrote:
[Severity: Low]
Since all the board files like arch/arm/mach-orion5x/ts78xx-setup.c are
removed in this patch, should the MAINTAINERS entry for the TS-78xx platform
further down in this file also be removed?

The pattern F: arch/arm/mach-orion5x/ts78xx-* now points to deleted files,
which leaves an orphaned entry for Alexander Clouter.
I fixed this instance now. I tried to automate this with a script
but could not figure out how to deal with wildcard matches and
only did the ones that have an exact file match.

If anyone has suggestions for doing this better, please let me know,
or run your script on 
https://git.kernel.org/pub/scm/linux/kernel/git/soc/soc.git/log/?h=board-remove-7.3
and send me the results.
[ ... ]
quoted
diff --git a/arch/arm/mach-orion5x/Makefile b/arch/arm/mach-orion5x/Makefile
index 6f54d7fef27a7..7e9634e043185 100644
--- a/arch/arm/mach-orion5x/Makefile
+++ b/arch/arm/mach-orion5x/Makefile
@@ -2,15 +2,6 @@
 ccflags-y := -I$(srctree)/arch/arm/plat-orion/include
 
 obj-y				+= common.o pci.o irq.o mpp.o
-obj-$(CONFIG_MACH_KUROBOX_PRO)	+= kurobox_pro-setup.o
-obj-$(CONFIG_MACH_TERASTATION_PRO2)	+= terastation_pro2-setup.o
[Severity: Low]
Are irq.o and mpp.o now completely unreachable code?

Since all the board files that called into these APIs have been removed,
it appears these objects are now unnecessarily compiled into the kernel
image. Should they be removed from obj-y here?
Added this to the patch, removing the two files and recursively
seven more that become orphaned afterwards:

 arch/arm/mach-orion5x/Makefile                |   2 +-
 arch/arm/mach-orion5x/irq.c                   |  51 ---
 arch/arm/mach-orion5x/irqs.h                  |   9 -
 arch/arm/mach-orion5x/mpp.c                   |  41 --
 arch/arm/mach-orion5x/mpp.h                   | 130 ------
 arch/arm/plat-orion/Makefile                  |   4 +-
 arch/arm/plat-orion/gpio.c                    | 616 --------------------------
 arch/arm/plat-orion/include/plat/irq.h        |  15 -
 arch/arm/plat-orion/include/plat/mpp.h        |  34 --
 arch/arm/plat-orion/include/plat/orion-gpio.h |  38 --
 arch/arm/plat-orion/irq.c                     |  39 --
 arch/arm/plat-orion/mpp.c                     |  82 ----
 13 files changed, 2 insertions(+), 1066 deletions(-)

This leaves only pcie.{c,h} in arch/arm/plat-orion/, which
should be folded into arch/arm/mach-orion5x/pci.c, along
with parts of arch/arm/kernel/bios32.c that nothing else
uses any more. I don't have a patch for that yet.

      Arnd

-- 
linux-phy mailing list
linux-phy@lists.infradead.org
https://lists.infradead.org/mailman/listinfo/linux-phy
Keyboard shortcuts
hback out one level
jnext message in thread
kprevious message in thread
ldrill in
Escclose help / fold thread tree
?toggle this help