Subject: Re: [PATCH v3 19/25] PCI: of: Add inbound resource parsing to helpers

From: Bjorn Helgaas <[email-masked]>
Date: Tue, 29 Oct 2019 17:12:24 -0500

On Tue, Oct 29, 2019 at 11:07:51AM +0000, Lorenzo Pieralisi wrote:
> On Mon, Oct 28, 2019 at 11:32:50AM -0500, Rob Herring wrote:
> > Extend devm_of_pci_get_host_bridge_resources() and
> > pci_parse_request_of_pci_ranges() helpers to also parse the inbound
> > addresses from DT 'dma-ranges' and populate a resource list with the
> > translated addresses. This will help ensure 'dma-ranges' is always
> > parsed in a consistent way.
> > 
> > Cc: Jingoo Han <[email-masked]>
> > Cc: Gustavo Pimentel <[email-masked]>
> > Cc: Lorenzo Pieralisi <[email-masked]>
> > Cc: Bjorn Helgaas <[email-masked]>
> > Cc: Thomas Petazzoni <[email-masked]>
> > Cc: Will Deacon <[email-masked]>
> > Cc: Linus Walleij <[email-masked]>
> > Cc: Toan Le <[email-masked]>
> > Cc: Ley Foon Tan <[email-masked]>
> > Cc: Tom Joseph <[email-masked]>
> > Cc: Ray Jui <[email-masked]>
> > Cc: Scott Branden <[email-masked]>
> > Cc: [email-masked]
> > Cc: Ryder Lee <[email-masked]>
> > Cc: Karthikeyan Mitran <[email-masked]>
> > Cc: Hou Zhiqiang <[email-masked]>
> > Cc: Simon Horman <[email-masked]>
> > Cc: Shawn Lin <[email-masked]>
> > Cc: Heiko Stuebner <[email-masked]>
> > Cc: Michal Simek <[email-masked]>
> > Cc: [email-masked]
> > Cc: [email-masked]
> > Cc: [email-masked]
> > Cc: [email-masked]
> > Tested-by: Thomas Petazzoni <[email-masked]> # for AArdvark
> > Reviewed-by: Andrew Murray <[email-masked]>
> > Acked-by: Gustavo Pimentel <[email-masked]>
> > Signed-off-by: Rob Herring <[email-masked]>
> > ---
> > v2:
> >  - Fix crash in INIT_LIST_HEAD when ib_resources is NULL
> > ---
> >  .../pci/controller/dwc/pcie-designware-host.c |  3 +-
> >  drivers/pci/controller/pci-aardvark.c         |  2 +-
> >  drivers/pci/controller/pci-ftpci100.c         |  3 +-
> >  drivers/pci/controller/pci-host-common.c      |  2 +-
> >  drivers/pci/controller/pci-v3-semi.c          |  3 +-
> >  drivers/pci/controller/pci-versatile.c        |  3 +-
> >  drivers/pci/controller/pci-xgene.c            |  3 +-
> >  drivers/pci/controller/pcie-altera.c          |  2 +-
> >  drivers/pci/controller/pcie-cadence-host.c    |  2 +-
> >  drivers/pci/controller/pcie-iproc-platform.c  |  3 +-
> >  drivers/pci/controller/pcie-mediatek.c        |  2 +-
> >  drivers/pci/controller/pcie-mobiveil.c        |  3 +-
> >  drivers/pci/controller/pcie-rcar.c            |  3 +-
> >  drivers/pci/controller/pcie-rockchip-host.c   |  3 +-
> >  drivers/pci/controller/pcie-xilinx-nwl.c      |  3 +-
> >  drivers/pci/controller/pcie-xilinx.c          |  3 +-
> >  drivers/pci/of.c                              | 45 ++++++++++++++++++-
> >  drivers/pci/pci.h                             |  8 +++-
> >  include/linux/pci.h                           |  9 ++--
> >  19 files changed, 82 insertions(+), 23 deletions(-)
> 
> Hi Bjorn,
> 
> please let me know if you are OK with this patch, the series is
> ready to go upstream IMO.

Yep, looks good to me.  If you want it:

Acked-by: Bjorn Helgaas <[email-masked]>

_______________________________________________
linux-arm-kernel mailing list
[email-masked]
http://lists.infradead.org/mailman/listinfo/linux-arm-kernel