From mboxrd@z Thu Jan 1 00:00:00 1970 From: Alexander Graf Date: Fri, 16 Mar 2018 00:29:43 +0100 Subject: [U-Boot] [PATCHv2 1/2] ARM: dts: zynq: Update dts for Z-turn board In-Reply-To: <20180315230808.10359-2-tossel@gmail.com> References: <20180315230808.10359-1-tossel@gmail.com> <20180315230808.10359-2-tossel@gmail.com> Message-ID: <8967789d-487f-41b4-7450-00bab2b3bb2b@suse.de> List-Id: MIME-Version: 1.0 Content-Type: text/plain; charset="us-ascii" Content-Transfer-Encoding: 7bit To: u-boot@lists.denx.de On 16.03.18 00:08, Anton Gerasimov wrote: > Delete devices implemented in PL, stylistic changes. > > Signed-off-by: Anton Gerasimov > --- > 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 > * Copyright (C) 2017 Alexander Graf > @@ -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 Thanks a lot! Alex