Skip to content

pci: remove msi_controller registration API

Marc Zyngier linux-arm-kernel linux-pci 2 versions – New

Get series with b4
b4 am [email protected]

Versions newest first · linked by author and title

  1. v2 Latest1 of 3 patches 0 repliesNew
  2. v12 of 3 patches 0 repliesSuperseded

1 of 3 patches is in this archive. The cover letter announces 3.

Patches in v2 No applies recorded yet

#Subject and filesSizeRepliesApplied
1/3pci: pci-aardvark: move to MSI handling using generic MSI support+0 -00 replies

Reviews trailers per patch

The patch has no Reviewed-by or Acked-by of their own; the series has one for all of them.

PatchMarc Zyngier
SeriesA-b
1/3

Trailers given in replies are filled; those carried in the patches from earlier versions are outlined with dashes.

Read the v2 thread

Cover letter [PATCHv2 0/3] pci: remove msi_controller registration API
Quoted 31 lines · Thomas Petazzoni

Hello,

This small patch series aims at removing the small msi_controller registration API implemented in drivers/of/of_pci.c.

In order to do so, it moves the last two users of this API to the generic MSI support:

- The pci-aardvark driver is moved to use the generic MSI support. This driver is both the provider of the MSI capability and the consumer of this capability.

- The pci-mvebu driver is cleaned up from using the msi_controller registration API. This driver is only consuming MSI capabilies from another driver (irq-armada-370-xp) which was migrated to the generic MSI support some time ago. Therefore, this patch is simply removing dead code.

Finally, the last commit removed the now unused API.

Changes between v1 to v2 are described in each individual patch.

Thanks,

Thomas

Thomas Petazzoni (3):
pci: pci-aardvark: move to MSI handling using generic MSI support
pci: pci-mvebu: remove useless MSI enabling code
of: pci: remove unused MSI controller helpers

Quoted 6 lines
 drivers/of/of_pci.c             |  45 -----------
 drivers/pci/host/pci-aardvark.c | 172 +++++++++++++++-------------------------
 drivers/pci/host/pci-mvebu.c    |  17 ----
 include/linux/of_pci.h          |  11 ---
 4 files changed, 66 insertions(+), 179 deletions(-)

I somehow missed this series during the Xmas craze. It is definitely a good cleanup, so for the two patches I haven't Acked yet:

Acked-by Marc Zyngier <[email protected]>

Thanks,

	M.
Signature · 1 line
Jazz is not dead. It just smells funny...

Read the cover letter in its thread