[PATCH v3 0/2] arm: pcibios: remove pci_sys_data domain

[Bjorn Helgaas]

[+cc Jason, Thomas, Simon -- no wonder there aren't acks from them :)]

On Mon, Dec 15, 2014 at 04:52:48PM -0700, Bjorn Helgaas wrote:
> On Fri, Nov 21, 2014 at 11:29:24AM +0000, Lorenzo Pieralisi wrote:
> > This patchset is a v3 of a previous posting:
> > 
> > http://www.spinics.net/lists/linux-pci/msg36502.html
> > 
> > v2 => v3
> > 
> > - Rebased on top of this patch dependency
> >   http://www.spinics.net/lists/linux-pci/msg36631.html
> > 
> > v1 => v2 changelog
> > 
> > - Removed stale hw_pci domain member
> > - Reworked pci-mvebu domain handling according to review
> > - Rebased against 3.18-rc3 and updated the logs
> > - Dropped RFC status
> > 
> > Original cover letter:
> > ----
> > This patchset is a first RFC stab at removing the dependency on pci_sys_data
> > domain field on ARM platforms and by replacing it with generic code that
> > stashes the domain value in the pci_bus control structure, introduced in
> > 
> > commit 41e5c0f81d3e676d671d96a0a1fafb27abfbd9
> > ("of/pci: Add pci_get_new_domain_nr() and of_get_pci_domain_nr()")
> > 
> > commit 670ba0c8883b576d0aec28bd7a838358a4be1
> > ("PCI: Add generic domain handling")
> > 
> > All the drivers converted (apart from PCIe designware, tested on iMX6SL)
> > were only compile tested for lack of HW, so along some comments, testing
> > and verifying that patchset does not break any existing platform are
> > very appreciated.
> > 
> > Code in drivers/pci/pci-mvebu.c has been changed to add a domain
> > number to PCI resources by using the nr value coming from the setup
> > pcibios32 callback, which may not be correct and should be considered
> > a temporary solution waiting for review comments.
> > 
> > The patchset removes entirely the pci_sys_data.domain field, since its
> > usage is removed at the same time in the respective host controllers
> > and ARM is made to select the configuration option
> > CONFIG_PCI_DOMAINS_GENERIC by default, which compiles a domain_nr in the
> > pci_bus structure so that the pci_domain_nr() look-up can rely on it.
> > 
> > 
> > Lorenzo Pieralisi (2):
> >   arm: cns3xxx: pci: remove artificial dependency on pci_sys_data domain
> >   arm: pcibios: move to generic PCI domains
> > 
> >  arch/arm/Kconfig                   |  3 +++
> >  arch/arm/include/asm/mach/pci.h    |  6 ------
> >  arch/arm/include/asm/pci.h         |  7 -------
> >  arch/arm/kernel/bios32.c           |  3 ---
> >  arch/arm/mach-cns3xxx/pcie.c       | 40 ++++++++++++++++----------------------
> >  drivers/pci/host/pci-mvebu.c       | 15 ++------------
> >  drivers/pci/host/pcie-designware.c |  3 ---
> >  drivers/pci/host/pcie-rcar.c       |  3 ---
> >  8 files changed, 22 insertions(+), 58 deletions(-)
> 
> I provisionally applied these to my next-pci/domain branch.  This will be
> rebased after v3.19-rc1.
> 
> I'd like to see acks from Thomas and/or Jason for MVEBU, and Simon for
> R-Car.