From mboxrd@z Thu Jan  1 00:00:00 1970
From: Michal Simek <michal.simek@xilinx.com>
Date: Thu, 2 Aug 2018 14:07:09 +0200
Subject: [U-Boot] [PATCH 1/1] arm: zynq: Fix device tree for Avnet
 Picozed boards
In-Reply-To: <20180731220038.15675-1-andreas@galauner.de>
References: <20180731220038.15675-1-andreas@galauner.de>
Message-ID: <079c4391-83f1-9b4b-4642-c534e89e44b6@xilinx.com>
List-Id: <u-boot.lists.denx.de>
MIME-Version: 1.0
Content-Type: text/plain; charset="us-ascii"
Content-Transfer-Encoding: 7bit
To: u-boot@lists.denx.de

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