| 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
| ||
|
Message-ID: <20251218164209.6ombc6g4gy7tuwcz@pengutronix.de>
Date: Thu, 18 Dec 2025 17:42:09 +0100
From: Marco Felsch <m.felsch@...gutronix.de>
To: Rob Herring <robh@...nel.org>
Cc: Geert Uytterhoeven <geert+renesas@...der.be>,
Magnus Damm <magnus.damm@...il.com>,
Krzysztof Kozlowski <krzk+dt@...nel.org>,
Conor Dooley <conor+dt@...nel.org>, Shawn Guo <shawnguo@...nel.org>,
Sascha Hauer <s.hauer@...gutronix.de>,
Pengutronix Kernel Team <kernel@...gutronix.de>,
Fabio Estevam <festevam@...il.com>,
linux-renesas-soc@...r.kernel.org, devicetree@...r.kernel.org,
linux-kernel@...r.kernel.org, linux-arm-kernel@...ts.infradead.org,
imx@...ts.linux.dev, barebox@...ts.infradead.org,
frieder.schrempf@...tron.de, annette.kobou@...tron.de,
oualid.derouiche@...tron.de, eberhard.stoll@...tron.de
Subject: Re: [PATCH] ARM: dts: nxp/imx6: Drop unused .dtsi
On 25-12-15, Rob Herring wrote:
> On Sun, Dec 14, 2025 at 12:58 PM Marco Felsch <m.felsch@...gutronix.de> wrote:
> >
> > On 25-12-14, Rob Herring wrote:
> > > On Sat, Dec 13, 2025 at 12:13 PM Marco Felsch <m.felsch@...gutronix.de> wrote:
> > > >
> > > > Hi Rob,
> > > >
> > > > On 25-12-12, Rob Herring (Arm) wrote:
> > > > > These .dtsi files are not included anywhere in the tree and can't be
> > > > > tested.
> > > >
> > > > not in the tree but by other projects, please see below.
> > > >
> > > > > Signed-off-by: Rob Herring (Arm) <robh@...nel.org>
> > > > > ---
> > > > > .../dts/nxp/imx/imx6dl-kontron-samx6i.dtsi | 12 -----
> > > > > .../dts/nxp/imx/imx6q-kontron-samx6i.dtsi | 12 -----
> > > > > .../boot/dts/nxp/imx/imx6qdl-pico-dwarf.dtsi | 45 ----------------
> > > > > .../boot/dts/nxp/imx/imx6qdl-pico-nymph.dtsi | 54 -------------------
> > > > > 4 files changed, 123 deletions(-)
> > > > > delete mode 100644 arch/arm/boot/dts/nxp/imx/imx6dl-kontron-samx6i.dtsi
> > > > > delete mode 100644 arch/arm/boot/dts/nxp/imx/imx6q-kontron-samx6i.dtsi
> > > > > delete mode 100644 arch/arm/boot/dts/nxp/imx/imx6qdl-pico-dwarf.dtsi
> > > > > delete mode 100644 arch/arm/boot/dts/nxp/imx/imx6qdl-pico-nymph.dtsi
> > > > >
> > > > > diff --git a/arch/arm/boot/dts/nxp/imx/imx6dl-kontron-samx6i.dtsi b/arch/arm/boot/dts/nxp/imx/imx6dl-kontron-samx6i.dtsi
> > > > > deleted file mode 100644
> > > > > index 5a9b819d7ee8..000000000000
> > > > > --- a/arch/arm/boot/dts/nxp/imx/imx6dl-kontron-samx6i.dtsi
> > > > > +++ /dev/null
> > > > > @@ -1,12 +0,0 @@
> > > > > -// SPDX-License-Identifier: GPL-2.0 OR X11
> > > > > -/*
> > > > > - * Copyright 2019 (C) Pengutronix, Marco Felsch <kernel@...gutronix.de>
> > > > > - */
> > > > > -
> > > > > -#include "imx6dl.dtsi"
> > > > > -#include "imx6qdl-kontron-samx6i.dtsi"
> > > > > -
> > > > > -/ {
> > > > > - model = "Kontron SMARC-sAMX6i Dual-Lite/Solo";
> > > > > - compatible = "kontron,imx6dl-samx6i", "fsl,imx6dl";
> > > > > -};
> > > > > diff --git a/arch/arm/boot/dts/nxp/imx/imx6q-kontron-samx6i.dtsi b/arch/arm/boot/dts/nxp/imx/imx6q-kontron-samx6i.dtsi
> > > > > deleted file mode 100644
> > > > > index e76963436079..000000000000
> > > > > --- a/arch/arm/boot/dts/nxp/imx/imx6q-kontron-samx6i.dtsi
> > > > > +++ /dev/null
> > > > > @@ -1,12 +0,0 @@
> > > > > -// SPDX-License-Identifier: GPL-2.0 OR X11
> > > > > -/*
> > > > > - * Copyright 2019 (C) Pengutronix, Marco Felsch <kernel@...gutronix.de>
> > > > > - */
> > > > > -
> > > > > -#include "imx6q.dtsi"
> > > > > -#include "imx6qdl-kontron-samx6i.dtsi"
> > > > > -
> > > > > -/ {
> > > > > - model = "Kontron SMARC-sAMX6i Quad/Dual";
> > > > > - compatible = "kontron,imx6q-samx6i", "fsl,imx6q";
> > > > > -};
> > > >
> > > > I can't speak for the other two .dtsi files but both kontron .dtsi files
> > > > are used by barebox:
> > > >
> > > > - https://elixir.bootlin.com/barebox/v2025.11.0/source/arch/arm/dts/imx6dl-samx6i.dts#L8
> > > > - https://elixir.bootlin.com/barebox/v2025.11.0/source/arch/arm/dts/imx6q-samx6i.dts#L8
> > > >
> > > > Removing them here will cause a regression for barebox.
> > >
> > > Please upstream something that uses the .dtsi. Other than the
> > > undocumented barebox property, no reason these can't be.
> >
> > My intention wasn't to point to the barebox-only
> > "barebox,disable-deep-probe" property. I was refering to the include
> > line.
> >
> > The two .dtsi files we're talking about are SoM .dtsi files. We don't
> > have access to the Kontron EVK baseboard which uses this SoM.
> >
> > I've added a few Kontron devs which may can add the missing baseboard
> > support.
> >
> > +Cc: frieder.schrempf@...tron.de
> > +Cc: annette.kobou@...tron.de
> > +Cc: oualid.derouiche@...tron.de
> > +Cc: eberhard.stoll@...tron.de
> >
> > I could also add the imx6dl-samx6i.dts and imx6q-samx6i.dts like we do
> > in barebox. But as said, this would be a devicetree for a SoM which
> > makes sense for the bootloader but not for the kernel.
>
> Why not? People use the kernel as a bootloader. There's also some
Using the kernel as bootloader is a good point, which I didn't had in my
mind!
I will add the som.dts, please don't remove the kontron dtsi files yet.
> discussions about splitting SoMs and baseboards into overlays for
> baseboards (and also even into SoC base DT plus overlay for boards).
Do you have pointers?
Regards,
Marco
>
> Rob
>
--
#gernperDu
#CallMeByMyFirstName
Pengutronix e.K. | |
Steuerwalder Str. 21 | https://www.pengutronix.de/ |
31137 Hildesheim, Germany | Phone: +49-5121-206917-0 |
Amtsgericht Hildesheim, HRA 2686 | Fax: +49-5121-206917-9 |
Powered by blists - more mailing lists