Re: [PATCH 03/13] ARM: remove riscpc

["Arnd Bergmann" <arnd@kernel.org>]

On Tue, Sep 8, 2026, at 17:52, sashiko-bot@kernel.org wrote:
>
> The Acorn Risc PC platform and its associated drivers, configurations,
> and board files are being removed as the required gcc-8 compiler and
> -march=armv3 option are no longer supported.
>
>>     The main limitation here is the CPU bus that does not allow 16-bit
>>     wide data transfers and hence requires the kernel to be built with the
>>     -march=armv3 compiler option that was discontinued in gcc-9.
>
> [Severity: Low]
> Since this compiler option is the explicitly stated reason for removal in the
> commit message, should we also delete the compiler flag from the tree?
>
> arch/arm/Makefile:
>     arch-$(CONFIG_CPU_32v3)         :=-march=armv3m
>
> Additionally, the config symbol for this architecture version is now
> dead code because its only selector (ARCH_RPC) was removed:
>
> arch/arm/mm/Kconfig:config CPU_32v3
>     config CPU_32v3
>         bool

I decided to remove these two as part of the later StrongARM
removal patch that depends on this one.

>> diff --git a/MAINTAINERS b/MAINTAINERS
>> index 81872f61b4317..fdd75b73bf293 100644
>> --- a/MAINTAINERS
>> +++ b/MAINTAINERS
> [ ... ]
>> -ARM MFM AND FLOPPY DRIVERS
>> -M:	Ian Molton <spyro@f2s.com>
>> -S:	Maintained
>> -F:	arch/arm/include/asm/floppy.h
>> -F:	arch/arm/mach-rpc/floppydma.S
>> -
>
> [Severity: Low]
> Does this leave the floppy header as dead code in the tree? It is removed
> from MAINTAINERS here, but the file itself was left in the tree:
>
> arch/arm/include/asm/floppy.h:
>     #ifndef __ASM_ARM_FLOPPY_H
>     #define __ASM_ARM_FLOPPY_H

This is removed along with ISA bus support later, which also
drops ARCH_MAY_HAVE_PC_FDC, ISA_DMA_API and GENERIC_ISA_DMA.

I could have split up the MAINTAINERS file change, but
it seemed better to do it in one place. Maybe the other
patch would have been better, but then I'd have another
dangling reference to mach-rpc here.

> The Makefile still references it:
>
> arch/arm/Makefile:
>     machine-$(CONFIG_ARCH_RPC)              += rpc

Fixed now, this ended up in the StrongARM patch by
accident.

> The machine type registry still contains it:
>
> arch/arm/tools/mach-types:
>     riscpc                  ARCH_RPC                RISCPC                  1

This is intentional.

> And the CPU selection still evaluates it:
>
> arch/arm/mm/Kconfig:config CPU_SA110
>     select CPU_32v3 if ARCH_RPC
>     select CPU_32v4 if !ARCH_RPC

Again, this is part of the StrongARM removal. I had considered
removing these two lines first, but then I'd have to split up
the CPU_32v3 removal out from that.

     Arnd