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 EEA39C433FE for ; Tue, 8 Nov 2022 13:32:52 +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:In-Reply-To:MIME-Version:References: Message-ID:Subject:CC:To:From:Date:Reply-To:Content-ID:Content-Description: Resent-Date:Resent-From:Resent-Sender:Resent-To:Resent-Cc:Resent-Message-ID: List-Owner; bh=cJBGNVeaHYW+FUfzf0pXtugyinMR1FVeouyyDDNHP9E=; b=NbFiTXrJvueHLa Css0zeD00D9JxE0bPiMNvoJZTzvpA95KsIqtTKtlsbzGPWrMdyIA9RFTB/drvJRhe6VB26qLQDEEU ZJ0GJLfia6AUxjVIjSYienOJZAJpnj3COviMIJE42gBPTqv47V87b4GxL0H9GJVgv6+et+Hj626k9 K18ODIfGY6X59t8wUpYn9KTidY9JHZwZ3L98MPSHOLn/lf3XCqQM8KrLQgGAV6upVqY+jENa9iDHE ShToNLtDaKXq/XesuN36Pbg5pSNB3b354wZ8Gc41/DZISUS9JbhI2ZReyKREZMpSyXlcAZmS5n8rS bX/PyYrc8CQf7jLyWFyQ==; Received: from localhost ([::1] helo=bombadil.infradead.org) by bombadil.infradead.org with esmtp (Exim 4.94.2 #2 (Red Hat Linux)) id 1osOiW-005czp-5v; Tue, 08 Nov 2022 13:32:40 +0000 Received: from esa.microchip.iphmx.com ([68.232.153.233]) by bombadil.infradead.org with esmtps (Exim 4.94.2 #2 (Red Hat Linux)) id 1osOiS-005cyt-85 for linux-riscv@lists.infradead.org; Tue, 08 Nov 2022 13:32:38 +0000 DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/simple; d=microchip.com; i=@microchip.com; q=dns/txt; s=mchp; t=1667914355; x=1699450355; h=date:from:to:cc:subject:message-id:references: mime-version:in-reply-to; bh=huxLAOzGx/LqzcIAiUSaJnN/HGBFm4LrG5K4GV5wwAk=; b=hVksi7ibAYIiS4+O5t9aUolztJgiE63q/WOPD/z49WeaePTVzUPdsuRU 3Ky8YjZ8ozkgVsnQRQ14GmR0LDSd2JfWUdnabKONFUZmOprG42TbII34G RP2jSvaBeLjkuh+PPncgSrh5LKpGNctA/ABGdTl96zmASFmtUm5OmfNqP JcTTZ2SSoslz+6eauv/euLbj0R/6LKCSnOSjns0dJSNVRK8wLyy7GveZr VW3edpVPmkiAVpo2EYZTIHEyldiksPpcHsoyWS6NlmDxJpeX0yFjGbG+D aDqZ9IYwIySKVyHZAdbF5HxJetWcVj2fS71X9guu57N1lgCsmXmN1HneR A==; X-IronPort-AV: E=Sophos;i="5.96,147,1665471600"; d="scan'208";a="188159649" Received: from unknown (HELO email.microchip.com) ([170.129.1.10]) by esa3.microchip.iphmx.com with ESMTP/TLS/AES256-SHA256; 08 Nov 2022 06:32:30 -0700 Received: from chn-vm-ex03.mchp-main.com (10.10.85.151) by chn-vm-ex04.mchp-main.com (10.10.85.152) with Microsoft SMTP Server (version=TLS1_2, cipher=TLS_ECDHE_RSA_WITH_AES_128_GCM_SHA256) id 15.1.2507.12; Tue, 8 Nov 2022 06:32:30 -0700 Received: from wendy (10.10.115.15) by chn-vm-ex03.mchp-main.com (10.10.85.151) with Microsoft SMTP Server (version=TLS1_2, cipher=TLS_ECDHE_RSA_WITH_AES_128_GCM_SHA256) id 15.1.2507.12 via Frontend Transport; Tue, 8 Nov 2022 06:32:28 -0700 Date: Tue, 8 Nov 2022 13:32:12 +0000 From: Conor Dooley To: Arnd Bergmann , , CC: Conor Dooley , Palmer Dabbelt , , Krzysztof Kozlowski , Masahiro Yamada , Marc Kleine-Budde , "David S . Miller" , , Subject: Re: Should we merge arch/riscv/boot/dts via the SOC tree? Message-ID: References: MIME-Version: 1.0 Content-Disposition: inline In-Reply-To: X-CRM114-Version: 20100106-BlameMichelson ( TRE 0.8.0 (BSD) ) MR-646709E3 X-CRM114-CacheID: sfid-20221108_053236_359795_85CAA641 X-CRM114-Status: GOOD ( 45.31 ) X-BeenThere: linux-riscv@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-riscv" Errors-To: linux-riscv-bounces+linux-riscv=archiver.kernel.org@lists.infradead.org +CC Nicolas On Tue, Nov 08, 2022 at 01:51:37PM +0100, Arnd Bergmann wrote: > On Mon, Nov 7, 2022, at 19:31, Conor Dooley wrote: > > On Mon, Nov 07, 2022 at 08:46:00AM -0800, Palmer Dabbelt wrote: > >> This has come up a bunch of times, but I don't think we've ever really > >> made a decision. Historically that's not been such a big deal because > >> the RISC-V device trees were pretty inactive, but that's changed -- both > >> because Conor has been cleaning everything up, and also because there's > >> a bunch of SOCs showing up with RISC-V cores in them. We talked about > >> this again at plumbers a few times, but Arnd wasn't around it person so > >> I figured it's best to just start an email thread and see how people > >> feel. > >> > >> A lot of these new SOCs are based on Arm designs and the device trees > >> very much reflect that, so it makes sense to me to just keep the device > >> tree merges via as similar a path as possible. IIUC that happens via > >> the SOC tree these days, so it makes sense to me that we start handling > >> the RISC-V device trees that way as well. That would make things easier > >> for contributors, as they'll have one workflow for all their SOCs, but > >> also easier for me as a lot of this SOC stuff touches bits I really > >> don't understand and thus get kind of lost trying to review. > > > > Reviewing the Renesas/Allwinner stuff, it's p apparent to me that they > > need to go via the same tree for RISC-V and ARM. > > > >> Arnd: looks like you're handling most of the merges these days so this > >> would be increasing your workload. I feel kind of bad just dumping a > >> bunch of stuff on you, but I think at least now the RISC-V DTS are in > >> reasonable shape so hopefully it's not that bad. > > > > Warning free at least... :) > > I don't see a problem with merging this through the SoC tree, it > was always the plan to pick up related changes across architectures > where needed. > > The MIPS and PowerPC maintainers have so far preferred to handle > the changes through their respective trees, everything else > has been in the noise. Looking at the number of DT changesets since > linux-5.0 per architecture, I see > > 7748 arch/arm64/boot/dts > 6654 arch/arm/boot/dts > 197 arch/mips/boot/dts > 155 arch/riscv/boot/dts > 67 arch/powerpc/boot/dts > 35 arch/arc/boot/dts > 6 arch/openrisc/boot/dts > 5 arch/xtensa/boot/dts > 5 arch/nios2/boot/dts > 5 arch/microblaze/boot/dts > 2 arch/csky/boot/dts > 1 arch/loongarch/boot/dts > > >> On a somewhat related note, Conor has offered to pick up the otherwise > >> unmaintained RISC-V SOCs. That's sort of its own discussion, but if we > >> change over to the SOC tree we might as well just do everything at the > >> same time. > >> > >> Presumably we'd want to adjust the MAINTAINERS file in a handful of ways > >> to make sure patches end up in the right place. > > > > Arnd mentioned that that should cover stuff in drivers/{soc,firmware} as > > well as the dt, so with the assumption that that MAINTAINERS entry looks > > something like: > > > > diff --git a/MAINTAINERS b/MAINTAINERS > > index cf0f18502372..03e78d2e5cc6 100644 > > --- a/MAINTAINERS > > +++ b/MAINTAINERS > > @@ -17709,6 +17709,16 @@ F: arch/riscv/ > > N: riscv > > K: riscv > > > > +RISC-V/MISC SOC SUPPORT > > +M: Conor Dooley > > +L: linux-riscv@lists.infradead.org > > +S: Maintained > > +Q: https://patchwork.kernel.org/project/linux-riscv/list/ > > +T: git https://git.kernel.org/pub/scm/linux/kernel/git/conor/linux.git/ > > +F: arch/riscv/boot/dts/ > > +F: drivers/soc/microchip/ > > +F: drivers/soc/sifive/ > > I'd probably make separate entries here, at least for the > drivers/soc/microchip directory, I can see that being shared with > architectures other than RISC-V in the future (Added Nicolas to CC so that he's in the loop) Uh sure. It'd crossed my mind, but I filed it away in the "may happen some day" category. The arm stuff is going via the atmel directory at the moment so I was operating on the basis of "do it this way until something changes". Splitting is fine by me. As things stand, anything drivers/soc/microchip already CCs the linux-riscv list so maybe that can change alongside this. > and the sifive > directory should probably have at least a reviewer from > sifive.com even if you plan to route the patches my way for > them. Anything sifive should already be covered by: SIFIVE DRIVERS M: Palmer Dabbelt M: Paul Walmsley L: linux-riscv@lists.infradead.org S: Supported T: git https://github.com/sifive/riscv-linux.git N: sifive K: [^@]sifive The git tree there is dead, but it does give you your SiFive reviewer? That leaves us with three entries then? Grand with me, I don't care :) Created the above to double check the scope more than anything else. The one I was wondering about but forgot to mention was: Documentation/devicetree/bindings/riscv/ It's mostly definitions of cpu, soc and board compatibles, so I figure it could go with the dt stuff - and it's covered by the generic RISC-V entry for the changes that reflect extensions etc. Thanks, Conor. _______________________________________________ linux-riscv mailing list linux-riscv@lists.infradead.org http://lists.infradead.org/mailman/listinfo/linux-riscv