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

lists.openwall.net		lists / announce owl-users owl-dev john-users john-dev passwdqc-users yescrypt popa3d-users / oss-security kernel-hardening musl sabotage tlsify passwords / crypt-dev xvendor / Bugtraq Full-Disclosure linux-kernel linux-netdev linux-ext4 linux-hardening linux-cve-announce PHC
Open Source and information security mailing list archives

Hash Suite: Windows password security audit tool. GUI, reports in PDF.

[<prev] [next>] [<thread-prev] [day] [month] [year] [list]

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

On Tue, May 9, 2023 at 4:55 PM Jonathan Neuschäfer
<j.neuschaefer@....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