On 12.01.22 09:57, Heiko Thiery wrote: > When using a board variant that selects the lvds specific dtb the > *.u-boot.dtsi file will not be included. To have a lvds dtb specific > u-boot.dtsi file move this part to a common board u-boot.dtsi file and > include this in the board base u-boot.dtsi and create an additional one > for the lvds variant. > > Signed-off-by: Heiko Thiery <[email protected]>
Reviewed-by: Frieder Schrempf <[email protected]> > --- > .../imx8mm-kontron-n801x-s-lvds-u-boot.dtsi | 6 + > .../dts/imx8mm-kontron-n801x-s-u-boot.dtsi | 124 +---------------- > arch/arm/dts/imx8mm-kontron-n801x-u-boot.dtsi | 128 ++++++++++++++++++ > 3 files changed, 135 insertions(+), 123 deletions(-) > create mode 100644 arch/arm/dts/imx8mm-kontron-n801x-s-lvds-u-boot.dtsi > create mode 100644 arch/arm/dts/imx8mm-kontron-n801x-u-boot.dtsi > > diff --git a/arch/arm/dts/imx8mm-kontron-n801x-s-lvds-u-boot.dtsi > b/arch/arm/dts/imx8mm-kontron-n801x-s-lvds-u-boot.dtsi > new file mode 100644 > index 0000000000..4bf75722bf > --- /dev/null > +++ b/arch/arm/dts/imx8mm-kontron-n801x-s-lvds-u-boot.dtsi > @@ -0,0 +1,6 @@ > +// SPDX-License-Identifier: GPL-2.0+ > +/* > + * Copyright (C) 2019 Kontron Electronics GmbH > + */ > + > +#include "imx8mm-kontron-n801x-u-boot.dtsi" > diff --git a/arch/arm/dts/imx8mm-kontron-n801x-s-u-boot.dtsi > b/arch/arm/dts/imx8mm-kontron-n801x-s-u-boot.dtsi > index 22d18e6f1c..4bf75722bf 100644 > --- a/arch/arm/dts/imx8mm-kontron-n801x-s-u-boot.dtsi > +++ b/arch/arm/dts/imx8mm-kontron-n801x-s-u-boot.dtsi > @@ -3,126 +3,4 @@ > * Copyright (C) 2019 Kontron Electronics GmbH > */ > > -#include "imx8mm-u-boot.dtsi" > - > -/ { > - aliases { > - usb0 = &usbotg1; > - usb1 = &usbotg2; > - }; > - > - wdt-reboot { > - compatible = "wdt-reboot"; > - wdt = <&wdog1>; > - u-boot,dm-spl; > - }; > - > - firmware { > - optee { > - compatible = "linaro,optee-tz"; > - method = "smc"; > - }; > - }; > -}; > - > -&fec1 { > - phy-mode = "rgmii-rxid"; > -}; > - > -&i2c1 { > - u-boot,dm-spl; > - u-boot,dm-pre-reloc; > -}; > - > -&i2c2 { > - status = "okay"; > - u-boot,dm-spl; > - u-boot,dm-pre-reloc; > -}; > - > -&pinctrl_ecspi1 { > - u-boot,dm-spl; > -}; > - > -&pinctrl_i2c1 { > - u-boot,dm-spl; > -}; > - > -&pinctrl_pmic { > - u-boot,dm-spl; > - fsl,pins = < > - MX8MM_IOMUXC_GPIO1_IO00_GPIO1_IO0 0x141 > - /* Disable Pullup for SD_VSEL */ > - MX8MM_IOMUXC_GPIO1_IO04_GPIO1_IO4 0x41 > - >; > -}; > - > -&pinctrl_uart3 { > - u-boot,dm-spl; > - u-boot,dm-pre-reloc; > -}; > - > -&pinctrl_usdhc1 { > - u-boot,dm-spl; > -}; > - > -&pinctrl_usdhc1_100mhz { > - u-boot,dm-spl; > -}; > - > -&pinctrl_usdhc1_200mhz { > - u-boot,dm-spl; > -}; > - > -&pinctrl_usdhc2 { > - u-boot,dm-spl; > -}; > - > -&pca9450 { > - u-boot,dm-spl; > -}; > - > -&{/soc@0/bus@30800000/i2c@30a20000/pmic@25/regulators} { > - u-boot,dm-spl; > -}; > - > -&ecspi1 { > - u-boot,dm-spl; > -}; > - > -&gpio1 { > - u-boot,dm-spl; > -}; > - > -&gpio2 { > - u-boot,dm-spl; > -}; > - > -&gpio3 { > - u-boot,dm-spl; > -}; > - > -&gpio4 { > - u-boot,dm-spl; > -}; > - > -&gpio5 { > - u-boot,dm-spl; > -}; > - > -&uart3 { > - u-boot,dm-spl; > - u-boot,dm-pre-reloc; > -}; > - > -&usdhc1 { > - u-boot,dm-spl; > -}; > - > -&usdhc2 { > - u-boot,dm-spl; > -}; > - > -&wdog1 { > - u-boot,dm-spl; > -}; > +#include "imx8mm-kontron-n801x-u-boot.dtsi" > diff --git a/arch/arm/dts/imx8mm-kontron-n801x-u-boot.dtsi > b/arch/arm/dts/imx8mm-kontron-n801x-u-boot.dtsi > new file mode 100644 > index 0000000000..22d18e6f1c > --- /dev/null > +++ b/arch/arm/dts/imx8mm-kontron-n801x-u-boot.dtsi > @@ -0,0 +1,128 @@ > +// SPDX-License-Identifier: GPL-2.0+ > +/* > + * Copyright (C) 2019 Kontron Electronics GmbH > + */ > + > +#include "imx8mm-u-boot.dtsi" > + > +/ { > + aliases { > + usb0 = &usbotg1; > + usb1 = &usbotg2; > + }; > + > + wdt-reboot { > + compatible = "wdt-reboot"; > + wdt = <&wdog1>; > + u-boot,dm-spl; > + }; > + > + firmware { > + optee { > + compatible = "linaro,optee-tz"; > + method = "smc"; > + }; > + }; > +}; > + > +&fec1 { > + phy-mode = "rgmii-rxid"; > +}; > + > +&i2c1 { > + u-boot,dm-spl; > + u-boot,dm-pre-reloc; > +}; > + > +&i2c2 { > + status = "okay"; > + u-boot,dm-spl; > + u-boot,dm-pre-reloc; > +}; > + > +&pinctrl_ecspi1 { > + u-boot,dm-spl; > +}; > + > +&pinctrl_i2c1 { > + u-boot,dm-spl; > +}; > + > +&pinctrl_pmic { > + u-boot,dm-spl; > + fsl,pins = < > + MX8MM_IOMUXC_GPIO1_IO00_GPIO1_IO0 0x141 > + /* Disable Pullup for SD_VSEL */ > + MX8MM_IOMUXC_GPIO1_IO04_GPIO1_IO4 0x41 > + >; > +}; > + > +&pinctrl_uart3 { > + u-boot,dm-spl; > + u-boot,dm-pre-reloc; > +}; > + > +&pinctrl_usdhc1 { > + u-boot,dm-spl; > +}; > + > +&pinctrl_usdhc1_100mhz { > + u-boot,dm-spl; > +}; > + > +&pinctrl_usdhc1_200mhz { > + u-boot,dm-spl; > +}; > + > +&pinctrl_usdhc2 { > + u-boot,dm-spl; > +}; > + > +&pca9450 { > + u-boot,dm-spl; > +}; > + > +&{/soc@0/bus@30800000/i2c@30a20000/pmic@25/regulators} { > + u-boot,dm-spl; > +}; > + > +&ecspi1 { > + u-boot,dm-spl; > +}; > + > +&gpio1 { > + u-boot,dm-spl; > +}; > + > +&gpio2 { > + u-boot,dm-spl; > +}; > + > +&gpio3 { > + u-boot,dm-spl; > +}; > + > +&gpio4 { > + u-boot,dm-spl; > +}; > + > +&gpio5 { > + u-boot,dm-spl; > +}; > + > +&uart3 { > + u-boot,dm-spl; > + u-boot,dm-pre-reloc; > +}; > + > +&usdhc1 { > + u-boot,dm-spl; > +}; > + > +&usdhc2 { > + u-boot,dm-spl; > +}; > + > +&wdog1 { > + u-boot,dm-spl; > +};

