[U-Boot] [PATCHv2 1/2] ARM: dts: zynq: Update dts for Z-turn board

[Alexander Graf]

On 16.03.18 00:08, Anton Gerasimov wrote:
> Delete devices implemented in PL, stylistic changes.
> 
> Signed-off-by: Anton Gerasimov <tossel at gmail.com>
> ---
>  arch/arm/dts/zynq-zturn-myir.dts | 62 ++++++++--------------------------------
>  1 file changed, 12 insertions(+), 50 deletions(-)
> 
> diff --git a/arch/arm/dts/zynq-zturn-myir.dts b/arch/arm/dts/zynq-zturn-myir.dts
> index a5ecfcc1d7..a9be2c8374 100644
> --- a/arch/arm/dts/zynq-zturn-myir.dts
> +++ b/arch/arm/dts/zynq-zturn-myir.dts
> @@ -1,3 +1,4 @@
> +// SPDX-License-Identifier: GPL-2.0

Is this intended? I thought you could just add the SPDK identifier right ...

>  /*
>   *  Copyright (C) 2015 Andrea Merello <adnrea.merello at gmail.com>
>   *  Copyright (C) 2017 Alexander Graf <agraf at suse.de>
> @@ -6,31 +7,22 @@
>   *  Copyright (C) 2011 - 2014 Xilinx
>   *  Copyright (C) 2012 National Instruments Corp.
>   *

... here :)

> - * This software is licensed under the terms of the GNU General Public
> - * License version 2, as published by the Free Software Foundation, and
> - * may be copied, distributed, and modified under those terms.
> - *
> - * This program is distributed in the hope that it will be useful,
> - * but WITHOUT ANY WARRANTY; without even the implied warranty of
> - * MERCHANTABILITY or FITNESS FOR A PARTICULAR PURPOSE.  See the
> - * GNU General Public License for more details.
>   */
> +
>  /dts-v1/;
>  /include/ "zynq-7000.dtsi"
>  
>  / {
>  	model = "Zynq Z-Turn MYIR Board";
> -	compatible = "xlnx,zynq-7000";
> +	compatible = "myir,zynq-zturn", "xlnx,zynq-7000";
>  
>  	aliases {
>  		ethernet0 = &gem0;
>  		serial0 = &uart1;
>  		serial1 = &uart0;
> -		spi0 = &qspi;
> -		mmc0 = &sdhci0;

Why? ;)

Sorry if you already explained it, but I don't quite grasp why we need
to remove aliases.


The rest is:

  Reviewed-by: Alexander Graf <agraf at suse.de>


Thanks a lot!

Alex