* [PATCH v3 0/5] PCI: Remove unnecessary <linux/of_irq.h> includes
@ 2022-10-31 15:39 Bjorn Helgaas
2022-10-31 15:39 ` [PATCH v3 1/5] PCI: altera-msi: Include <linux/irqdomain.h> explicitly Bjorn Helgaas
` (6 more replies)
0 siblings, 7 replies; 9+ messages in thread
From: Bjorn Helgaas @ 2022-10-31 15:39 UTC (permalink / raw)
To: Lorenzo Pieralisi
Cc: Krzysztof Wilczyński, Heiko Stuebner, linux-pci,
Linus Walleij, Minghuan Lian, Conor Dooley, Thierry Reding,
Thomas Petazzoni, Bharat Kumar Gogada, Toan Le, linux-riscv,
Kishon Vijay Abraham I, Rob Herring, Joyce Ooi, Jonathan Hunter,
linux-rockchip, bcm-kernel-feedback-list, linux-arm-kernel,
Shawn Lin, Ray Jui, linux-tegra, linux-omap, Michal Simek,
Mingkai Hu, Roy Zang, Bjorn Helgaas
From: Bjorn Helgaas <bhelgaas@google.com>
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@kernel.org/
v1: https://lore.kernel.org/r/20221019195452.37606-1-helgaas@kernel.org/
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(-)
--
2.25.1
^ permalink raw reply [flat|nested] 9+ messages in thread
* [PATCH v3 1/5] PCI: altera-msi: Include <linux/irqdomain.h> explicitly
2022-10-31 15:39 [PATCH v3 0/5] PCI: Remove unnecessary <linux/of_irq.h> includes Bjorn Helgaas
@ 2022-10-31 15:39 ` Bjorn Helgaas
2022-10-31 15:39 ` [PATCH v3 2/5] PCI: microchip: " Bjorn Helgaas
` (5 subsequent siblings)
6 siblings, 0 replies; 9+ messages in thread
From: Bjorn Helgaas @ 2022-10-31 15:39 UTC (permalink / raw)
To: Lorenzo Pieralisi
Cc: Krzysztof Wilczyński, Heiko Stuebner, linux-pci,
Linus Walleij, Minghuan Lian, Conor Dooley, Thierry Reding,
Thomas Petazzoni, Bharat Kumar Gogada, Toan Le, linux-riscv,
Kishon Vijay Abraham I, Rob Herring, Joyce Ooi, Jonathan Hunter,
linux-rockchip, bcm-kernel-feedback-list, linux-arm-kernel,
Shawn Lin, Ray Jui, linux-tegra, linux-omap, Michal Simek,
Mingkai Hu, Roy Zang, Bjorn Helgaas
From: Bjorn Helgaas <bhelgaas@google.com>
pcie-altera-msi.c uses irq_domain_add_linear() and related interfaces, so
it needs <linux/irqdomain.h> but doesn't include it directly; it relies on
the fact that <linux/of_irq.h> includes it.
But pcie-altera-msi.c *doesn't* need <linux/of_irq.h> itself. Include
<linux/irqdomain.h> directly to remove this implicit dependency so a future
patch can drop <linux/of_irq.h>.
Signed-off-by: Bjorn Helgaas <bhelgaas@google.com>
---
drivers/pci/controller/pcie-altera-msi.c | 1 +
1 file changed, 1 insertion(+)
diff --git a/drivers/pci/controller/pcie-altera-msi.c b/drivers/pci/controller/pcie-altera-msi.c
index 7b1d3ebc34ec..4366e042e98b 100644
--- a/drivers/pci/controller/pcie-altera-msi.c
+++ b/drivers/pci/controller/pcie-altera-msi.c
@@ -9,6 +9,7 @@
#include <linux/interrupt.h>
#include <linux/irqchip/chained_irq.h>
+#include <linux/irqdomain.h>
#include <linux/init.h>
#include <linux/module.h>
#include <linux/msi.h>
--
2.25.1
^ permalink raw reply related [flat|nested] 9+ messages in thread
* [PATCH v3 2/5] PCI: microchip: Include <linux/irqdomain.h> explicitly
2022-10-31 15:39 [PATCH v3 0/5] PCI: Remove unnecessary <linux/of_irq.h> includes Bjorn Helgaas
2022-10-31 15:39 ` [PATCH v3 1/5] PCI: altera-msi: Include <linux/irqdomain.h> explicitly Bjorn Helgaas
@ 2022-10-31 15:39 ` Bjorn Helgaas
2022-10-31 15:39 ` [PATCH v3 3/5] PCI: mvebu: " Bjorn Helgaas
` (4 subsequent siblings)
6 siblings, 0 replies; 9+ messages in thread
From: Bjorn Helgaas @ 2022-10-31 15:39 UTC (permalink / raw)
To: Lorenzo Pieralisi
Cc: Krzysztof Wilczyński, Heiko Stuebner, linux-pci,
Linus Walleij, Minghuan Lian, Conor Dooley, Thierry Reding,
Thomas Petazzoni, Bharat Kumar Gogada, Toan Le, linux-riscv,
Kishon Vijay Abraham I, Rob Herring, Joyce Ooi, Jonathan Hunter,
linux-rockchip, bcm-kernel-feedback-list, linux-arm-kernel,
Shawn Lin, Ray Jui, linux-tegra, linux-omap, Michal Simek,
Mingkai Hu, Roy Zang, Bjorn Helgaas
From: Bjorn Helgaas <bhelgaas@google.com>
pcie-microchip-host.c uses irq_domain_add_linear() and related interfaces,
so it needs <linux/irqdomain.h> but doesn't include it directly; it relies
on the fact that <linux/of_irq.h> includes it.
But pcie-microchip-host.c *doesn't* need <linux/of_irq.h> itself. Include
<linux/irqdomain.h> directly to remove this implicit dependency so a future
patch can drop <linux/of_irq.h>.
Signed-off-by: Bjorn Helgaas <bhelgaas@google.com>
Reviewed-by: Conor Dooley <conor.dooley@microchip.com>
---
drivers/pci/controller/pcie-microchip-host.c | 1 +
1 file changed, 1 insertion(+)
diff --git a/drivers/pci/controller/pcie-microchip-host.c b/drivers/pci/controller/pcie-microchip-host.c
index 7263d175b5ad..57b2a62f52c8 100644
--- a/drivers/pci/controller/pcie-microchip-host.c
+++ b/drivers/pci/controller/pcie-microchip-host.c
@@ -9,6 +9,7 @@
#include <linux/clk.h>
#include <linux/irqchip/chained_irq.h>
+#include <linux/irqdomain.h>
#include <linux/module.h>
#include <linux/msi.h>
#include <linux/of_address.h>
--
2.25.1
^ permalink raw reply related [flat|nested] 9+ messages in thread
* [PATCH v3 3/5] PCI: mvebu: Include <linux/irqdomain.h> explicitly
2022-10-31 15:39 [PATCH v3 0/5] PCI: Remove unnecessary <linux/of_irq.h> includes Bjorn Helgaas
2022-10-31 15:39 ` [PATCH v3 1/5] PCI: altera-msi: Include <linux/irqdomain.h> explicitly Bjorn Helgaas
2022-10-31 15:39 ` [PATCH v3 2/5] PCI: microchip: " Bjorn Helgaas
@ 2022-10-31 15:39 ` Bjorn Helgaas
2022-10-31 15:39 ` [PATCH v3 4/5] PCI: xgene-msi: " Bjorn Helgaas
` (3 subsequent siblings)
6 siblings, 0 replies; 9+ messages in thread
From: Bjorn Helgaas @ 2022-10-31 15:39 UTC (permalink / raw)
To: Lorenzo Pieralisi
Cc: Krzysztof Wilczyński, Heiko Stuebner, linux-pci,
Linus Walleij, Minghuan Lian, Conor Dooley, Thierry Reding,
Thomas Petazzoni, Bharat Kumar Gogada, Toan Le, linux-riscv,
Kishon Vijay Abraham I, Rob Herring, Joyce Ooi, Jonathan Hunter,
linux-rockchip, bcm-kernel-feedback-list, linux-arm-kernel,
Shawn Lin, Ray Jui, linux-tegra, linux-omap, Michal Simek,
Mingkai Hu, Roy Zang, Bjorn Helgaas
From: Bjorn Helgaas <bhelgaas@google.com>
pci-mvebu.c uses irq_domain_add_linear() and related interfaces but relies
on <linux/irqdomain.h> but doesn't include it directly; it relies on the
fact that <linux/of_irq.h> includes it.
Include <linux/irqdomain.h> directly to remove this implicit dependency.
Signed-off-by: Bjorn Helgaas <bhelgaas@google.com>
Acked-by: Thomas Petazzoni <thomas.petazzoni@bootlin.com>
---
drivers/pci/controller/pci-mvebu.c | 1 +
1 file changed, 1 insertion(+)
diff --git a/drivers/pci/controller/pci-mvebu.c b/drivers/pci/controller/pci-mvebu.c
index 1ced73726a26..73db99035c2b 100644
--- a/drivers/pci/controller/pci-mvebu.c
+++ b/drivers/pci/controller/pci-mvebu.c
@@ -13,6 +13,7 @@
#include <linux/delay.h>
#include <linux/gpio.h>
#include <linux/init.h>
+#include <linux/irqdomain.h>
#include <linux/mbus.h>
#include <linux/slab.h>
#include <linux/platform_device.h>
--
2.25.1
^ permalink raw reply related [flat|nested] 9+ messages in thread
* [PATCH v3 4/5] PCI: xgene-msi: Include <linux/irqdomain.h> explicitly
2022-10-31 15:39 [PATCH v3 0/5] PCI: Remove unnecessary <linux/of_irq.h> includes Bjorn Helgaas
` (2 preceding siblings ...)
2022-10-31 15:39 ` [PATCH v3 3/5] PCI: mvebu: " Bjorn Helgaas
@ 2022-10-31 15:39 ` Bjorn Helgaas
2022-10-31 15:39 ` [PATCH v3 5/5] PCI: Remove unnecessary <linux/of_irq.h> includes Bjorn Helgaas
` (2 subsequent siblings)
6 siblings, 0 replies; 9+ messages in thread
From: Bjorn Helgaas @ 2022-10-31 15:39 UTC (permalink / raw)
To: Lorenzo Pieralisi
Cc: Krzysztof Wilczyński, Heiko Stuebner, linux-pci,
Linus Walleij, Minghuan Lian, Conor Dooley, Thierry Reding,
Thomas Petazzoni, Bharat Kumar Gogada, Toan Le, linux-riscv,
Kishon Vijay Abraham I, Rob Herring, Joyce Ooi, Jonathan Hunter,
linux-rockchip, bcm-kernel-feedback-list, linux-arm-kernel,
Shawn Lin, Ray Jui, linux-tegra, linux-omap, Michal Simek,
Mingkai Hu, Roy Zang, Bjorn Helgaas
From: Bjorn Helgaas <bhelgaas@google.com>
pci-xgene-msi.c uses irq_domain_add_linear() and related interfaces, so it
needs <linux/irqdomain.h> but doesn't include it directly; it relies on the
fact that <linux/of_irq.h> includes it.
But pci-xgene-msi.c *doesn't* need <linux/of_irq.h> itself. Include
<linux/irqdomain.h> directly to remove this implicit dependency so a future
patch can drop <linux/of_irq.h>.
Signed-off-by: Bjorn Helgaas <bhelgaas@google.com>
---
drivers/pci/controller/pci-xgene-msi.c | 1 +
1 file changed, 1 insertion(+)
diff --git a/drivers/pci/controller/pci-xgene-msi.c b/drivers/pci/controller/pci-xgene-msi.c
index bfa259781b69..bacb14e558ee 100644
--- a/drivers/pci/controller/pci-xgene-msi.c
+++ b/drivers/pci/controller/pci-xgene-msi.c
@@ -8,6 +8,7 @@
*/
#include <linux/cpu.h>
#include <linux/interrupt.h>
+#include <linux/irqdomain.h>
#include <linux/module.h>
#include <linux/msi.h>
#include <linux/of_irq.h>
--
2.25.1
^ permalink raw reply related [flat|nested] 9+ messages in thread
* [PATCH v3 5/5] PCI: Remove unnecessary <linux/of_irq.h> includes
2022-10-31 15:39 [PATCH v3 0/5] PCI: Remove unnecessary <linux/of_irq.h> includes Bjorn Helgaas
` (3 preceding siblings ...)
2022-10-31 15:39 ` [PATCH v3 4/5] PCI: xgene-msi: " Bjorn Helgaas
@ 2022-10-31 15:39 ` Bjorn Helgaas
2022-10-31 16:58 ` Roy Zang
2022-11-10 21:02 ` [PATCH v3 0/5] " Bjorn Helgaas
2022-11-11 18:27 ` Rob Herring
6 siblings, 1 reply; 9+ messages in thread
From: Bjorn Helgaas @ 2022-10-31 15:39 UTC (permalink / raw)
To: Lorenzo Pieralisi
Cc: Krzysztof Wilczyński, Heiko Stuebner, linux-pci,
Linus Walleij, Minghuan Lian, Conor Dooley, Thierry Reding,
Thomas Petazzoni, Bharat Kumar Gogada, Toan Le, linux-riscv,
Kishon Vijay Abraham I, Rob Herring, Joyce Ooi, Jonathan Hunter,
linux-rockchip, bcm-kernel-feedback-list, linux-arm-kernel,
Shawn Lin, Ray Jui, linux-tegra, linux-omap, Michal Simek,
Mingkai Hu, Roy Zang, Bjorn Helgaas
From: Bjorn Helgaas <bhelgaas@google.com>
Many host controller drivers #include <linux/of_irq.h> even though they
don't need it. Remove the unnecessary #includes.
Signed-off-by: Bjorn Helgaas <bhelgaas@google.com>
---
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-v3-semi.c | 1 -
drivers/pci/controller/pci-xgene-msi.c | 1 -
drivers/pci/controller/pci-xgene.c | 1 -
drivers/pci/controller/pcie-altera-msi.c | 1 -
drivers/pci/controller/pcie-iproc-platform.c | 1 -
drivers/pci/controller/pcie-iproc.c | 1 -
drivers/pci/controller/pcie-microchip-host.c | 1 -
drivers/pci/controller/pcie-rockchip-host.c | 1 -
drivers/pci/controller/pcie-xilinx-cpm.c | 1 -
drivers/pci/controller/pcie-xilinx-nwl.c | 1 -
14 files changed, 14 deletions(-)
diff --git a/drivers/pci/controller/cadence/pci-j721e.c b/drivers/pci/controller/cadence/pci-j721e.c
index a82f845cc4b5..cc83a8925ce0 100644
--- a/drivers/pci/controller/cadence/pci-j721e.c
+++ b/drivers/pci/controller/cadence/pci-j721e.c
@@ -15,7 +15,6 @@
#include <linux/mfd/syscon.h>
#include <linux/of.h>
#include <linux/of_device.h>
-#include <linux/of_irq.h>
#include <linux/pci.h>
#include <linux/pm_runtime.h>
#include <linux/regmap.h>
diff --git a/drivers/pci/controller/dwc/pci-layerscape.c b/drivers/pci/controller/dwc/pci-layerscape.c
index 879b8692f96a..ed5fb492fe08 100644
--- a/drivers/pci/controller/dwc/pci-layerscape.c
+++ b/drivers/pci/controller/dwc/pci-layerscape.c
@@ -13,7 +13,6 @@
#include <linux/init.h>
#include <linux/of_pci.h>
#include <linux/of_platform.h>
-#include <linux/of_irq.h>
#include <linux/of_address.h>
#include <linux/pci.h>
#include <linux/platform_device.h>
diff --git a/drivers/pci/controller/dwc/pcie-armada8k.c b/drivers/pci/controller/dwc/pcie-armada8k.c
index dc469ef8e99b..5c999e15c357 100644
--- a/drivers/pci/controller/dwc/pcie-armada8k.c
+++ b/drivers/pci/controller/dwc/pcie-armada8k.c
@@ -21,7 +21,6 @@
#include <linux/platform_device.h>
#include <linux/resource.h>
#include <linux/of_pci.h>
-#include <linux/of_irq.h>
#include "pcie-designware.h"
diff --git a/drivers/pci/controller/dwc/pcie-tegra194.c b/drivers/pci/controller/dwc/pcie-tegra194.c
index 1b6b437823d2..02d78a12b6e7 100644
--- a/drivers/pci/controller/dwc/pcie-tegra194.c
+++ b/drivers/pci/controller/dwc/pcie-tegra194.c
@@ -21,7 +21,6 @@
#include <linux/of.h>
#include <linux/of_device.h>
#include <linux/of_gpio.h>
-#include <linux/of_irq.h>
#include <linux/of_pci.h>
#include <linux/pci.h>
#include <linux/phy/phy.h>
diff --git a/drivers/pci/controller/pci-v3-semi.c b/drivers/pci/controller/pci-v3-semi.c
index 154a5398633c..784fcf35599c 100644
--- a/drivers/pci/controller/pci-v3-semi.c
+++ b/drivers/pci/controller/pci-v3-semi.c
@@ -22,7 +22,6 @@
#include <linux/kernel.h>
#include <linux/of_address.h>
#include <linux/of_device.h>
-#include <linux/of_irq.h>
#include <linux/of_pci.h>
#include <linux/pci.h>
#include <linux/platform_device.h>
diff --git a/drivers/pci/controller/pci-xgene-msi.c b/drivers/pci/controller/pci-xgene-msi.c
index bacb14e558ee..d7987b281f79 100644
--- a/drivers/pci/controller/pci-xgene-msi.c
+++ b/drivers/pci/controller/pci-xgene-msi.c
@@ -11,7 +11,6 @@
#include <linux/irqdomain.h>
#include <linux/module.h>
#include <linux/msi.h>
-#include <linux/of_irq.h>
#include <linux/irqchip/chained_irq.h>
#include <linux/pci.h>
#include <linux/platform_device.h>
diff --git a/drivers/pci/controller/pci-xgene.c b/drivers/pci/controller/pci-xgene.c
index 549d3bd6d1c2..887b4941ff32 100644
--- a/drivers/pci/controller/pci-xgene.c
+++ b/drivers/pci/controller/pci-xgene.c
@@ -14,7 +14,6 @@
#include <linux/init.h>
#include <linux/of.h>
#include <linux/of_address.h>
-#include <linux/of_irq.h>
#include <linux/of_pci.h>
#include <linux/pci.h>
#include <linux/pci-acpi.h>
diff --git a/drivers/pci/controller/pcie-altera-msi.c b/drivers/pci/controller/pcie-altera-msi.c
index 4366e042e98b..65e8a20cc442 100644
--- a/drivers/pci/controller/pcie-altera-msi.c
+++ b/drivers/pci/controller/pcie-altera-msi.c
@@ -14,7 +14,6 @@
#include <linux/module.h>
#include <linux/msi.h>
#include <linux/of_address.h>
-#include <linux/of_irq.h>
#include <linux/of_pci.h>
#include <linux/pci.h>
#include <linux/platform_device.h>
diff --git a/drivers/pci/controller/pcie-iproc-platform.c b/drivers/pci/controller/pcie-iproc-platform.c
index 538115246c79..4142a73e611d 100644
--- a/drivers/pci/controller/pcie-iproc-platform.c
+++ b/drivers/pci/controller/pcie-iproc-platform.c
@@ -12,7 +12,6 @@
#include <linux/platform_device.h>
#include <linux/of_address.h>
#include <linux/of_pci.h>
-#include <linux/of_irq.h>
#include <linux/of_platform.h>
#include <linux/phy/phy.h>
diff --git a/drivers/pci/controller/pcie-iproc.c b/drivers/pci/controller/pcie-iproc.c
index 2519201b0e51..83029bdfd884 100644
--- a/drivers/pci/controller/pcie-iproc.c
+++ b/drivers/pci/controller/pcie-iproc.c
@@ -18,7 +18,6 @@
#include <linux/platform_device.h>
#include <linux/of_address.h>
#include <linux/of_pci.h>
-#include <linux/of_irq.h>
#include <linux/of_platform.h>
#include <linux/phy/phy.h>
diff --git a/drivers/pci/controller/pcie-microchip-host.c b/drivers/pci/controller/pcie-microchip-host.c
index 57b2a62f52c8..0ebf7015e9af 100644
--- a/drivers/pci/controller/pcie-microchip-host.c
+++ b/drivers/pci/controller/pcie-microchip-host.c
@@ -13,7 +13,6 @@
#include <linux/module.h>
#include <linux/msi.h>
#include <linux/of_address.h>
-#include <linux/of_irq.h>
#include <linux/of_pci.h>
#include <linux/pci-ecam.h>
#include <linux/platform_device.h>
diff --git a/drivers/pci/controller/pcie-rockchip-host.c b/drivers/pci/controller/pcie-rockchip-host.c
index 7352b5ff8d35..c96c0f454570 100644
--- a/drivers/pci/controller/pcie-rockchip-host.c
+++ b/drivers/pci/controller/pcie-rockchip-host.c
@@ -28,7 +28,6 @@
#include <linux/of_device.h>
#include <linux/of_pci.h>
#include <linux/of_platform.h>
-#include <linux/of_irq.h>
#include <linux/pci.h>
#include <linux/pci_ids.h>
#include <linux/phy/phy.h>
diff --git a/drivers/pci/controller/pcie-xilinx-cpm.c b/drivers/pci/controller/pcie-xilinx-cpm.c
index e4ab48041eb6..4a787a941674 100644
--- a/drivers/pci/controller/pcie-xilinx-cpm.c
+++ b/drivers/pci/controller/pcie-xilinx-cpm.c
@@ -16,7 +16,6 @@
#include <linux/of_address.h>
#include <linux/of_pci.h>
#include <linux/of_platform.h>
-#include <linux/of_irq.h>
#include <linux/pci.h>
#include <linux/platform_device.h>
#include <linux/pci-ecam.h>
diff --git a/drivers/pci/controller/pcie-xilinx-nwl.c b/drivers/pci/controller/pcie-xilinx-nwl.c
index 40d070e54ad2..f0271b6c6f8d 100644
--- a/drivers/pci/controller/pcie-xilinx-nwl.c
+++ b/drivers/pci/controller/pcie-xilinx-nwl.c
@@ -17,7 +17,6 @@
#include <linux/of_address.h>
#include <linux/of_pci.h>
#include <linux/of_platform.h>
-#include <linux/of_irq.h>
#include <linux/pci.h>
#include <linux/pci-ecam.h>
#include <linux/platform_device.h>
--
2.25.1
^ permalink raw reply related [flat|nested] 9+ messages in thread
* RE: [PATCH v3 5/5] PCI: Remove unnecessary <linux/of_irq.h> includes
2022-10-31 15:39 ` [PATCH v3 5/5] PCI: Remove unnecessary <linux/of_irq.h> includes Bjorn Helgaas
@ 2022-10-31 16:58 ` Roy Zang
0 siblings, 0 replies; 9+ messages in thread
From: Roy Zang @ 2022-10-31 16:58 UTC (permalink / raw)
To: Bjorn Helgaas, Lorenzo Pieralisi
Cc: Krzysztof Wilczyński, Heiko Stuebner,
linux-pci@vger.kernel.org, Linus Walleij, M.H. Lian, Conor Dooley,
Thierry Reding, Thomas Petazzoni, Bharat Kumar Gogada, Toan Le,
linux-riscv@lists.infradead.org, Kishon Vijay Abraham I,
Rob Herring, Joyce Ooi, Jonathan Hunter,
linux-rockchip@lists.infradead.org,
bcm-kernel-feedback-list@broadcom.com,
linux-arm-kernel@lists.infradead.org, Shawn Lin, Ray Jui,
linux-tegra@vger.kernel .org, linux-omap@vger.kernel.org,
Michal Simek, Mingkai Hu, Bjorn Helgaas, Scott Branden,
Daire McNamara, linux-kernel@vger.kernel.org, Tom Joseph,
linuxppc-dev@lists.ozlabs.org
> -----Original Message-----
> From: Bjorn Helgaas <helgaas@kernel.org>
> Subject: [PATCH v3 5/5] PCI: Remove unnecessary <linux/of_irq.h> includes
>
> From: Bjorn Helgaas <bhelgaas@google.com>
>
> Many host controller drivers #include <linux/of_irq.h> even though they
> don't need it. Remove the unnecessary #includes.
>
> Signed-off-by: Bjorn Helgaas <bhelgaas@google.com>
> ---
> 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-v3-semi.c | 1 -
> drivers/pci/controller/pci-xgene-msi.c | 1 -
> drivers/pci/controller/pci-xgene.c | 1 -
> drivers/pci/controller/pcie-altera-msi.c | 1 -
> drivers/pci/controller/pcie-iproc-platform.c | 1 -
> drivers/pci/controller/pcie-iproc.c | 1 -
> drivers/pci/controller/pcie-microchip-host.c | 1 - drivers/pci/controller/pcie-
> rockchip-host.c | 1 -
> drivers/pci/controller/pcie-xilinx-cpm.c | 1 -
> drivers/pci/controller/pcie-xilinx-nwl.c | 1 -
> 14 files changed, 14 deletions(-)
Acked-by: Roy Zang <roy.zang@nxp.com>
-R
^ permalink raw reply [flat|nested] 9+ messages in thread
* Re: [PATCH v3 0/5] PCI: Remove unnecessary <linux/of_irq.h> includes
2022-10-31 15:39 [PATCH v3 0/5] PCI: Remove unnecessary <linux/of_irq.h> includes Bjorn Helgaas
` (4 preceding siblings ...)
2022-10-31 15:39 ` [PATCH v3 5/5] PCI: Remove unnecessary <linux/of_irq.h> includes Bjorn Helgaas
@ 2022-11-10 21:02 ` Bjorn Helgaas
2022-11-11 18:27 ` Rob Herring
6 siblings, 0 replies; 9+ messages in thread
From: Bjorn Helgaas @ 2022-11-10 21:02 UTC (permalink / raw)
To: Lorenzo Pieralisi
Cc: Krzysztof Wilczyński, Heiko Stuebner, linux-pci,
Linus Walleij, Minghuan Lian, Conor Dooley, Thierry Reding,
Thomas Petazzoni, Bharat Kumar Gogada, Toan Le, linux-riscv,
Kishon Vijay Abraham I, Rob Herring, Joyce Ooi, Jonathan Hunter,
linux-rockchip, bcm-kernel-feedback-list, linux-arm-kernel,
Shawn Lin, Ray Jui, linux-tegra, linux-omap, Michal Simek,
Mingkai Hu, Roy Zang, Bjorn Helgaas
On Mon, Oct 31, 2022 at 10:39:49AM -0500, Bjorn Helgaas wrote:
> From: Bjorn Helgaas <bhelgaas@google.com>
>
> 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@kernel.org/
> v1: https://lore.kernel.org/r/20221019195452.37606-1-helgaas@kernel.org/
>
> 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
^ permalink raw reply [flat|nested] 9+ messages in thread
* Re: [PATCH v3 0/5] PCI: Remove unnecessary <linux/of_irq.h> includes
2022-10-31 15:39 [PATCH v3 0/5] PCI: Remove unnecessary <linux/of_irq.h> includes Bjorn Helgaas
` (5 preceding siblings ...)
2022-11-10 21:02 ` [PATCH v3 0/5] " Bjorn Helgaas
@ 2022-11-11 18:27 ` Rob Herring
6 siblings, 0 replies; 9+ messages in thread
From: Rob Herring @ 2022-11-11 18:27 UTC (permalink / raw)
To: Bjorn Helgaas
Cc: Krzysztof Wilczyński, Heiko Stuebner, linux-pci,
Lorenzo Pieralisi, Minghuan Lian, Conor Dooley, Thierry Reding,
Thomas Petazzoni, Bharat Kumar Gogada, Toan Le, linux-riscv,
Kishon Vijay Abraham I, Linus Walleij, Joyce Ooi, Jonathan Hunter,
linux-rockchip, bcm-kernel-feedback-list, linux-arm-kernel,
Shawn Lin, Ray Jui, linux-tegra, linux-omap, Michal Simek,
Mingkai Hu, Roy Zang, Bjorn Helgaas, Scott B randen,
Daire McNamara, linux-kernel, Tom Joseph, linuxppc-dev
On Mon, Oct 31, 2022 at 10:40 AM Bjorn Helgaas <helgaas@kernel.org> wrote:
>
> From: Bjorn Helgaas <bhelgaas@google.com>
>
> Many host controller drivers #include <linux/of_irq.h> even though they
> don't need it. Remove the unnecessary #includes.
Note that the same is often true of the other of_*.h headers.
of_device.h and of_platform.h are tricky though as one includes the
other (with a decade old comment to fix). I started down that rabbit
hole once...
Rob
^ permalink raw reply [flat|nested] 9+ messages in thread
end of thread, other threads:[~2022-11-11 18:28 UTC | newest]
Thread overview: 9+ messages (download: mbox.gz follow: Atom feed
-- links below jump to the message on this page --
2022-10-31 15:39 [PATCH v3 0/5] PCI: Remove unnecessary <linux/of_irq.h> includes Bjorn Helgaas
2022-10-31 15:39 ` [PATCH v3 1/5] PCI: altera-msi: Include <linux/irqdomain.h> explicitly Bjorn Helgaas
2022-10-31 15:39 ` [PATCH v3 2/5] PCI: microchip: " Bjorn Helgaas
2022-10-31 15:39 ` [PATCH v3 3/5] PCI: mvebu: " Bjorn Helgaas
2022-10-31 15:39 ` [PATCH v3 4/5] PCI: xgene-msi: " Bjorn Helgaas
2022-10-31 15:39 ` [PATCH v3 5/5] PCI: Remove unnecessary <linux/of_irq.h> includes Bjorn Helgaas
2022-10-31 16:58 ` Roy Zang
2022-11-10 21:02 ` [PATCH v3 0/5] " Bjorn Helgaas
2022-11-11 18:27 ` Rob Herring
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox;
as well as URLs for NNTP newsgroup(s).