[PATCH v12 00/12] Support for creating generic PCI host bridges from DT

From: Liviu Dudau
Date: Tue Sep 23 2014 - 16:05:04 EST


This is my version 12 of the attempt at adding support for generic PCI host
bridge controllers that make use of device tree information to
configure themselves. It contains cleanups to address comments posted for v11.

Bjorn intends to pull this series into his tree and send it to linux-next if
no major issues arise.


Changes from v11:
- fixed race condition in pci_register_io_range() and bug where the function
would return with the io_range_lock held.
- split patch 4/10 that fixes the convertion of IO ranges into resources into
two patches. The new 4/12 just moves the function into drivers/of/address.c
and 5/12 makes the functional changes.
- Added Linus Walleij ACK on 5/12 for the arch/arm/mach-integrator/pci_v3.c
part
- Removed the old PCI domain number parser and allocator and split the code
into two independent functions: 'int pci_get_new_domain_nr(void)' will
always return a new domain number when called while 'of_get_pci_domain_nr()'
will parse the DT and try to find a property of the given node called
'linux,pci-domain'. The property value, if found, will be truncated to
16-bit value and returned, or a negative value if an error occurs.
- Made the io_base pointer parameter that gets passed to
of_pci_get_host_bridge_resources() optional (thanks Robert Richter) and
be explicit when detecting multiple IO ranges about the device used.
- Fixed comments on commits to explain who uses pci_remap_iospace() as well
as making clear that the value used inside the function is a virtual address
and not a physical one. Also be explicit in the warning generated when
architectures without memory mapped IO support make use of this function
- Merged the arm64 patch into this series as Bjorn is going to pull it
into his tree.

Changes from v10:
- Addressed Lorenzo's comments wrt io_base not being initialised inside the
function that is supposed to modify it and also updated the comments to
explain the values returned by the function
- Updated pci_domain_nr helper function to split the allocation of the domain
number from the parsing of the device tree to find maximum defined domain
value
- Fixed up the commit author for commit 6/10

Changes from v9:
- Moved the DT parsing and assigning of IRQ patch from this series into arm64
specific patch. This keeps existing pcibios_add_device() unchanged and adds
an arch-specific version that can later be expanded to cater for dma_ops.
- Incorporated the fix for users of of_pci_range_to_resources() into the patch
that changes the behaviour for easier bisection.
- Added fixes for tegra and rcar host drivers in their usage of
of_pci_range_to_resources()
- Broke up of_create_pci_host_bridge() to remove the callback function. The
function left has been renamed into of_pci_get_host_bridge_resources(). The
added benefit of that is that the architectural hook for fixing up host bridge
resources now dissappears.
- Reshuffled the way pgprot_device gets introduced. It is now part of the patch
that adds pci_remap_iospace() function. The arm64 specific override is moved
into the arm64 patchset.
- Added a patch to pci_scan_root_bus() to assign unassigned resources if PCI
flags are not PCI_PROBE_ONLY

v11 thread here: https://lkml.org/lkml/2014/9/17/732
v10 thread here: https://lkml.org/lkml/2014/9/8/333
v9 thread here, with links to previous threads: https://lkml.org/lkml/2014/8/12/361

Best regards,
Liviu

Catalin Marinas (1):
PCI: Introduce generic domain handling for PCI busses.

Liviu Dudau (11):
Fix ioport_map() for !CONFIG_GENERIC_IOMAP cases.
PCI: Introduce helper functions to deal with PCI I/O ranges.
ARM: Define PCI_IOBASE as the base of virtual PCI IO space.
PCI: OF: Move of_pci_range_to_resources() into address.c
PCI: OF: Fix the conversion of IO ranges into IO resources.
PCI: Create pci_host_bridge before its associated bus in pci_create_root_bus.
PCI: OF: Introduce helper function for retrieving PCI domain numbers
OF: PCI: Add support for parsing PCI host bridge resources from DT
PCI: Assign unassigned bus resources in pci_scan_root_bus()
PCI: Introduce pci_remap_iospace() for remapping PCI I/O bus resources into CPU space
arm64: Add architectural support for PCIe

arch/arm/include/asm/io.h | 1 +
arch/arm/mach-integrator/pci_v3.c | 23 +++---
arch/arm64/Kconfig | 22 +++++-
arch/arm64/include/asm/Kbuild | 1 +
arch/arm64/include/asm/io.h | 3 +-
arch/arm64/include/asm/pci.h | 37 +++++++++
arch/arm64/include/asm/pgtable.h | 2 +
arch/arm64/kernel/Makefile | 1 +
arch/arm64/kernel/pci.c | 70 +++++++++++++++++
drivers/of/address.c | 155 ++++++++++++++++++++++++++++++++++++++
drivers/of/of_pci.c | 140 ++++++++++++++++++++++++++++++++++
drivers/pci/host/pci-tegra.c | 10 ++-
drivers/pci/host/pcie-rcar.c | 21 ++++--
drivers/pci/pci.c | 43 +++++++++++
drivers/pci/probe.c | 46 ++++++-----
include/asm-generic/io.h | 2 +-
include/asm-generic/pgtable.h | 4 +
include/linux/of_address.h | 22 +++---
include/linux/of_pci.h | 18 +++++
include/linux/pci.h | 27 +++++++
20 files changed, 596 insertions(+), 52 deletions(-)
create mode 100644 arch/arm64/include/asm/pci.h
create mode 100644 arch/arm64/kernel/pci.c

--
2.1.0

--
To unsubscribe from this list: send the line "unsubscribe linux-kernel" in
the body of a message to majordomo@xxxxxxxxxxxxxxx
More majordomo info at http://vger.kernel.org/majordomo-info.html
Please read the FAQ at http://www.tux.org/lkml/