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 8C2A5C79F99 for ; Tue, 8 Sep 2026 20:10:03 +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=XGzc/5Eb+lr6gHxtsWUeqlpbhH2hHOwRDxbQuElICwQ=; b=y7LV/7GcU5UD4FXS/D533sqyyb nbXi4OFVkW8KqPvmdaLsxmB0vQKNFXuAIUni/1MqbgaaMWMuheJwJP5/sEMwQx9VIKwRW+v2P8F31 jSJmOqmas9Ty0MCTn72/xgI8phqNRSxfhBRWz2GkLSM4VmhbEClQU1Fp5Iea2/jOh59LOndPAdCPW OB4Wo4RDWgbPtWl/KTr2mTvVSMZDnyHEjZ+YiosbituZ4QQpyBWjTbYkAg8rhsZuHVdXRsPFQ4x92 e0L6ODLEhZniZqImhtuRJ6nRnO0gJyW3+y7noWfb2UU7fSb0aWE/r6v88zWhMiXvrTQzLT3Y78LJO ElawGo9A==; Received: from localhost ([::1] helo=bombadil.infradead.org) by bombadil.infradead.org with esmtp (Exim 4.99.1 #2 (Red Hat Linux)) id 1x428j-0000000A9Bs-0Tzy; Tue, 08 Sep 2026 20:09:57 +0000 Received: from flow-b8-smtp.messagingengine.com ([202.12.124.143]) by bombadil.infradead.org with esmtps (Exim 4.99.1 #2 (Red Hat Linux)) id 1x428g-0000000A9BH-48oB for linux-arm-kernel@lists.infradead.org; Tue, 08 Sep 2026 20:09:56 +0000 Received: from ams-compute-02.internal (ams-compute-02.internal [10.64.2.62]) by mailflow.stl.internal (Postfix) with ESMTP id 29C0A1300C52; Tue, 8 Sep 2026 16:09:49 -0400 (EDT) Received: from ams-imap-03 ([10.64.2.23]) by ams-compute-02.internal (MEProxy); Tue, 08 Sep 2026 16:09:54 -0400 DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=arndb.de; h=cc :cc:content-transfer-encoding:content-type:content-type:date :date:from:from:in-reply-to:in-reply-to:message-id:mime-version :references:reply-to:subject:subject:to:to; s=fm3; t=1788898188; x=1788905388; bh=XGzc/5Eb+lr6gHxtsWUeqlpbhH2hHOwRDxbQuElICwQ=; b= Ku2CBzv1lbwKt7FkbK1hK9z6lUhC4lyMvWVuJCDperQCrGR4/rovcYZT14+F2GRU Rav/lw7wDP1LZUNHanxxKPv9TiMXrWJeF4XcFPkUtSvKXHgFfqD1zYYomxsGk/6w D0ugcsY4ct992ReVcs1IUi3KQk3MwK2m0/nVfeh2El0YDoKepsDCsUsJEw2Xopqf vUaW1KTQtRtXeElKC9QKtKuQIKUo2YJjj/bJt4iyEuMRznXurrxMtEOYF0Vypnae hzllfDFfMF4MLpuFLIHlHRQICSgPksOWfpJUZziA6rKH7zrejjbOHOF06vLIb/zc w5HH3y04fc96xLHdT/IkTQ== DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d= messagingengine.com; h=cc:cc:content-transfer-encoding :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:subject:subject:to:to:x-me-proxy :x-me-sender:x-me-sender:x-sasl-enc; s=fm1; t=1788898188; x= 1788905388; bh=XGzc/5Eb+lr6gHxtsWUeqlpbhH2hHOwRDxbQuElICwQ=; b=u jMQYS0sL+Aowo4kNVDUIZ6sxKERcte19OIubGhUn842ROtFnMhzcn9nOmWNq8m7C FHmwKHYgoJ/jG2pkJGkAUm7NezuYNDuHEtea2iCCY1h68vHHRw1tbwvNgBC7NsFY rbLssRXRc0/8Z7/FVbSv77yvARhK4G2eGFJ9PzzEfE+9jQ1o9Hc+VoCcqrJKguIG cVWq5h0ss748M5e43R0TGSageD0VACosHeHSAOqC2oKrcgC+Dywg34eoHUA61vUD Om+SXPAR6IF9uWHBupFJaffqsw5yQdw56xQkgFp2LSYGh5EmlOC0wvW/IaDQZnCb LMTZ1CVIosTouKpnUkAaA== X-ME-Sender: X-ME-Proxy-Cause: dmFkZTEOp8fHm5Y3J5V2OI0TYboevCVDMAAGYDcPknsZa+0X0RAcfHjfIunQUCrjwll0ea rAQKkk63eMYO5/YsUuJodvREtVTrCuYoOIZ9cdLF4PumfhTpYgh50Lfo6o9D8QrGucOErt 1sYxt2QrPbopoRmAf/GENKdf06cSRGVrvjFFQcAbo5nmTHwS3A0bt3HwuabeSgvhnsObAg dbPo0mT2Qbsifo2veGFIaoL0lJSdd83VAXqWQzrdwJ/OSfRVxeUmrfI9jQpSwWye6jKqFi NOft3xXk0NdTzkITEAa33TJA6ix1zF0AwdpfuVEyfJmRNLBy11archzASh1BSGprZa93Ds C+Sv9U/weUns6mx2nwFA2lCN3zLi/Rk8p/J83yqZM7VlcK675+DuzSxAZqfRkMJBci1i1e eqsl5xkRkXCorwVc/2lQanGN1S0IbMaPI/UhcGXT2qI1gtN8VcS4pCweSc1uJ3MgWVP5Fr fd6wuggi9qR2Is2m5jQDxFEcLJqjzcHUFW7JybEqzCT2wTsaTx7pu+YFjg6kopHPn0PbED uooY3D/duB0e3xj8T9BQsm45luJjYM2Rz0k3B09xmHAt8V34sz5JvPIlWK2+nJkOpdV5Fl Crqztnvqs4a/7+lCs3/5TbA3fZD27fPI9F+o3hImahc2L0qBOLFE1RAotMHQ X-ME-Proxy: Feedback-ID: i56a14606:Fastmail Received: by mailuser.ams.internal (Postfix, from userid 501) id 5E56132A0078; Tue, 8 Sep 2026 16:09:34 -0400 (EDT) X-Mailer: MessagingEngine.com Webmail Interface MIME-Version: 1.0 X-ThreadId: AwGOxrB98c9B Date: Tue, 08 Sep 2026 22:09:14 +0200 From: "Arnd Bergmann" To: "Karl Mehltretter" , "Arnd Bergmann" Cc: "Krzysztof Kozlowski" , "Alexandre Belloni" , "Linus Walleij" , "Russell King" , "Drew Fustini" , "Rob Herring" , "Conor Dooley" , "Liviu Dudau" , "Sudeep Holla" , "Lorenzo Pieralisi" , "Frank Li" , "Sascha Hauer" , "Pengutronix Kernel Team" , "Fabio Estevam" , "Vladimir Zapolskiy" , "Stefan Agner" , "Maxime Coquelin" , "Alexandre Torgue" , "Aaro Koskinen" , "Andreas Kemnade" , "Kevin Hilman" , "Roger Quadros" , "Tony Lindgren" , "Nicolas Ferre" , "Claudiu Beznea" , "Andrew Lunn" , "Sebastian Hesselbarth" , "Gregory Clement" , "Robert Jarzmik" , "Kristoffer Ericson" , "Ethan Nelson-Moore" , "Geert Uytterhoeven" , "A. Sverdlin" , "Shawn Guo" , "David Hildenbrand (Red Hat)" , "Nicolas Pitre" , 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 , "Guenter Roeck" Message-Id: In-Reply-To: References: <20260908152808.3928630-1-arnd@kernel.org> <20260908152808.3928630-2-arnd@kernel.org> Subject: Re: [PATCH 01/13] ARM: remove sa1100 platform Content-Type: text/plain Content-Transfer-Encoding: 7bit X-CRM114-Version: 20100106-BlameMichelson ( TRE 0.9.0 (BSD) ) MR-646709E3 X-CRM114-CacheID: sfid-20260908_130955_075957_8001D0B7 X-CRM114-Status: GOOD ( 18.41 ) 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 Tue, Sep 8, 2026, at 21:39, Karl Mehltretter wrote: > On Tue, Sep 08, 2026 at 05:27:51PM +0100, Arnd Bergmann wrote: >> 62 files changed, 19 insertions(+), 10501 deletions(-) > > Note that Sashiko skipped reviewing this patch, because it exceeded > its current 10000 line limit. > >> diff --git a/drivers/rtc/rtc-sa1100.c b/drivers/rtc/rtc-sa1100.c >> @@ -275,18 +275,10 @@ static int sa1100_rtc_probe(struct platform_device *pdev) >> - if (IS_ENABLED(CONFIG_ARCH_SA1100) || >> - of_device_is_compatible(pdev->dev.of_node, "mrvl,sa1100-rtc")) { >> - info->rcnr = base + 0x04; >> - info->rtsr = base + 0x10; >> - info->rtar = base + 0x00; >> - info->rttr = base + 0x08; >> - } else { >> - info->rcnr = base + 0x0; >> - info->rtsr = base + 0x8; >> - info->rtar = base + 0x4; >> - info->rttr = base + 0xc; >> - } >> + info->rcnr = base + 0x0; >> + info->rtsr = base + 0x8; >> + info->rtar = base + 0x4; >> + info->rttr = base + 0xc; > > This drops the layout switch but keeps "mrvl,sa1100-rtc" in > sa1100_rtc_dt_ids[] and in bindings/rtc/sa1100-rtc.yaml, so a DT with > that compatible would now silently get the MMP register offsets instead > of the SA1100 ones. This one is interesting: this was already the behavior before my patch, as CONFIG_ARCH_SA1100 is mutually exclusive with all other targets. That obviously wasn't the intention of the code. I've removed the sa1100_rtc_dt_ids[] entry now. >> diff --git a/drivers/watchdog/Kconfig b/drivers/watchdog/Kconfig >> @@ -614,16 +614,16 @@ config S3C2410_WATCHDOG >> be called s3c2410_wdt. >> >> config SA1100_WATCHDOG >> - tristate "SA1100/PXA2xx watchdog" >> - depends on ARCH_SA1100 || ARCH_PXA || COMPILE_TEST >> + tristate "PXA2xx watchdog" >> + depends on ARCH_PXA || COMPILE_TEST >> help >> - Watchdog timer embedded into SA11x0 and PXA2xx chips. This will >> - reboot your system when timeout is reached. >> + Watchdog timer embedded PXA2xx chips. This will reboot your >> + system when timeout is reached. >> > > "embedded into PXA2xx chips" is better. > > The RTC_DRV_SA1100 help text a bit earlier in the patch also still says > "built into your SA11x0 or PXA2xx CPU". Fixed all these now, thanks for the review! Arnd