linux-kernel - Re: [PATCH] arm: dt: zynq: Add support for the zc706 platform

lists.openwall.net		lists / announce owl-users owl-dev john-users john-dev passwdqc-users yescrypt popa3d-users / oss-security kernel-hardening musl sabotage tlsify passwords / crypt-dev xvendor / Bugtraq Full-Disclosure linux-kernel linux-netdev linux-ext4 linux-hardening linux-cve-announce PHC
Open Source and information security mailing list archives

Hash Suite: Windows password security audit tool. GUI, reports in PDF.

[<prev] [next>] [<thread-prev] [thread-next>] [day] [month] [year] [list]

Message-ID: <785766f9-0d23-4a78-8638-fac05920049d@CO1EHSMHS013.ehs.local>
Date:	Tue, 11 Jun 2013 14:28:41 -0700
From:	Sören Brinkmann <soren.brinkmann@...inx.com>
To:	Dinh Nguyen <dinguyen@...era.com>
CC:	Michal Simek <michal.simek@...inx.com>,
	Russell King <linux@....linux.org.uk>,
	Olof Johansson <olof@...om.net>,
	Shawn Guo <shawn.guo@...aro.org>,
	<linux-kernel@...r.kernel.org>,
	<linux-arm-kernel@...ts.infradead.org>
Subject: Re: [PATCH] arm: dt: zynq: Add support for the zc706 platform

On Tue, Jun 11, 2013 at 04:24:48PM -0500, Dinh Nguyen wrote:
> Hi Soren,
> 
> On Mon, 2013-06-10 at 10:10 -0700, Soren Brinkmann wrote:
> > Add a DT fragment for the zc706 Zynq platform. Also, adding a
> > corresponding target to the Makefile and adding an appropriate
> > compatibility string in the BSP.
> > 
> > Signed-off-by: Soren Brinkmann <soren.brinkmann@...inx.com>
> > ---
> >  arch/arm/boot/dts/Makefile       |  3 ++-
> >  arch/arm/boot/dts/zynq-zc706.dts | 31 +++++++++++++++++++++++++++++++
> >  arch/arm/mach-zynq/common.c      |  1 +
> >  3 files changed, 34 insertions(+), 1 deletion(-)
> >  create mode 100644 arch/arm/boot/dts/zynq-zc706.dts
> > 
> > diff --git a/arch/arm/boot/dts/Makefile b/arch/arm/boot/dts/Makefile
> > index b9f7121..2aecf7e 100644
> > --- a/arch/arm/boot/dts/Makefile
> > +++ b/arch/arm/boot/dts/Makefile
> > @@ -206,7 +206,8 @@ dtb-$(CONFIG_ARCH_VT8500) += vt8500-bv07.dtb \
> >  	wm8505-ref.dtb \
> >  	wm8650-mid.dtb \
> >  	wm8850-w70v2.dtb
> > -dtb-$(CONFIG_ARCH_ZYNQ) += zynq-zc702.dtb
> > +dtb-$(CONFIG_ARCH_ZYNQ) += zynq-zc702.dtb \
> > +	zynq-zc706.dtb
> >  
> >  targets += dtbs
> >  targets += $(dtb-y)
> > diff --git a/arch/arm/boot/dts/zynq-zc706.dts b/arch/arm/boot/dts/zynq-zc706.dts
> > new file mode 100644
> > index 0000000..a8a1d7d
> > --- /dev/null
> > +++ b/arch/arm/boot/dts/zynq-zc706.dts
> > @@ -0,0 +1,31 @@
> > +/*
> > + *  Copyright (C) 2011 Xilinx
> > + *  Copyright (C) 2012 National Instruments Corp.
> > + *  Copyright (C) 2013 Xilinx
> > + *
> > + * 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 ZC706 Development Board";
> > +	compatible = "xlnx,zynq-zc706", "xlnx,zynq-7000";
> > +
> > +	memory {
> > +		device_type = "memory";
> > +		reg = <0x0 0x40000000>;
> > +	};
> > +
> > +	chosen {
> > +		bootargs = "console=ttyPS1,115200 earlyprintk";
> > +	};
> > +
> > +};
> > diff --git a/arch/arm/mach-zynq/common.c b/arch/arm/mach-zynq/common.c
> > index 5bfe703..221694b 100644
> > --- a/arch/arm/mach-zynq/common.c
> > +++ b/arch/arm/mach-zynq/common.c
> > @@ -99,6 +99,7 @@ static void zynq_system_reset(char mode, const char *cmd)
> >  
> >  static const char * const zynq_dt_match[] = {
> >  	"xlnx,zynq-zc702",
> > +	"xlnx,zynq-zc706",
> 
> You really don't need to add zc706 here if it is already compatable with
> zynq-7000.
I actually thought about this and decided to add it, since there already
was an entry for the zc702. What is the best methord for this cases?
Rather remove the specific ones and migrate all to the zynq-7000
compatibility string?

	Sören


--
To unsubscribe from this list: send the line "unsubscribe linux-kernel" in
the body of a message to majordomo@...r.kernel.org
More majordomo info at  http://vger.kernel.org/majordomo-info.html
Please read the FAQ at  http://www.tux.org/lkml/