On 25-12-15, Rob Herring wrote: > On Sun, Dec 14, 2025 at 12:58 PM Marco Felsch <[email protected]> wrote: > > > > On 25-12-14, Rob Herring wrote: > > > On Sat, Dec 13, 2025 at 12:13 PM Marco Felsch <[email protected]> > > > 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) <[email protected]> > > > > > --- > > > > > .../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 > > > > > <[email protected]> > > > > > - */ > > > > > - > > > > > -#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 > > > > > <[email protected]> > > > > > - */ > > > > > - > > > > > -#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: [email protected] > > +Cc: [email protected] > > +Cc: [email protected] > > +Cc: [email protected] > > > > 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 |
