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

[Bjorn Helgaas <bhelgaas@google.com>]

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.