diff mbox series

[2/2] imx: imx8mm: imx8mm-kontron-n801x-s: add common board u-boot.dtsi

Message ID 20220112085755.7396-2-heiko.thiery@gmail.com
State Accepted
Commit 1bcda4efed931bb72bfe8a6fbe0f0e8170a5f9ba
Delegated to: Stefano Babic
Headers show
Series [1/2] imx: imx8mm: imx8mm-kontron-n801x-s: convert options to Kconfig | expand

Commit Message

Heiko Thiery Jan. 12, 2022, 8:57 a.m. UTC
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 <heiko.thiery@gmail.com>
---
 .../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

Comments

Frieder Schrempf Jan. 12, 2022, 1:03 p.m. UTC | #1
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 <heiko.thiery@gmail.com>

Reviewed-by: Frieder Schrempf <frieder.schrempf@kontron.de>

> ---
>  .../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;
> +};
Stefano Babic Feb. 5, 2022, 4:42 p.m. UTC | #2
> 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 <heiko.thiery@gmail.com>
> Reviewed-by: Frieder Schrempf <frieder.schrempf@kontron.de>
Applied to u-boot-imx, master, thanks !

Best regards,
Stefano Babic
diff mbox series

Patch

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;
+};