[PATCHv4 00/11] MSI support for Marvell EBU PCIe driver
Thomas Petazzoni
thomas.petazzoni at free-electrons.com
Mon Jul 1 09:42:05 EDT 2013
Hello,
This set of patches introduces Message Signaled Interrupt support in
the Marvell EBU PCIe driver. It has been successfully tested on the
Armada XP GP platform with an Intel e1000e PCIe network card that
supports MSI.
The patches do the following:
* Patch 1 adds an IRQ domain function that allows to allocate
dynamically a free hwirq number from an IRQ domain, and create a
virq mapping to it. It was suggested by Grant Likely in order to
remove the hwirq allocation code from the IRQ driver itself.
This patch needs the Ack of Grant Likely.
* Patch 2 reworks how the architecture-specific MSI functions can be
overriden by architecture-specific code, by using weak
functions. It was suggested by Bjorn Helgaas.
This patch needs the Ack from Bjorn Helgaas, and the relevant
architecture maintainers.
* Patch 3 removes the ARCH_SUPPORTS_MSI hidden kconfig boolean which
is no longer needed now that we have weak functions for all MSI
architecture-specific hooks.
This patch needs the Ack from Bjorn Helgaas, and the relevant
architecture maintainers.
* Patch 4 adds a minimal msi_chip infrastructure, that allows a
pci_bus to be connected to a msi_chip, and that provides default
implementations of the architecture-specific MSI functions to use
msi_chip.
This patch needs the Ack from Bjorn Helgaas.
* Patch 5 adds a small registry of msi_chip <-> of_node in the OF
code. It was originally located in drivers/pci/msi.c, but Bjorn
Helgaas suggested that it probably belongs to drivers/of/.
This patch needs the Ack from Grant Likely or Rob Herring, and
probably Bjorn Helgaas as well.
* Patch 6 makes some not very interesting preparation in the Armada
370/XP IRQ controller driver.
This patch needs the Ack from Thomas Gleixner.
* Patch 7 implements the MSI support in the Armada 370/XP IRQ
controller driver. It registers an msi_chip using the
msi_chip_add() function added in PATCH 3.
This patch needs the Ack from Thomas Gleixner.
* Patch 8 extends the ARM PCI core to expose ->add_bus() and
->remove_bus() hooks to PCI drivers. This was suggested by Bjorn
Helgaas to allow the PCI driver to connected the PCI busses with
the corresponding MSI chip.
This patch needs the Ack from Russell King.
* Patch 9 adjust the Armada 370/XP Device Tree to indicate that the
MPIC is not only an interrupt-controller, but also an
msi-controller.
This patch needs the Ack from Jason Cooper, Gregory Clement
and/or Andrew Lunn, the Marvell Maintainers.
* Patch 10 adds MSI support in the Marvell PCIe host controller
driver. The work to do here is minimal: get a reference to the
msi-parent controller thanks to msi_chip_find_by_of_node(), and
link it to the pci_bus structure before the bus gets enumerated.
This patch needs the Ack from Bjorn Helgaas.
* Patch 11 adjusts the Armada 370/XP Device Tree to add the msi-parent
properties in the PCIe controller nodes.
This patch needs the Ack from Jason Cooper, Gregory Clement
and/or Andrew Lunn, the Marvell Maintainers.
This version 4 follows:
* PATCH version 3 sent on June, 19th 2013
* PATCH version 2 sent on June, 6th 2013
* RFC version 1 sent on March, 26th 2013
Changes since v3:
* Keep only a default_teardown_msi_irqs() function needed for the Xen
PCI x86 code, and remove all other default_*() function and put the
default behavior directly in the weak functions. Suggested by
Thierry Redding and Bjorn Helgaas.
* Misc small improvements the MSI chip registry code: compiled only
under CONFIG_PCI_MSI, functions exported to modules, addition of an
of_msi_chip_remove() function, renaming of
of_msi_chip_find_by_node() to of_find_msi_chip_by_node(), move the
test of the "msi-controller" property to the of_msi_chip_add()
function, renamed the list_head field from link to list, added
dummy functions in the header files when !CONFIG_PCI_MSI &&
!CONFIG_OF. All suggested by Thierry Redding.
* Add a patch that entirely removes the ARCH_SUPPORTS_MSI, as we now
have weak functions, so even if an arch doesn't actually
use/support MSI, the code will build properly. Suggested by Thierry
Redding.
* Added Device Tree binding documentation updates for the IRQ
controller and the PCIe controller.
Changes since v2:
* Add an IRQ domain function that allows to allocate dynamically a
free hwirq number from an IRQ domain, and create a virq mapping to
it. It was suggested by Grant Likely in order to remove the hwirq
allocation code from the IRQ driver itself. (PATCH 1)
* Separate the use of weak functions from the introduction of the
msi_chip infrastructure, and use weak functions for all
architecture-specific MSI hooks. Suggested by Bjorn Helgaas (PATCH
2).
* Move the msi_chip registry to drivers/of. Suggested by Bjorn
Helgaas (PATCH 4).
* Use pcibios_add_bus() and pcibios_remove_bus() to connect the PCI
busses to their msi_chip, as suggested by Bjorn Helgaas. Requires
some ARM PCI core changes (PATCH 7), and then changes to the PCI
driver itself (PATCH 9).
This set of patches currently applies on top of the current "for-next"
branch of Jason Cooper's repository + the irqdomain/test branch of
Grant Likely's tree. I will rebase them on top of 3.11-rc1 when it
lands, but I'd like to start collecting Acked-by from maintainers in
the mean time.
Thanks,
Thomas
Thierry Reding (1):
PCI: Introduce new MSI chip infrastructure
Thomas Petazzoni (10):
irqdomain: add irq_alloc_mapping() function
pci: use weak functions for MSI arch-specific functions
pci: remove ARCH_SUPPORTS_MSI kconfig option
of: pci: add registry of MSI chips
irqchip: armada-370-xp: properly request resources
irqchip: armada-370-xp: implement MSI support
arm: pci: add ->add_bus() and ->remove_bus() hooks to hw_pci
arm: mvebu: the MPIC now provides MSI controller features
pci: mvebu: add support for MSI
arm: mvebu: link PCIe controllers to the MSI controller
.../devicetree/bindings/arm/armada-370-xp-mpic.txt | 3 +
.../devicetree/bindings/pci/mvebu-pci.txt | 3 +
arch/arm/Kconfig | 1 -
arch/arm/boot/dts/armada-370-xp.dtsi | 1 +
arch/arm/boot/dts/armada-370.dtsi | 1 +
arch/arm/boot/dts/armada-xp-mv78230.dtsi | 1 +
arch/arm/boot/dts/armada-xp-mv78260.dtsi | 1 +
arch/arm/boot/dts/armada-xp-mv78460.dtsi | 1 +
arch/arm/include/asm/mach/pci.h | 4 +
arch/arm/kernel/bios32.c | 16 ++
arch/ia64/Kconfig | 1 -
arch/mips/Kconfig | 2 -
arch/mips/include/asm/pci.h | 5 -
arch/powerpc/Kconfig | 1 -
arch/powerpc/include/asm/pci.h | 5 -
arch/s390/Kconfig | 1 -
arch/s390/include/asm/pci.h | 4 -
arch/sparc/Kconfig | 1 -
arch/tile/Kconfig | 1 -
arch/x86/Kconfig | 1 -
arch/x86/include/asm/pci.h | 28 ----
arch/x86/kernel/x86_init.c | 21 +++
drivers/irqchip/irq-armada-370-xp.c | 161 ++++++++++++++++++++-
drivers/of/of_pci.c | 40 +++++
drivers/pci/Kconfig | 4 -
drivers/pci/host/pci-mvebu.c | 26 ++++
drivers/pci/msi.c | 67 +++++----
drivers/pci/probe.c | 1 +
include/linux/irqdomain.h | 2 +
include/linux/msi.h | 20 ++-
include/linux/of_pci.h | 12 ++
include/linux/pci.h | 1 +
kernel/irq/irqdomain.c | 32 ++++
33 files changed, 383 insertions(+), 86 deletions(-)
--
1.8.1.2
More information about the linux-arm-kernel
mailing list