[U-Boot] [PATCH 1/1] arm: zynq: Fix device tree for Avnet Picozed boards

[U-Boot] [PATCH 1/1] arm: zynq: Fix device tree for Avnet Picozed boards

[Andreas Galauner]

This change adds the necessary /chosen/stdout-path to the device tree
for the Avnet Picozed boards. This node is mandatory for the SPL. Without
it the board resets in a loop without any output. The change also adds
the USB, QSPI and Ethernet MAC nodes of the corresponding devices present
on the board.

Signed-off-by: Andreas Galauner <andreas@galauner.de>
---
 arch/arm/dts/zynq-picozed.dts | 40 +++++++++++++++++++++++++++++++++--
 1 file changed, 38 insertions(+), 2 deletions(-)

diff --git a/arch/arm/dts/zynq-picozed.dts b/arch/arm/dts/zynq-picozed.dts
index dea6a422c3..db5bb8a6fe 100644
--- a/arch/arm/dts/zynq-picozed.dts
+++ b/arch/arm/dts/zynq-picozed.dts
@@ -14,13 +14,33 @@
 	aliases {
 		serial0 = &uart1;
 		spi0 = &qspi;
-		mmc0 = &sdhci1;
+		mmc0 = &sdhci0;
+		mmc1 = &sdhci1;
 	};
 
 	memory at 0 {
 		device_type = "memory";
 		reg = <0 0x40000000>;
 	};
+
+	chosen {
+		bootargs = "earlyprintk";
+		stdout-path = "serial0:115200n8";
+	};
+
+	usb_phy0: phy0 {
+		compatible = "usb-nop-xceiv";
+		#phy-cells = <0>;
+	};
+};
+
+&clkc {
+	ps-clk-frequency = <33333333>;
+};
+
+&qspi {
+	u-boot,dm-pre-reloc;
+	status = "okay";
 };
 
 &uart1 {
@@ -28,7 +48,17 @@
 	status = "okay";
 };
 
-&qspi {
+&gem0 {
+	status = "okay";
+	phy-mode = "rgmii-id";
+	phy-handle = <&ethernet_phy>;
+
+	ethernet_phy: ethernet-phy at 0 {
+		reg = <0>;
+	};
+};
+
+&sdhci0 {
 	u-boot,dm-pre-reloc;
 	status = "okay";
 };
@@ -37,3 +67,9 @@
 	u-boot,dm-pre-reloc;
 	status = "okay";
 };
+
+&usb0 {
+	status = "okay";
+	dr_mode = "host";
+	usb-phy = <&usb_phy0>;
+};
-- 
2.18.0

[U-Boot] [PATCH 1/1] arm: zynq: Fix device tree for Avnet Picozed boards

[Michal Simek]

Hi Andreas,

On 1.8.2018 00:00, Andreas Galauner wrote:
> This change adds the necessary /chosen/stdout-path to the device tree
> for the Avnet Picozed boards. This node is mandatory for the SPL. Without
> it the board resets in a loop without any output. The change also adds
> the USB, QSPI and Ethernet MAC nodes of the corresponding devices present
> on the board.
> 
> Signed-off-by: Andreas Galauner <andreas@galauner.de>
> ---
>  arch/arm/dts/zynq-picozed.dts | 40 +++++++++++++++++++++++++++++++++--
>  1 file changed, 38 insertions(+), 2 deletions(-)
> 
> diff --git a/arch/arm/dts/zynq-picozed.dts b/arch/arm/dts/zynq-picozed.dts
> index dea6a422c3..db5bb8a6fe 100644
> --- a/arch/arm/dts/zynq-picozed.dts
> +++ b/arch/arm/dts/zynq-picozed.dts
> @@ -14,13 +14,33 @@
>  	aliases {
>  		serial0 = &uart1;
>  		spi0 = &qspi;
> -		mmc0 = &sdhci1;
> +		mmc0 = &sdhci0;
> +		mmc1 = &sdhci1;
>  	};
>  
>  	memory at 0 {
>  		device_type = "memory";
>  		reg = <0 0x40000000>;
>  	};
> +
> +	chosen {
> +		bootargs = "earlyprintk";
> +		stdout-path = "serial0:115200n8";
> +	};
> +
> +	usb_phy0: phy0 {
> +		compatible = "usb-nop-xceiv";
> +		#phy-cells = <0>;
> +	};
> +};
> +
> +&clkc {
> +	ps-clk-frequency = <33333333>;
> +};
> +
> +&qspi {
> +	u-boot,dm-pre-reloc;
> +	status = "okay";
>  };
>  
>  &uart1 {
> @@ -28,7 +48,17 @@
>  	status = "okay";
>  };
>  
> -&qspi {
> +&gem0 {
> +	status = "okay";
> +	phy-mode = "rgmii-id";
> +	phy-handle = <&ethernet_phy>;
> +
> +	ethernet_phy: ethernet-phy at 0 {
> +		reg = <0>;
> +	};
> +};
> +
> +&sdhci0 {
>  	u-boot,dm-pre-reloc;
>  	status = "okay";
>  };
> @@ -37,3 +67,9 @@
>  	u-boot,dm-pre-reloc;
>  	status = "okay";
>  };
> +
> +&usb0 {
> +	status = "okay";
> +	dr_mode = "host";
> +	usb-phy = <&usb_phy0>;
> +};
> 

We have been discussing this topic some months ago here
https://lists.denx.de/pipermail/u-boot/2018-May/327267.html

This is a SoM. You need to do it differently because you need to
describe carrier board and then include SoM dtsi.

Thanks,
Michal