Received-SPF: pass (google.com: best guess record for domain of linux-kernel-owner@vger.kernel.org designates 209.132.180.67 as permitted sender) client-ip=209.132.180.67;
MIME-Version: 1.0
References: <20180903140840.23648-1-alexandre.belloni@bootlin.com> <20180910003946.GE21218@dragon>
In-Reply-To: <20180910003946.GE21218@dragon>
From:   Li Yang <leoyang.li@nxp.com>
Date:   Mon, 10 Sep 2018 00:56:40 -0500
Message-ID: <CADRPPNS1N7J_F4ApqSKPrOJ4JuPwgV8srsGsS9DSSURvyhhM9A@mail.gmail.com>
Subject: Re: [PATCH v2 1/5] arm64: dts: fsl: use a generic node name for
 m25p80 flashes
To:     Shawn Guo <shawnguo@kernel.org>
Cc:     alexandre.belloni@bootlin.com, lkml <linux-kernel@vger.kernel.org>,
        "moderated list:ARM/FREESCALE IMX / MXC ARM ARCHITECTURE" 
        <linux-arm-kernel@lists.infradead.org>
Content-Type: text/plain; charset="UTF-8"
Sender: linux-kernel-owner@vger.kernel.org
Precedence: bulk

On Sun, Sep 9, 2018 at 10:11 PM Shawn Guo <shawnguo@kernel.org> wrote:
>
> The series looks good to me.  @Leo, what about you?

Looks fine to me too.

Acked-by: Li Yang <leoyang.li@nxp.com>

>
> Shawn
>
> On Mon, Sep 03, 2018 at 04:08:36PM +0200, Alexandre Belloni wrote:
> > Use a generic node name for the m25p80 flashes on ls1043 and ls1046 boards.
> >
> > Signed-off-by: Alexandre Belloni <alexandre.belloni@bootlin.com>
> > ---
> >  arch/arm64/boot/dts/freescale/fsl-ls1043a-qds.dts | 2 +-
> >  arch/arm64/boot/dts/freescale/fsl-ls1046a-qds.dts | 2 +-
> >  arch/arm64/boot/dts/freescale/fsl-ls1046a-rdb.dts | 4 ++--
> >  3 files changed, 4 insertions(+), 4 deletions(-)
> >
> > diff --git a/arch/arm64/boot/dts/freescale/fsl-ls1043a-qds.dts b/arch/arm64/boot/dts/freescale/fsl-ls1043a-qds.dts
> > index c7b8d2c009cd..af972e91d6bc 100644
> > --- a/arch/arm64/boot/dts/freescale/fsl-ls1043a-qds.dts
> > +++ b/arch/arm64/boot/dts/freescale/fsl-ls1043a-qds.dts
> > @@ -136,7 +136,7 @@
> >       bus-num = <0>;
> >       status = "okay";
> >
> > -     qflash0: s25fl128s@0 {
> > +     qflash0: flash@0 {
> >               compatible = "spansion,m25p80";
> >               #address-cells = <1>;
> >               #size-cells = <1>;
> > diff --git a/arch/arm64/boot/dts/freescale/fsl-ls1046a-qds.dts b/arch/arm64/boot/dts/freescale/fsl-ls1046a-qds.dts
> > index e69306e6b0b1..219068220b84 100644
> > --- a/arch/arm64/boot/dts/freescale/fsl-ls1046a-qds.dts
> > +++ b/arch/arm64/boot/dts/freescale/fsl-ls1046a-qds.dts
> > @@ -165,7 +165,7 @@
> >       bus-num = <0>;
> >       status = "okay";
> >
> > -     qflash0: s25fl128s@0 {
> > +     qflash0: flash@0 {
> >               compatible = "spansion,m25p80";
> >               #address-cells = <1>;
> >               #size-cells = <1>;
> > diff --git a/arch/arm64/boot/dts/freescale/fsl-ls1046a-rdb.dts b/arch/arm64/boot/dts/freescale/fsl-ls1046a-rdb.dts
> > index 440e111651d5..df676b30357b 100644
> > --- a/arch/arm64/boot/dts/freescale/fsl-ls1046a-rdb.dts
> > +++ b/arch/arm64/boot/dts/freescale/fsl-ls1046a-rdb.dts
> > @@ -103,7 +103,7 @@
> >       bus-num = <0>;
> >       status = "okay";
> >
> > -     qflash0: s25fs512s@0 {
> > +     qflash0: flash@0 {
> >               compatible = "spansion,m25p80";
> >               #address-cells = <1>;
> >               #size-cells = <1>;
> > @@ -111,7 +111,7 @@
> >               reg = <0>;
> >       };
> >
> > -     qflash1: s25fs512s@1 {
> > +     qflash1: flash@1 {
> >               compatible = "spansion,m25p80";
> >               #address-cells = <1>;
> >               #size-cells = <1>;
> > --
> > 2.19.0.rc1
> >