From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: X-Spam-Checker-Version: SpamAssassin 3.4.0 (2014-02-07) on aws-us-west-2-korg-lkml-1.web.codeaurora.org Received: from bombadil.infradead.org (bombadil.infradead.org [198.137.202.133]) (using TLSv1.2 with cipher ECDHE-RSA-AES256-GCM-SHA384 (256/256 bits)) (No client certificate requested) by smtp.lore.kernel.org (Postfix) with ESMTPS id 96BFDC79FB6 for ; Wed, 9 Sep 2026 20:11:04 +0000 (UTC) DKIM-Signature: v=1; a=rsa-sha256; q=dns/txt; c=relaxed/relaxed; d=lists.infradead.org; s=bombadil.20210309; h=Sender:List-Subscribe:List-Help :List-Post:List-Archive:List-Unsubscribe:List-Id:Content-Transfer-Encoding: Content-Type:Subject:References:In-Reply-To:Message-Id:Cc:To:From:Date: MIME-Version:Reply-To:Content-ID:Content-Description:Resent-Date:Resent-From: Resent-Sender:Resent-To:Resent-Cc:Resent-Message-ID:List-Owner; bh=dMpeLqTNXDRpS+r+jORTF862uDqb/CqEpFeuPMT9TQY=; b=awgoCmdBUz7mrIpSpEleZMpUHZ G8Ig9owupT0HGNw2cjKXujnEWyoJd0ehTgk/W9GjS44LzARdsgvewCI+zxmdLkUEODWW1iFg5AhwX ViuTyxqyKMJAQa216kbcU9BgA16S9T/cHC3MaD+m1s1E5NVdhe7kPYkOqBm9XgbMHfD6P7wSPQdlt GdlyweWc+WVVVSolKimW581HlPyTTJwHkk47xWmwe0EUWlkTYz9xOAzZx3uJ49DKsN4EH4tzviQXm ybReLPx5XOBZv2EkuzGvR/JvUm/0k6l7/YfuJbL+7fkoiDvuTW1bwux5E6sZGc7W/GTrY4VwWvBna a4knt//g==; Received: from localhost ([::1] helo=bombadil.infradead.org) by bombadil.infradead.org with esmtp (Exim 4.99.1 #2 (Red Hat Linux)) id 1x4OdG-0000000Cn9O-027I; Wed, 09 Sep 2026 20:10:58 +0000 Received: from sea.source.kernel.org ([172.234.252.31]) by bombadil.infradead.org with esmtps (Exim 4.99.1 #2 (Red Hat Linux)) id 1x4OdE-0000000Cn96-16Xr for linux-arm-kernel@lists.infradead.org; Wed, 09 Sep 2026 20:10:56 +0000 Received: from smtp.kernel.org (quasi.space.kernel.org [100.103.45.18]) by sea.source.kernel.org (Postfix) with ESMTP id B304643E32; Wed, 9 Sep 2026 20:10:55 +0000 (UTC) Received: by smtp.kernel.org (Postfix) with ESMTPSA id 06ADC1F00893; Wed, 9 Sep 2026 20:10:54 +0000 (UTC) DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=kernel.org; s=k20260515; t=1788984655; bh=dMpeLqTNXDRpS+r+jORTF862uDqb/CqEpFeuPMT9TQY=; h=Date:From:To:Cc:In-Reply-To:References:Subject; b=Y7ak+DteDAyaPCk16KIM3ABbCFWIdaOUPtP3CA84IcjaDKB56nEBmNI3LT9hwEqis 43S4ORpDR/gwIJALm1llpMxHjDUTN7c8CCcDn3FR1t+Y3LgXo1iiUUqE8Wzg9CPk0T J2oZvHCitZ6pIxXsdhRabrf3sNvioO4u5DVp8BjvWOeKAVHS7h0QHTW9G2SztdKKjP J7/noU1s4AtqITJZSMRwc/wDLPNfKpEvlV6a2rsOikazDEXnqlyCOVhoLy3HH9twCx Gl69U48p9UaYrvuO90k86iIqlGhqqRyBrfA0EX5tJ89AELqJ8BWnFgFulWVbKl9Rvm 9AJga9OEM2Qmw== Received: from ams-compute-02.internal (ams-compute-02.internal [10.64.2.62]) by mailfauth.ams.internal (Postfix) with ESMTP id 48EF31980052; Wed, 9 Sep 2026 16:10:53 -0400 (EDT) Received: from ams-imap-03 ([10.64.2.23]) by ams-compute-02.internal (MEProxy); Wed, 09 Sep 2026 16:10:53 -0400 X-ME-Sender: X-ME-Proxy-Cause: dmFkZTGP2bhb/XYhsNLB9S53X9rzoPN54qfE/KjrIQ+39ezusymjgkFG9cet+423ozVpDR QFvnmTNBMcgGscLcMMj7nBUY31hzJeOaGY2jSrT3p2YxejgdAUzrKdVOR8lCRBS16Gn88X /+XGMDl8CSXVYCQsr0wErwVPc+V6zmpd4d51AzTQO8MWr89uCO460fbix+l7zmvoJuzkvE RaO6/PdGwEVTqYIezlus3lDCEAnry2C8vj8V499E/dlZoQ4ScXku67uqDjBlKaXI4HB2gC KacujXVdDoEwzQ2dyYPEUzaZ7+e2mOEhOGj1Fetuz0MEj4OcAAwjVvsFr0sHvhPpz277Gf eKzGoSn6iqanxzNgqqMPsXCY76vQD+ea58iHSG5APtokUDPToSvfELosscq5Qt4yP2Q9+t y1tk8mh/YV6JND1qNbS+O2favpqJYSFmbwPFawNlshEMNgXzINZxwKWiewG0SxhFiQFIL9 4DpI0upYjn0k3v2eBM7Iml4anYd8gF3WPX86kKH7R0o4H64FA7zDDSXAF3jY5QkB0ZSiW6 sIKc9JmLSD+i8R4Jef0OcLMUt1jMXi6faA5jYE6vH5UfgYoDR3/ZIP9Q6vSw4pAYMA/Nxa V2SShKsn+8CcVH/XBZJ+tLKfo4Kqs/CxHjYl1vrrgz/DIr30WA6FviGoSjxg X-ME-Proxy: Feedback-ID: i36794607:Fastmail Received: by mailuser.ams.internal (Postfix, from userid 501) id 78FB232A007B; Wed, 9 Sep 2026 16:10:50 -0400 (EDT) X-Mailer: MessagingEngine.com Webmail Interface MIME-Version: 1.0 X-ThreadId: AGTXWeH1PU47 Date: Wed, 09 Sep 2026 22:09:43 +0200 From: "Arnd Bergmann" To: "Bart Van Assche" , "Krzysztof Kozlowski" , "Alexandre Belloni" , "Linus Walleij" , "Russell King" Cc: linux-doc@vger.kernel.org, linux-kernel@vger.kernel.org, linux-arm-kernel@lists.infradead.org, devicetree@vger.kernel.org, imx@lists.linux.dev, linux-stm32@st-md-mailman.stormreply.com, Linux-OMAP , linux-block@vger.kernel.org, linux-serial@vger.kernel.org, linux-fbdev@vger.kernel.org Message-Id: In-Reply-To: <2da174ef-1d5a-425d-b885-be9482b0be90@acm.org> References: <20260908152808.3928630-1-arnd@kernel.org> <20260908152808.3928630-4-arnd@kernel.org> <2da174ef-1d5a-425d-b885-be9482b0be90@acm.org> Subject: Re: [PATCH 03/13] ARM: remove riscpc Content-Type: text/plain; charset=utf-8 Content-Transfer-Encoding: quoted-printable X-BeenThere: linux-arm-kernel@lists.infradead.org X-Mailman-Version: 2.1.34 Precedence: list List-Id: List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , Sender: "linux-arm-kernel" Errors-To: linux-arm-kernel-bounces+linux-arm-kernel=archiver.kernel.org@lists.infradead.org On Wed, Sep 9, 2026, at 18:42, Bart Van Assche wrote: > On 9/8/26 8:27 AM, Arnd Bergmann wrote: >> The Acorn Risc PC is the oldest still support Arm machine in the kern= el, >> originally shipped with an ARMv3 ARM610 support in 1994, it later all= owed >> faster StrongARM CPUs. >>=20 >> 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=3Darmv3 compiler option that was discontinued in gcc-9. >>=20 >> As gcc-8 is barely supported for compiling the kernel, this one >> will not see another LTS kernel release, so remove it now. > > If this patch has to be reposted, here is the feedback of my AI coding > assistant (based on an LLM and hence may be wrong): Most of this was already discussed, but there are two additional changes that I made based on it. > Issues & Observations > --------------------- > 1. Missed arch/arm/Makefile cleanup > Patch 03 deletes the entire directory arch/arm/mach-rpc/, but does not=20 > touch arch/arm/Makefile. As a result, two orphaned entries remain: > > machine-$(CONFIG_ARCH_RPC) +=3D rpc > machdirs-$(CONFIG_ARCH_RPC) +=3D arch/arm/mach-rpc > Furthermore, machdirs-$(CONFIG_ARCH_RPC) was the very last user of=20 > machdirs- in arch/arm/Makefile (after Footbridge and SA-1100 are remov= ed=20 > in patches 1 & 2): Fixed already after sashiko-bot found the same thing. > kconfig > > > config FONT_ACORN_8x8 > bool "Acorn console 8x8 font" if FONTS > depends on FRAMEBUFFER_CONSOLE || DRM_PANIC > default y if !SPARC && !FONTS && ARM && ARCH_ACORN > Because ARCH_ACORN is removed, this default can never trigger and the=20 > reference should be cleaned up. I have this in the patch that removes acornfb. It could go into either one, but I felt it fits better there > kconfig > > > config CPU_SA110 > bool > select CPU_32v3 if ARCH_RPC > select CPU_32v4 if !ARCH_RPC > Since ARCH_RPC is removed, select CPU_32v3 if ARCH_RPC references a=20 > non-existent symbol. If CPU_SA110 remains temporarily (before the=20 > architectural cleanup in 7.5), this should be simplified to select CPU= _32v4. Also explained already that I preferred removing CPU_32v3 along with CPU_SA110 in a later patch, not together with ARCH_RPC. > 4. Dead code in arch/arm/kernel/atags_compat.c > With CONFIG_ARCH_ACORN removed from arch/arm/Kconfig, two blocks in > arch/arm/kernel/atags_compat.c > are now dead code: > > Lines 150=E2=80=93157: #ifdef CONFIG_ARCH_ACORN / if (machine_is_riscp= c())=20 > memory bank tagging. > Lines 186=E2=80=93194: #ifdef CONFIG_ARCH_ACORN ATAG_ACORN translation. > These blocks should be removed as part of this patch. I have a patch to remove that file. > > /* > * Now, pick up the machine-defined IO definitions > */ > #if IS_ENABLED(CONFIG_PCMCIA) || defined(CONFIG_PCI) > CONFIG_NEED_MACH_IO_H and #include were removed, leaving=20 > this comment describing generic PCI/PCMCIA I/O limits. Fixed now. > arch/arm/boot/bootp/Makefile:21 > : # guess an initrd location if possible is left dangling right before=20 > ifdef INITRD_OFFSET after all initrd_offset-$(CONFIG_...) logic has be= en=20 > stripped out. Removed that entire block now: --- a/arch/arm/boot/bootp/Makefile +++ b/arch/arm/boot/bootp/Makefile @@ -17,12 +17,6 @@ add_hex =3D $(shell printf 0x%x $$(( $(1) + $(2) )) ) # INITRD_PHYS must be in RAM =20 PARAMS_PHYS :=3D $(call add_hex, $(PHYS_OFFSET), 0x100) - -# guess an initrd location if possible -ifdef INITRD_OFFSET -INITRD_PHYS :=3D $(call add_hex, $(PHYS_OFFSET), $(INITRD_OFFSET)) -endif - endif =20 PHONY +=3D initrd @@ -30,7 +24,7 @@ initrd: @test "$(PARAMS_PHYS)" !=3D "" || \ (echo bootpImage: You must specify PHYS_OFFSET of PARAMS_PHYS ; = exit -1) @test "$(INITRD_PHYS)" !=3D "" || \ - (echo bootpImage: You must specify INITRD_OFFSET or INITRD_PHYS = ; exit -1) + (echo bootpImage: You must specify INITRD_PHYS ; exit -1) @test "$(INITRD)" !=3D "" || \ (echo bootpImage: You must specify INITRD; exit -1) =20 > 6. Floppy driver & arch/arm/include/asm/floppy.h status > The patch removes the ARM MFM AND FLOPPY DRIVERS entry from MAINTAINER= S: commented already that this is removed along with CONFIG_ISA in a later patch. Arnd