ARM: dts: nxp/imx6: Drop unused .dtsi
Subsystems (2)
Rob Herring (Arm)
maintainerPATCH 1/1 ARM: dts: nxp/imx6: Drop unused .dtsi
These .dtsi files are not included anywhere in the tree and can't be tested.
--- .../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
arch/arm/boot/dts/nxp/imx/imx6dl-kontron-samx6i.dtsi deleted +0 -12
@@ -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"; -};
arch/arm/boot/dts/nxp/imx/imx6q-kontron-samx6i.dtsi deleted +0 -12
@@ -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"; -};
arch/arm/boot/dts/nxp/imx/imx6qdl-pico-dwarf.dtsi deleted +0 -45
45 lines not on this page · raw message
arch/arm/boot/dts/nxp/imx/imx6qdl-pico-nymph.dtsi deleted +0 -54
54 lines not on this page · raw message
Signature · 1 line
2.51.0
Hi Rob,
Quoted 2 lines · Rob Herring (Arm)
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.
Quoted 38 lines
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.dtsiarch/arm/boot/dts/nxp/imx/imx6dl-kontron-samx6i.dtsi deleted +0
-12@@ -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"; -};arch/arm/boot/dts/nxp/imx/imx6q-kontron-samx6i.dtsi deleted +0
-2@@ -1,12 +0,0 @@ -// SPDX-License-Identifier: GPL-2.0 OR X11 -/*
arch/arm/boot/dts/nxp/imx/imx6q-kontron-samx6i.dtsi line 12
Quoted 10 lines
arch/arm/boot/dts/nxp/imx/imx6q-kontron-samx6i.dtsi deleted +0
-10- * 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.
Regards,
Marco
Quoted 116 lines
arch/arm/boot/dts/nxp/imx/imx6qdl-pico-dwarf.dtsi deleted +0
-45@@ -1,45 +0,0 @@ -// SPDX-License-Identifier: GPL-2.0+ OR MIT -// -// Copyright 2017 NXP - -#include "imx6qdl-pico.dtsi" - -/ { - leds { - compatible = "gpio-leds"; - pinctrl-names = "default"; - pinctrl-0 = <&pinctrl_gpio_leds>; - - led { - label = "gpio-led"; - gpios = <&gpio5 31 GPIO_ACTIVE_HIGH>; - }; - }; - -}; - -&i2c1 { - mpl3115@60 { - compatible = "fsl,mpl3115"; - reg = <0x60>; - }; -}; - -&i2c2 { - io-expander@25 { - compatible = "nxp,pca9554"; - reg = <0x25>; - gpio-controller; - #gpio-cells = <2>; - #interrupt-cells = <2>; - }; - -}; - -&iomuxc { - pinctrl_gpio_leds: gpioledsgrp { - fsl,pins = < - MX6QDL_PAD_CSI0_DAT13__GPIO5_IO31 0x1b0b0 - >; - }; -};arch/arm/boot/dts/nxp/imx/imx6qdl-pico-nymph.dtsi deleted +0
-54@@ -1,54 +0,0 @@ -// SPDX-License-Identifier: GPL-2.0+ OR MIT - -#include "imx6qdl-pico.dtsi" - -/ { - leds { - compatible = "gpio-leds"; - pinctrl-names = "default"; - pinctrl-0 = <&pinctrl_gpio_leds>; - - led { - label = "gpio-led"; - gpios = <&gpio5 31 GPIO_ACTIVE_HIGH>; - }; - }; - -}; - -&i2c1 { - adc@52 { - compatible = "ti,adc081c"; - reg = <0x52>; - vref-supply = <®_2p5v>; - }; -}; - -&i2c2 { - io-expander@25 { - compatible = "nxp,pca9554"; - reg = <0x25>; - gpio-controller; - #gpio-cells = <2>; - #interrupt-cells = <2>; - }; -}; - -&i2c3 { - rtc@68 { - compatible = "dallas,ds1337"; - reg = <0x68>; - }; -}; - -&pcie { - status = "okay"; -}; - -&iomuxc { - pinctrl_gpio_leds: gpioledsgrp { - fsl,pins = < - MX6QDL_PAD_CSI0_DAT13__GPIO5_IO31 0x1b0b0 - >; - }; -};Signature · 1 line
2.51.0
Signature · 7 lines
#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 |
Rob Herring
maintainerQuoted 61 lines · Marco Felsch
Hi Rob,
Quoted 2 lines · Rob Herring (Arm)
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.
Quoted 38 lines
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.dtsiarch/arm/boot/dts/nxp/imx/imx6dl-kontron-samx6i.dtsi deleted +0
-12@@ -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"; -};arch/arm/boot/dts/nxp/imx/imx6q-kontron-samx6i.dtsi deleted +0
-2@@ -1,12 +0,0 @@ -// SPDX-License-Identifier: GPL-2.0 OR X11 -/*arch/arm/boot/dts/nxp/imx/imx6q-kontron-samx6i.dtsi line 12
Quoted 10 lines
arch/arm/boot/dts/nxp/imx/imx6q-kontron-samx6i.dtsi deleted +0
-10- * 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:
Quoted 4 lines
- 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#L8Removing 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.
Rob
Quoted 62 lines · Rob Herring
Quoted 61 lines · Marco Felsch
Hi Rob,
Quoted 2 lines · Rob Herring (Arm)
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.
Quoted 38 lines
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.dtsiarch/arm/boot/dts/nxp/imx/imx6dl-kontron-samx6i.dtsi deleted +0
-12@@ -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"; -};arch/arm/boot/dts/nxp/imx/imx6q-kontron-samx6i.dtsi deleted +0
-2@@ -1,12 +0,0 @@ -// SPDX-License-Identifier: GPL-2.0 OR X11 -/*arch/arm/boot/dts/nxp/imx/imx6q-kontron-samx6i.dtsi line 12
Quoted 10 lines
arch/arm/boot/dts/nxp/imx/imx6q-kontron-samx6i.dtsi deleted +0
-10- * 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:
Quoted 7 lines
Quoted 4 lines
- 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#L8Removing 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.
Regards,
Marco
Quoted 3 lines
Rob
Signature · 7 lines
#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 |
Rob Herring
maintainerQuoted 87 lines · Marco Felsch
Quoted 62 lines · Rob Herring
Quoted 61 lines · Marco Felsch
Hi Rob,
Quoted 2 lines · Rob Herring (Arm)
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.
Quoted 38 lines
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.dtsiarch/arm/boot/dts/nxp/imx/imx6dl-kontron-samx6i.dtsi deleted +0
-12@@ -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"; -};arch/arm/boot/dts/nxp/imx/imx6q-kontron-samx6i.dtsi deleted +0
-2@@ -1,12 +0,0 @@ -// SPDX-License-Identifier: GPL-2.0 OR X11 -/*arch/arm/boot/dts/nxp/imx/imx6q-kontron-samx6i.dtsi line 12
Quoted 10 lines
arch/arm/boot/dts/nxp/imx/imx6q-kontron-samx6i.dtsi deleted +0
-10- * 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:
Quoted 7 lines
Quoted 4 lines
- 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#L8Removing 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.
(quoted diff) +4
-0+Cc: [email protected] +Cc: [email protected] +Cc: [email protected] +Cc: [email protected]
Quoted 3 lines
I could also add the
imx6dl-samx6i.dtsandimx6q-samx6i.dtslike 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 discussions about splitting SoMs and baseboards into overlays for baseboards (and also even into SoC base DT plus overlay for boards).
Rob
Quoted 88 lines · Rob Herring
Quoted 87 lines · Marco Felsch
Quoted 62 lines · Rob Herring
Quoted 61 lines · Marco Felsch
Hi Rob,
Quoted 2 lines · Rob Herring (Arm)
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.
Quoted 38 lines
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.dtsiarch/arm/boot/dts/nxp/imx/imx6dl-kontron-samx6i.dtsi deleted +0
-12@@ -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"; -};arch/arm/boot/dts/nxp/imx/imx6q-kontron-samx6i.dtsi deleted +0
-2@@ -1,12 +0,0 @@ -// SPDX-License-Identifier: GPL-2.0 OR X11 -/*arch/arm/boot/dts/nxp/imx/imx6q-kontron-samx6i.dtsi line 12
Quoted 10 lines
arch/arm/boot/dts/nxp/imx/imx6q-kontron-samx6i.dtsi deleted +0
-10- * 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:
Quoted 7 lines
Quoted 4 lines
- 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#L8Removing 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.
(quoted diff) +4
-0+Cc: [email protected] +Cc: [email protected] +Cc: [email protected] +Cc: [email protected]
Quoted 5 lines
Quoted 3 lines
I could also add the
imx6dl-samx6i.dtsandimx6q-samx6i.dtslike 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.
Quoted 2 lines
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
Quoted 3 lines
Rob
Signature · 7 lines
#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 |