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 706F6C4167B for ; Tue, 5 Dec 2023 07:00:44 +0000 (UTC) DKIM-Signature: v=1; a=rsa-sha256; q=dns/txt; c=relaxed/relaxed; d=lists.infradead.org; s=bombadil.20210309; h=Sender: Content-Transfer-Encoding:Content-Type:List-Subscribe:List-Help:List-Post: List-Archive:List-Unsubscribe:List-Id:Subject:Cc:To:From:Date:References: In-Reply-To:Message-Id:MIME-Version:Reply-To:Content-ID:Content-Description: Resent-Date:Resent-From:Resent-Sender:Resent-To:Resent-Cc:Resent-Message-ID: List-Owner; bh=kX+qVpJ/FcnlyqUfQg05Er/hn977tiC6/RGsPyWnKAE=; b=MyRlwqa7OPbPg3 msMdxA0zHSFITkYz82X/7Pds/61Kcq9Al9jCzOdgY4MEfkcg9YDWVfL/hi9Q4Xh0IWywGMkxfymPY oW0+1E/wmS7GTwJMKMMh7LWfPV+iAXE2by0FGQQSveXBdcl23ntpT1FmGMlQgVD7fmnMQ1v5AWEJU R1dyQ7rg9ovelKY/IZhh2PhNFhZQ+7mOfU5wJBbIFg+xjFQmr9VjUPefupkOkY7Vz7ADTVf70YmG0 Uwqcf45MdBX1+zMxkWdPT+cWzmCzW0UKIQDNLMDTDZvY32kOMCrPBeUUCL69DvqeJeNFxYM9xriAD sbk+k29/rUfx4p9Uv/bg==; Received: from localhost ([::1] helo=bombadil.infradead.org) by bombadil.infradead.org with esmtp (Exim 4.96 #2 (Red Hat Linux)) id 1rAPPf-006SI3-0f; Tue, 05 Dec 2023 07:00:11 +0000 Received: from wout1-smtp.messagingengine.com ([64.147.123.24]) by bombadil.infradead.org with esmtps (Exim 4.96 #2 (Red Hat Linux)) id 1rAPPc-006SHE-0G for linux-arm-kernel@lists.infradead.org; Tue, 05 Dec 2023 07:00:09 +0000 Received: from compute5.internal (compute5.nyi.internal [10.202.2.45]) by mailout.west.internal (Postfix) with ESMTP id 830413200A66; Tue, 5 Dec 2023 02:00:06 -0500 (EST) Received: from imap51 ([10.202.2.101]) by compute5.internal (MEProxy); Tue, 05 Dec 2023 02:00:07 -0500 DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=arndb.de; h=cc :cc:content-type:content-type:date:date:from:from:in-reply-to :in-reply-to:message-id:mime-version:references:reply-to:sender :subject:subject:to:to; s=fm3; t=1701759606; x=1701846006; bh=Ia LuBMUbt7Bq6MzW3x/Fa4v5CIE8FfyrZOGwykmrdHQ=; b=gpss7fs/8TqAz0mnjc ePlqe94cRe8/+7NvchDndFI47/mtjircKNfVxxl0zdtjFprD4MFGtQLBDcIvCcO1 d2ETFZXcN18h2Eo8qMZxvXD/ZJDQi+UF6FdNpMvehXnz1/R/cMPnxqOCorTeZRRL 2nr3N9WNYr8MvxsDZsw/k6E02t8VybFEVhON8MautrJI18u5SOSA+2EZQpd1yt9j ny0NuFyvL4Bf+yCVE0Zs+knYAja1Isk2aJCPSj7MFd5TNc2UG1NwbD6wpOqFtO8/ TZEa0rmTbxesXuu6aNQUUz4d9ipCppTt+bzryrmw4iRDaQXsCdA/yrMS9iZdOt0l WEuw== DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d= messagingengine.com; h=cc:cc:content-type:content-type:date:date :feedback-id:feedback-id:from:from:in-reply-to:in-reply-to :message-id:mime-version:references:reply-to:sender:subject :subject:to:to:x-me-proxy:x-me-proxy:x-me-sender:x-me-sender :x-sasl-enc; s=fm1; t=1701759606; x=1701846006; bh=IaLuBMUbt7Bq6 MzW3x/Fa4v5CIE8FfyrZOGwykmrdHQ=; b=hR+arfHVmyfVNJNNEiA+zYxpJGK38 phH8sLYsStaiR087KNHHyIpNkNRSVKSFfN1z7i9WzmVMstGEOTFa+Vv4dCzxB/jV ADw7/w4J1T0ooMlcGjJKHVLt9lZZ1jONUuVQu1J5EBfEG2+LHifDH8W0ctXGFSB/ agpIgt/WQ0ZFBN1NFSbkXwpjBLIxaYf8oOXsMobU3rO6pkghFafRvri+OjDFaIYS aBgc5nFpbzOaOZivg2dbgjf9z4bEVctGjqz950iuJNAEKb0+JDK6FNrzSSwh83r0 Tu7nlAblu1G9yBNa8LBqNX2QDoStvf45H4rwRuMiSAnKq5+McWaF+XBCQ== X-ME-Sender: X-ME-Proxy-Cause: gggruggvucftvghtrhhoucdtuddrgedvkedrudejjedguddtfecutefuodetggdotefrod ftvfcurfhrohhfihhlvgemucfhrghsthforghilhdpqfgfvfdpuffrtefokffrpgfnqfgh necuuegrihhlohhuthemuceftddtnecusecvtfgvtghiphhivghnthhsucdlqddutddtmd enucfjughrpefofgggkfgjfhffhffvvefutgesthdtredtreertdenucfhrhhomhepfdet rhhnugcuuegvrhhgmhgrnhhnfdcuoegrrhhnugesrghrnhgusgdruggvqeenucggtffrrg htthgvrhhnpeffheeugeetiefhgeethfejgfdtuefggeejleehjeeutefhfeeggefhkedt keetffenucevlhhushhtvghrufhiiigvpedtnecurfgrrhgrmhepmhgrihhlfhhrohhmpe grrhhnugesrghrnhgusgdruggv X-ME-Proxy: Feedback-ID: i56a14606:Fastmail Received: by mailuser.nyi.internal (Postfix, from userid 501) id A7F24B60089; Tue, 5 Dec 2023 02:00:05 -0500 (EST) X-Mailer: MessagingEngine.com Webmail Interface User-Agent: Cyrus-JMAP/3.9.0-alpha0-1178-geeaf0069a7-fm-20231114.001-geeaf0069 MIME-Version: 1.0 Message-Id: <93cc5193-aa1b-4df8-8d00-c3149c33b7e7@app.fastmail.com> In-Reply-To: <20231204-drop-11mpcore-v1-1-f96cd3938cf1@linaro.org> References: <20231204-drop-11mpcore-v1-1-f96cd3938cf1@linaro.org> Date: Tue, 05 Dec 2023 07:59:32 +0100 From: "Arnd Bergmann" To: "Linus Walleij" , "Mark Rutland" , "Will Deacon" , "Russell King" , "Liviu Dudau" , "Sudeep Holla" , "Lorenzo Pieralisi" , "Lubomir Rintel" Cc: linux-perf-users@vger.kernel.org, linux-arm-kernel@lists.infradead.org Subject: Re: [PATCH] ARM: Delete ARM11MPCore (ARM11 ARMv6K SMP) support X-CRM114-Version: 20100106-BlameMichelson ( TRE 0.8.0 (BSD) ) MR-646709E3 X-CRM114-CacheID: sfid-20231204_230008_156278_C6C39721 X-CRM114-Status: GOOD ( 28.54 ) 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: , Content-Type: text/plain; charset="us-ascii" Content-Transfer-Encoding: 7bit Sender: "linux-arm-kernel" Errors-To: linux-arm-kernel-bounces+linux-arm-kernel=archiver.kernel.org@lists.infradead.org On Mon, Dec 4, 2023, at 23:36, Linus Walleij wrote: > This ARM11 SMP configuration was one of the first SMP configurations > the ARM kernel supported, but it has the downside of odd DMA handling, > odd cache tagging, and often (as of recent) completely broken cache > handling on the ARM RealView PB11MPCore test chips. To boot the > platform it was necessary to completely disable the cache. > When it comes to the EB 11MPCore it is unclear if this ever worked. > > These reference designs are now the only ARMv6K SMP platforms. > > As only reference designs of purely academic interest remain, and > since the special-cased DMA and PMU code is hard to maintain and > doesn't really work, it is not really worth our time. > > Delete the ARM11MPCore support along with: > > - The special DMA quirk CONFIG_DMA_CACHE_RWFO that is only used > on ARMv6K SMP, and we are the last ARMV6K system leaving the > building and the cache handling is awkward, so good-bye. > > - The special PMU handling that was only used by ARM11MPCore. > > The following is left behind: > > - TIMER_OF_DECLARE(arm_twd_11mp, "arm,arm11mp-twd-timer", ...) > in arch/arm/kernel/smp_twd.c, this is still in use by Marvell MMP3 > arch/arm/boot/dts/marvell/mmp3.dtsi > > - IRQCHIP_DECLARE(arm11mp_gic, "arm,arm11mp-gic", ...) > in drivers/irqchip/irq-gic.c, this is still in use by Marvell MMP3 > arch/arm/boot/dts/marvell/mmp3.dtsi > > These are unfortunate but will need to be kept around for > compatibility. New Marvell-specific compatibles should however probably > be added. > > Signed-off-by: Linus Walleij This looks good to me (one small issue below), thanks for the cleanup! Reviewed-by: Arnd Bergmann > arch/arm/kernel/perf_event_v6.c | 34 ------------------------------ > arch/arm/mach-versatile/Kconfig | 17 --------------- > arch/arm/mach-versatile/platsmp-realview.c | 2 -- > arch/arm/mach-versatile/realview.c | 1 - > arch/arm/mm/Kconfig | 18 ---------------- > arch/arm/mm/cache-v6.S | 31 --------------------------- > 6 files changed, 103 deletions(-) Russell, this touches both mach-versatile and core Arm code, any preference for how we should merge it (provided you are happy with the changes)? I'm fine with the combined change going through your tree or mine, or splitting it up. --- a/arch/arm/mach-versatile/platsmp-realview.c +++ b/arch/arm/mach-versatile/platsmp-realview.c @@ -18,7 +18,6 @@ #define REALVIEW_SYS_FLAGSSET_OFFSET 0x30 static const struct of_device_id realview_scu_match[] = { - { .compatible = "arm,arm11mp-scu", }, { .compatible = "arm,cortex-a9-scu", }, { .compatible = "arm,cortex-a5-scu", }, { } I see that the arm,arm11mp-scu device is provided by arm-realview-eb-mp.dtsi, which in turn gets included by both arm-realview-eb-11mp.dts and arm-realview-eb-a9mp.dts, so the Cortex-A9 dtb ends up with the wrong compatible string at the moment. The driver doesn't care either way, and we can easily change the dtsi file use the Cortex-A9 compatibles later, but I think we probably want to leave the platsmp code to match the old string as well for compatibility with old dtb files. Arnd _______________________________________________ linux-arm-kernel mailing list linux-arm-kernel@lists.infradead.org http://lists.infradead.org/mailman/listinfo/linux-arm-kernel