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.

Regards,
  Marco


> diff --git a/arch/arm/boot/dts/nxp/imx/imx6qdl-pico-dwarf.dtsi 
> b/arch/arm/boot/dts/nxp/imx/imx6qdl-pico-dwarf.dtsi
> deleted file mode 100644
> index 3a968782e854..000000000000
> --- a/arch/arm/boot/dts/nxp/imx/imx6qdl-pico-dwarf.dtsi
> +++ /dev/null
> @@ -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
> -             >;
> -     };
> -};
> diff --git a/arch/arm/boot/dts/nxp/imx/imx6qdl-pico-nymph.dtsi 
> b/arch/arm/boot/dts/nxp/imx/imx6qdl-pico-nymph.dtsi
> deleted file mode 100644
> index 3d56a4216448..000000000000
> --- a/arch/arm/boot/dts/nxp/imx/imx6qdl-pico-nymph.dtsi
> +++ /dev/null
> @@ -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 = <&reg_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
> -             >;
> -     };
> -};
> -- 
> 2.51.0
> 
> 
> 

-- 
#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    |

Reply via email to