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: <20200725014529.1143208-1-jiaxun.yang@flygoat.com>
 <20200725014529.1143208-2-jiaxun.yang@flygoat.com> <CAL_JsqKePrUW3-HoSnQawqhgg23XJ7MxzawD7TKt-__q3jM55g@mail.gmail.com>
 <20200728083308.GD9062@alpha.franken.de>
In-Reply-To: <20200728083308.GD9062@alpha.franken.de>
From:   Rob Herring <robh+dt@kernel.org>
Date:   Tue, 28 Jul 2020 07:46:04 -0600
Message-ID: <CAL_Jsq+oswHvuF0BaM27CP0j9aHg1whAOv7bZneFjdkDX=MnSg@mail.gmail.com>
Subject: Re: [PATCH v3 1/5] of_address: Add bus type match for pci ranges parser
To:     Thomas Bogendoerfer <tsbogend@alpha.franken.de>
Cc:     Jiaxun Yang <jiaxun.yang@flygoat.com>,
        "open list:MIPS" <linux-mips@vger.kernel.org>,
        Huacai Chen <chenhc@lemote.com>,
        Frank Rowand <frowand.list@gmail.com>,
        Paul Burton <paulburton@kernel.org>,
        Nathan Chancellor <natechancellor@gmail.com>,
        Arnd Bergmann <arnd@arndb.de>,
        Nick Desaulniers <ndesaulniers@google.com>,
        devicetree@vger.kernel.org,
        "linux-kernel@vger.kernel.org" <linux-kernel@vger.kernel.org>
Content-Type: text/plain; charset="UTF-8"
Sender: linux-kernel-owner@vger.kernel.org
Precedence: bulk

On Tue, Jul 28, 2020 at 2:39 AM Thomas Bogendoerfer
<tsbogend@alpha.franken.de> wrote:
>
> On Mon, Jul 27, 2020 at 11:50:14AM -0600, Rob Herring wrote:
> > On Fri, Jul 24, 2020 at 7:45 PM Jiaxun Yang <jiaxun.yang@flygoat.com> wrote:
> > >
> > > So the parser can be used to parse range property of ISA bus.
> > >
> > > As they're all using PCI-like method of range property, there is no need
> > > start a new parser.
> > >
> > > Signed-off-by: Jiaxun Yang <jiaxun.yang@flygoat.com>
> > >
> > > --
> > > v2: Drop useless check, fix some na for bus_addr
> > >         add define of of_range_parser_init according to
> > >         Rob's suggestion.
> > > v3: Abstract out has_flags. simplify define.
> > > ---
> > >  drivers/of/address.c       | 29 +++++++++++++++++------------
> > >  include/linux/of_address.h |  4 ++++
> > >  2 files changed, 21 insertions(+), 12 deletions(-)
> >
> > Reviewed-by: Rob Herring <robh@kernel.org>
>
> Rob, are you ok with merging this via the mips-next tree ?

That's my expectation hence the Reviewed-by.

Rob