Re: [PATCH 2/2] ARM: dts: r7s9210-rza2mevb: Add support for RZ/A2M EVB

2018-12-04 Thread Geert Uytterhoeven
Hi Chris, On Tue, Dec 4, 2018 at 5:25 PM Chris Brandt wrote: > Besides, this board will only boot as XIP_KERNEL which is an impossible > configuration in the mainline kernel, so this .dts file is mostly for > checking the .dtsi file. Let's hope we can fix that anytime soon. Gr{oetje,eeting}s,

RE: [PATCH 2/2] ARM: dts: r7s9210-rza2mevb: Add support for RZ/A2M EVB

2018-12-04 Thread Chris Brandt
Hi Geert, Thanks for your review. On Tuesday, December 04, 2018 1, Geert Uytterhoeven wrote: > > + * This file is dual-licensed: you can use it either under the terms > > + * of the GPL or the X11 license, at your option. Note that this dual > > + * licensing only applies to this file, and not

Re: [PATCH 2/2] ARM: dts: r7s9210-rza2mevb: Add support for RZ/A2M EVB

2018-12-04 Thread Geert Uytterhoeven
Hi Chris, On Thu, Nov 29, 2018 at 2:07 PM Chris Brandt wrote: > Add support for Renesas RZ/A2M evaluation board. > > Signed-off-by: Chris Brandt Thanks for your patch! > index ..9570aeb8f1ce > --- /dev/null > +++ b/arch/arm/boot/dts/r7s9210-rza2mevb.dts > @@ -0,0 +1,133 @@ > +/* >

Re: [PATCH 2/2] ARM: dts: r7s9210-rza2mevb: Add support for RZ/A2M EVB

2018-11-30 Thread Geert Uytterhoeven
Hi Chris, On Fri, Nov 30, 2018 at 5:10 PM Chris Brandt wrote: > On Friday, November 30, 2018 1, Geert wrote: > > > So in our RZ/A BSP that I release to customers I would use this dual > > > license. You can see the exact same license in a number of dts files in > > > mainline. > > > > Note that

RE: [PATCH 2/2] ARM: dts: r7s9210-rza2mevb: Add support for RZ/A2M EVB

2018-11-30 Thread Chris Brandt
Hi Geert, On Friday, November 30, 2018 1, Geert wrote: > > So in our RZ/A BSP that I release to customers I would use this dual > > license. You can see the exact same license in a number of dts files in > > mainline. > > Note that your file includes > > #include > #include > > both

Re: [PATCH 2/2] ARM: dts: r7s9210-rza2mevb: Add support for RZ/A2M EVB

2018-11-30 Thread Geert Uytterhoeven
Hi Chris, On Fri, Nov 30, 2018 at 1:20 PM Chris Brandt wrote: > On Friday, November 30, 2018, Simon Horman wrote: > > I am wondering about the motivation for dual-licensing this file. > > It does not seem to be something Renesas has done before with > > upstream DT. > > > > I am also wondering

RE: [PATCH 2/2] ARM: dts: r7s9210-rza2mevb: Add support for RZ/A2M EVB

2018-11-30 Thread Chris Brandt
Hi Simon, On Friday, November 30, 2018, Simon Horman wrote: > > + - RZ/A2M Eval Board (RTK7921053S0BE) > > +compatible = "renesas,rza2mevb", "renesas,r7s9210" > >- RZN1D-DB (RZ/N1D Demo Board for the RZ/N1D 400 pins package) > > compatible = "renesas,rzn1d400-db",

Re: [PATCH 2/2] ARM: dts: r7s9210-rza2mevb: Add support for RZ/A2M EVB

2018-11-30 Thread Simon Horman
On Thu, Nov 29, 2018 at 08:05:59AM -0500, Chris Brandt wrote: > Add support for Renesas RZ/A2M evaluation board. > > Signed-off-by: Chris Brandt > --- > Documentation/devicetree/bindings/arm/shmobile.txt | 2 + > arch/arm/boot/dts/Makefile | 1 + >