On Tue, Oct 25, 2022 at 01:51:43PM -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. > > v1: https://lore.kernel.org/all/20221019195452.37606-1-helgaas@xxxxxxxxxx/ > > 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 (4): > PCI: altera-msi: Include <linux/irqdomain.h> explicitly > PCI: microchip: Include <linux/irqdomain.h> explicitly > PCI: mvebu: Include <linux/irqdomain.h> explicitly > PCI: Remove unnecessary <linux/of_irq.h> includes > drivers/pci/controller/pcie-microchip-host.c | 2 +- Hey Bjorn, actually did the build this time rather than visually inspecting... For the microchip bits: Reviewed-by: Conor Dooley <conor.dooley@xxxxxxxxxxxxx> Thanks!