Received-SPF: pass (google.com: domain of linux-kernel-owner@vger.kernel.org designates 23.128.96.18 as permitted sender) client-ip=23.128.96.18;
MIME-Version: 1.0
References: <20200827181842.1000451-1-f.fainelli@gmail.com> <5afe8762-c0a3-ff10-5946-0eb2e7ddc0e5@broadcom.com>
In-Reply-To: <5afe8762-c0a3-ff10-5946-0eb2e7ddc0e5@broadcom.com>
From:   Kamal Dasu <kdasu.kdev@gmail.com>
Date:   Fri, 28 Aug 2020 11:27:36 -0400
Message-ID: <CAC=U0a3JwZr5tpNZ1ba63ORw5gfiBPAhOqYo4UZTOE-eJm8mjw@mail.gmail.com>
Subject: Re: [PATCH 0/5] qspi binding and DTS fixes
To:     Scott Branden <scott.branden@broadcom.com>
Cc:     Florian Fainelli <f.fainelli@gmail.com>,
        linux-arm-kernel@lists.infradead.org,
        "maintainer:BROADCOM SPI DRIVER" 
        <bcm-kernel-feedback-list@broadcom.com>,
        Mark Brown <broonie@kernel.org>,
        Rob Herring <robh+dt@kernel.org>, Ray Jui <rjui@broadcom.com>,
        Scott Branden <sbranden@broadcom.com>,
        Hauke Mehrtens <hauke@hauke-m.de>,
        =?UTF-8?B?UmFmYcWCIE1pxYJlY2tp?= <zajec5@gmail.com>,
        Yendapally Reddy Dhananjaya Reddy 
        <yendapally.reddy@broadcom.com>,
        "open list:SPI SUBSYSTEM" <linux-spi@vger.kernel.org>,
        "open list:OPEN FIRMWARE AND FLATTENED DEVICE TREE BINDINGS" 
        <devicetree@vger.kernel.org>,
        open list <linux-kernel@vger.kernel.org>
Content-Type: text/plain; charset="UTF-8"
Sender: linux-kernel-owner@vger.kernel.org
Precedence: bulk

Looks good to me.

Signed-off-by: Kamal Dasu <kdasu.kdev@gmail.com>

On Thu, Aug 27, 2020 at 3:30 PM Scott Branden
<scott.branden@broadcom.com> wrote:
>
> Patch series looks good.
>
> Acked-by: Scott Branden <scott.branden@broadcom.com>
>
> On 2020-08-27 11:18 a.m., Florian Fainelli wrote:
> > Hi all,
> >
> > This patch series fixes incorrectly defined compatible strings for the
> > Broadcom QSPI controller which resulted in the strings not being
> > ordered from most to least compatible.
> >
> > We will need to apply some changes to the spi-bcm-qspi.c driver in
> > the future to assume no revision register exist, and these patches
> > are a preliminary step towards that goal.
> >
> > Florian Fainelli (5):
> >   dt-bindings: spi: Fix spi-bcm-qspi compatible ordering
> >   ARM: dts: bcm: HR2: Fixed QSPI compatible string
> >   ARM: dts: NSP: Fixed QSPI compatible string
> >   ARM: dts: BCM5301X: Fixed QSPI compatible string
> >   arm64: dts: ns2: Fixed QSPI compatible string
> >
> >  .../bindings/spi/brcm,spi-bcm-qspi.txt           | 16 ++++++++--------
> >  arch/arm/boot/dts/bcm-hr2.dtsi                   |  2 +-
> >  arch/arm/boot/dts/bcm-nsp.dtsi                   |  2 +-
> >  arch/arm/boot/dts/bcm5301x.dtsi                  |  2 +-
> >  arch/arm64/boot/dts/broadcom/northstar2/ns2.dtsi |  2 +-
> >  5 files changed, 12 insertions(+), 12 deletions(-)
> >
>