[PATCH v1 0/4] PCI: dwc: Drop or replace unused linux/gpio.h

[PATCH v1 0/4] PCI: dwc: Drop or replace unused linux/gpio.h

[Andy Shevchenko]

The <linux/gpio.h> is a legacy header that is going to die.
Make sure that PCI DesignWare drivers do not include it.

Compile tested on x86_64.

Andy Shevchenko (4):
  PCI: amd-mdb: Use the right GPIO header
  PCI: designware-plat: Drop unused include
  PCI: fu740: Drop unused include
  PCI: visconti: Drop unused include

 drivers/pci/controller/dwc/pcie-amd-mdb.c         | 2 +-
 drivers/pci/controller/dwc/pcie-designware-plat.c | 1 -
 drivers/pci/controller/dwc/pcie-fu740.c           | 1 -
 drivers/pci/controller/dwc/pcie-visconti.c        | 1 -
 4 files changed, 1 insertion(+), 4 deletions(-)

-- 
2.50.1

[PATCH v1 1/4] PCI: amd-mdb: Use the right GPIO header

[Andy Shevchenko]

The driver includes the legacy GPIO header <linux/gpio.h> but does not
use any symbols from it and actually wants <linux/gpio/consumer.h>,
so fix this up.

Signed-off-by: Andy Shevchenko <andriy.shevchenko@linux.intel.com>
---
 drivers/pci/controller/dwc/pcie-amd-mdb.c | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

diff --git a/drivers/pci/controller/dwc/pcie-amd-mdb.c b/drivers/pci/controller/dwc/pcie-amd-mdb.c
index 7e50e11fbffd..88208e967201 100644
--- a/drivers/pci/controller/dwc/pcie-amd-mdb.c
+++ b/drivers/pci/controller/dwc/pcie-amd-mdb.c
@@ -7,7 +7,7 @@
 
 #include <linux/clk.h>
 #include <linux/delay.h>
-#include <linux/gpio.h>
+#include <linux/gpio/consumer.h>
 #include <linux/interrupt.h>
 #include <linux/irqdomain.h>
 #include <linux/kernel.h>
-- 
2.50.1

[PATCH v1 2/4] PCI: designware-plat: Drop unused include

[Andy Shevchenko]

This file does not use the symbols from the legacy
<linux/gpio.h> header, so let's drop it.

Signed-off-by: Andy Shevchenko <andriy.shevchenko@linux.intel.com>
---
 drivers/pci/controller/dwc/pcie-designware-plat.c | 1 -
 1 file changed, 1 deletion(-)

diff --git a/drivers/pci/controller/dwc/pcie-designware-plat.c b/drivers/pci/controller/dwc/pcie-designware-plat.c
index d103ab759c4e..7d896752d43c 100644
--- a/drivers/pci/controller/dwc/pcie-designware-plat.c
+++ b/drivers/pci/controller/dwc/pcie-designware-plat.c
@@ -8,7 +8,6 @@
  */
 #include <linux/clk.h>
 #include <linux/delay.h>
-#include <linux/gpio.h>
 #include <linux/interrupt.h>
 #include <linux/kernel.h>
 #include <linux/init.h>
-- 
2.50.1

[PATCH v1 3/4] PCI: fu740: Drop unused include

[Andy Shevchenko]

This file does not use the symbols from the legacy
<linux/gpio.h> header, so let's drop it.

Signed-off-by: Andy Shevchenko <andriy.shevchenko@linux.intel.com>
---
 drivers/pci/controller/dwc/pcie-fu740.c | 1 -
 1 file changed, 1 deletion(-)

diff --git a/drivers/pci/controller/dwc/pcie-fu740.c b/drivers/pci/controller/dwc/pcie-fu740.c
index 66367252032b..d0a34f680397 100644
--- a/drivers/pci/controller/dwc/pcie-fu740.c
+++ b/drivers/pci/controller/dwc/pcie-fu740.c
@@ -13,7 +13,6 @@
 
 #include <linux/clk.h>
 #include <linux/delay.h>
-#include <linux/gpio.h>
 #include <linux/gpio/consumer.h>
 #include <linux/kernel.h>
 #include <linux/module.h>
-- 
2.50.1

[PATCH v1 4/4] PCI: visconti: Drop unused include

[Andy Shevchenko]

This file does not use the symbols from the legacy
<linux/gpio.h> header, so let's drop it.

Signed-off-by: Andy Shevchenko <andriy.shevchenko@linux.intel.com>
---
 drivers/pci/controller/dwc/pcie-visconti.c | 1 -
 1 file changed, 1 deletion(-)

diff --git a/drivers/pci/controller/dwc/pcie-visconti.c b/drivers/pci/controller/dwc/pcie-visconti.c
index cdeac6177143..f21775c3b3a1 100644
--- a/drivers/pci/controller/dwc/pcie-visconti.c
+++ b/drivers/pci/controller/dwc/pcie-visconti.c
@@ -10,7 +10,6 @@
 
 #include <linux/clk.h>
 #include <linux/delay.h>
-#include <linux/gpio.h>
 #include <linux/interrupt.h>
 #include <linux/init.h>
 #include <linux/iopoll.h>
-- 
2.50.1

Re: [PATCH v1 0/4] PCI: dwc: Drop or replace unused linux/gpio.h

[Manivannan Sadhasivam]

On Wed, 06 May 2026 10:47:23 +0200, Andy Shevchenko wrote:
> The <linux/gpio.h> is a legacy header that is going to die.
> Make sure that PCI DesignWare drivers do not include it.
> 
> Compile tested on x86_64.
> 
> Andy Shevchenko (4):
>   PCI: amd-mdb: Use the right GPIO header
>   PCI: designware-plat: Drop unused include
>   PCI: fu740: Drop unused include
>   PCI: visconti: Drop unused include
> 
> [...]

Applied, thanks!

[1/4] PCI: amd-mdb: Use the right GPIO header
      commit: 8d846a0a6f95cd717df892b077a1c990a86b29fa
[2/4] PCI: designware-plat: Drop unused include
      commit: 800c861cfcbedd033bdffa65c00188fefdcf0767
[3/4] PCI: fu740: Drop unused include
      commit: 71d007f6fb17f519c2fe51d35e0bee5f602169e6
[4/4] PCI: visconti: Drop unused include
      commit: 7cc21269ef74269e423d64defc940808ccbf9774

Best regards,
-- 
Manivannan Sadhasivam <mani@kernel.org>