On Mon, Oct 31, 2022 at 10:39:49AM -0500, Bjorn Helgaas wrote: > From: Bjorn Helgaas <bhelgaas@xxxxxxxxxx> > > Many host controller drivers #include <linux/of_irq.h> even though they > don't need it. Remove the unnecessary #includes. > > v2: https://lore.kernel.org/r/20221025185147.665365-1-helgaas@xxxxxxxxxx/ > v1: https://lore.kernel.org/r/20221019195452.37606-1-helgaas@xxxxxxxxxx/ > > Changes from v2 to v3: > - Include <linux/irqdomain.h> explicitly in xgene-msi, which doesn't need > <linux/of_irq.h> itself, but relied on it to include > <linux/irqdomain.h>. On x86, this was covered up by the fact that > <linux/msi.h> includes <asm/msi.h>, which includes <asm/irqdomain.h>, > which includes <linux/irqdomain.h>. But on parisc, <asm/msi.h> is > actually asm-generic/msi.h, which does *not* include > <linux/irqdomain.h> > - Pick up tags from Conor Dooley and Thomas Petazzoni > > Changes from v1 to v2: > - Include <linux/irqdomain.h> explicitly in altera-msi and microchip, > which don't need <linux/of_irq.h> itself, but relied on it to include > <linux/irqdomain.h> > - Include <linux/irqdomain.h> explicitly in mvebu, which needs both it > and <linux/of_irq.h> > > Bjorn Helgaas (5): > PCI: altera-msi: Include <linux/irqdomain.h> explicitly > PCI: microchip: Include <linux/irqdomain.h> explicitly > PCI: mvebu: Include <linux/irqdomain.h> explicitly > PCI: xgene-msi: Include <linux/irqdomain.h> explicitly > PCI: Remove unnecessary <linux/of_irq.h> includes > > drivers/pci/controller/cadence/pci-j721e.c | 1 - > drivers/pci/controller/dwc/pci-layerscape.c | 1 - > drivers/pci/controller/dwc/pcie-armada8k.c | 1 - > drivers/pci/controller/dwc/pcie-tegra194.c | 1 - > drivers/pci/controller/pci-mvebu.c | 1 + > drivers/pci/controller/pci-v3-semi.c | 1 - > drivers/pci/controller/pci-xgene-msi.c | 2 +- > drivers/pci/controller/pci-xgene.c | 1 - > drivers/pci/controller/pcie-altera-msi.c | 2 +- > drivers/pci/controller/pcie-iproc-platform.c | 1 - > drivers/pci/controller/pcie-iproc.c | 1 - > drivers/pci/controller/pcie-microchip-host.c | 2 +- > drivers/pci/controller/pcie-rockchip-host.c | 1 - > drivers/pci/controller/pcie-xilinx-cpm.c | 1 - > drivers/pci/controller/pcie-xilinx-nwl.c | 1 - > 15 files changed, 4 insertions(+), 14 deletions(-) I put these on a pci/kbuild branch that I propose to merge in after everything else. Lorenzo, let me know if you prefer another approach. Bjorn
