From mboxrd@z Thu Jan 1 00:00:00 1970 From: Geert Uytterhoeven Subject: Re: [PATCH 3/4] ARM: dts: r8a77470: Add QSPI support Date: Thu, 8 Nov 2018 11:37:12 +0100 Message-ID: References: <1541075705-1333-1-git-send-email-fabrizio.castro@bp.renesas.com> <1541075705-1333-4-git-send-email-fabrizio.castro@bp.renesas.com> <20181102114946.5tqk2lyisaey475w@verge.net.au> Mime-Version: 1.0 Content-Type: text/plain; charset="us-ascii" Content-Transfer-Encoding: 7bit Cc: Mark Rutland , "open list:OPEN FIRMWARE AND FLATTENED DEVICE TREE BINDINGS" , Chris Paterson , Geert Uytterhoeven , Marek Vasut , Richard Weinberger , Mark Brown , Magnus Damm , Rob Herring , linux-spi , Linux-Renesas , Boris Brezillon , Simon Horman , MTD Maling List , Brian Norris , David Woodhouse , Biju Das To: Fabrizio Castro Return-path: In-Reply-To: List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , Sender: "linux-mtd" Errors-To: linux-mtd-bounces+gldm-linux-mtd-36=gmane.org@lists.infradead.org List-Id: linux-spi.vger.kernel.org Hi Fabrizio, On Thu, Nov 8, 2018 at 11:31 AM Fabrizio Castro wrote: > > On Thu, Nov 8, 2018 at 11:22 AM Fabrizio Castro > > wrote: > > > > From: Simon Horman > > > > Sent: 02 November 2018 11:50 > > > > Subject: Re: [PATCH 3/4] ARM: dts: r8a77470: Add QSPI support > > > > > > > > On Thu, Nov 01, 2018 at 12:35:04PM +0000, Fabrizio Castro wrote: > > > > > Add QSPI[01] support to the RZ/G1C SoC specific device tree. > > > > > > > > > > Signed-off-by: Fabrizio Castro > > > > > --- > > > > > arch/arm/boot/dts/r8a77470.dtsi | 34 ++++++++++++++++++++++++++++++++++ > > > > > 1 file changed, 34 insertions(+) > > > > > > > > > > diff --git a/arch/arm/boot/dts/r8a77470.dtsi b/arch/arm/boot/dts/r8a77470.dtsi > > > > > index e6407e5..04a8877 100644 > > > > > --- a/arch/arm/boot/dts/r8a77470.dtsi > > > > > +++ b/arch/arm/boot/dts/r8a77470.dtsi > > > > > @@ -20,6 +20,8 @@ > > > > > i2c2 = &i2c2; > > > > > i2c3 = &i2c3; > > > > > i2c4 = &i2c4; > > > > > +spi0 = &qspi0; > > > > > +spi1 = &qspi1; > > > > > }; > > > > > > > > Geert can comment but I believe we are moving away from using aliases > > > > in this way and that it would be best if the above hunk was dropped > > > > from this patch. > > > > > > > > https://patchwork.kernel.org/patch/10644159/ > > > > > > Geert, what do you want me to do here? > > > > Please drop the aliases. Thanks! > > Will do, will send a new version without the additional aliases. Thanks! > My understanding is that your personal preference is to only leave debug console and ethernet, Labeled serial ports and primary Ethernet, so U-Boot knows which MAC address property to update. > shall I also get rid of what was already upstreamed for RZ/G related boards that is not in line > with the new policy? I believe these are all serial ports, using aliases that match the documented and/or labeled port names? So they don't have to be removed. Gr{oetje,eeting}s, Geert -- Geert Uytterhoeven -- There's lots of Linux beyond ia32 -- geert@linux-m68k.org In personal conversations with technical people, I call myself a hacker. But when I'm talking to journalists I just say "programmer" or something like that. -- Linus Torvalds ______________________________________________________ Linux MTD discussion mailing list http://lists.infradead.org/mailman/listinfo/linux-mtd/