Re: [RFC PATCH 0/1] Categorize ARM dts directory

From: Rob Herring <robh+dt@kernel.org>
To: "Jonathan Neuschäfer" <j.neuschaefer@gmx.net>
Cc: Arnd Bergmann <arnd@arndb.de>,
	linux-aspeed@lists.ozlabs.org,
	linux-realtek-soc@lists.infradead.org, linux-arm-kernel@axis.com,
	linux-stm32@st-md-mailman.stormreply.com,
	chrome-platform@lists.linux.dev,
	linux-samsung-soc@vger.kernel.org, openbmc@lists.ozlabs.org,
	Krzysztof Kozlowski <krzk@kernel.org>,
	linux-rockchip@lists.infradead.org,
	Geert Uytterhoeven <geert@linux-m68k.org>,
	linux-sunxi@lists.linux.dev, devicetree@vger.kernel.org,
	linux-arm-msm@vger.kernel.org, linux-actions@lists.infradead.org,
	linux-unisoc@lists.infradead.org,
	linux-mediatek@lists.infradead.org,
	linux-rpi-kernel@lists.infradead.org,
	linux-tegra@vger.kernel.org, linux-amlogic@lists.infradead.org,
	Linux-OMAP <linux-omap@vger.kernel.org>,
	linux-arm-kernel@lists.infradead.org,
	linux-kernel@vger.kernel.org,
	Christian Marangi <ansuelsmth@gmail.com>,
	Linux-Renesas <linux-renesas-soc@vger.kernel.org>,
	kernel@dh-electronics.com, Olof Johansson <olof@lixom.net>,
	Krzysztof Kozlowski <krzk+dt@kernel.org>,
	"linux-oxnas@groups.io" <linux-oxnas@groups.io>
Subject: Re: [RFC PATCH 0/1] Categorize ARM dts directory
Date: Thu, 8 Jun 2023 14:33:53 -0600	[thread overview]
Message-ID: <CAL_JsqLTj_L-V8HR=TzO6+r9Xew=yivaKG1ngCn+NCjgPZwZzw@mail.gmail.com> (raw)
In-Reply-To: <ZFrPJQdwoxqFpzUO@probook>

On Tue, May 9, 2023 at 4:55 PM Jonathan Neuschäfer
<j.neuschaefer@gmx.net> wrote:
>
> On Tue, May 02, 2023 at 02:40:19PM -0500, Rob Herring wrote:
> [...]
> > I've dusted off my script and made a branch[1] with the result.
> > There's just a couple of fixes needed after the script is run (see the
> > top commit). The cross arch includes are all fixed up by the script.
> > dtbs_install maintains a flat install. I compared the number of .dtbs
> > before and after to check the script.
> >
> > I think the only issue remaining is finalizing the mapping of
> > platforms to subdirs. What I have currently is a mixture of SoC
> > families and vendors. The most notable are all the Freescale/NXP
> > platforms, pxa, socfpga, and stm32. It's not consistent with arm64
> > either. Once that's finalized, I still need to go update MAINTAINERS.
> >
> > Here's the current mapping:
> >
> > vendor_map = {
> [...]
> >     'aspeed' : 'aspeed',
> >     'ast2' : 'aspeed',
> >     'facebook' : 'aspeed',
> >     'ibm' : 'aspeed',
>
> >     'openbmc' : 'aspeed',
>
> The openbmc flash layouts are currently only used by aspeed devicetrees,
> but they don't really depend on any aspeed details. It would be possible
> to reuse them in Nuvoton BMC devicetrees in the future, for example.
>
> In that sense, I think putting them in a separate "openbmc" directory
> would be slightly better.

Could be used on arm64 or riscv too at some point. We do some cross
arch includes, but IMO it would be better to move to
include/dt-bindings/ or somewhere outside of arch/. Other common
things I didn't move. I could do that here too. I prefer to that the
sub-directories are just chip vendors/families.

Rob