[PATCH 0/3] PCI: Remove unnecessary headers include

[PATCH 0/3] PCI: Remove unnecessary headers include

[Gustavo Pimentel]

Coverity Scan has detected 3 headers include that don't provide
any symbol required by the current code implemented on
drivers/pci/pci.c, therefore this patchset aims to remove those
entries.

Gustavo Pimentel (3):
  PCI: Remove unnecessary header include (linux/of_pci.h)
  PCI: Remove unnecessary header include (linux/pci-ats.h)
  PCI: Remove unnecessary header include (asm/setup.h)

 drivers/pci/pci.c | 3 ---
 1 file changed, 3 deletions(-)

-- 
2.7.4

[PATCH 1/3] PCI: Remove unnecessary header include (linux/of_pci.h)

[Gustavo Pimentel]

Remove unnecessary header include (linux/of_pci.h) since it doesn't provide
any needed symbols.

Detected by CoverityScan CID 16442 ("Unnecessary header file (HFA)")

Cc: Joao Pinto <jpinto@synopsys.com>
Signed-off-by: Gustavo Pimentel <gustavo.pimentel@synopsys.com>
---
 drivers/pci/pci.c | 1 -
 1 file changed, 1 deletion(-)

diff --git a/drivers/pci/pci.c b/drivers/pci/pci.c
index c9338f9..e88a169 100644
--- a/drivers/pci/pci.c
+++ b/drivers/pci/pci.c
@@ -15,7 +15,6 @@
 #include <linux/init.h>
 #include <linux/msi.h>
 #include <linux/of.h>
-#include <linux/of_pci.h>
 #include <linux/pci.h>
 #include <linux/pm.h>
 #include <linux/slab.h>
-- 
2.7.4

[PATCH 2/3] PCI: Remove unnecessary header include (linux/pci-ats.h)

[Gustavo Pimentel]

Remove unnecessary header include (linux/pci-ats.h) since it doesn't provide
any needed symbols.

Detected by CoverityScan CID 16443 ("Unnecessary header file (HFA)")

Cc: Joao Pinto <jpinto@synopsys.com>
Signed-off-by: Gustavo Pimentel <gustavo.pimentel@synopsys.com>
---
 drivers/pci/pci.c | 1 -
 1 file changed, 1 deletion(-)

diff --git a/drivers/pci/pci.c b/drivers/pci/pci.c
index e88a169..af68e79 100644
--- a/drivers/pci/pci.c
+++ b/drivers/pci/pci.c
@@ -29,7 +29,6 @@
 #include <linux/pm_runtime.h>
 #include <linux/pci_hotplug.h>
 #include <linux/vmalloc.h>
-#include <linux/pci-ats.h>
 #include <asm/setup.h>
 #include <asm/dma.h>
 #include <linux/aer.h>
-- 
2.7.4

[PATCH 3/3] PCI: Remove unnecessary header include (asm/setup.h)

[Gustavo Pimentel]

Remove unnecessary header include (asm/setup.h) since it doesn't provide
any needed symbols.

Detected by CoverityScan CID 16444 ("Unnecessary header file (HFA)")

Cc: Joao Pinto <jpinto@synopsys.com>
Signed-off-by: Gustavo Pimentel <gustavo.pimentel@synopsys.com>
---
 drivers/pci/pci.c | 1 -
 1 file changed, 1 deletion(-)

diff --git a/drivers/pci/pci.c b/drivers/pci/pci.c
index af68e79..39d808a 100644
--- a/drivers/pci/pci.c
+++ b/drivers/pci/pci.c
@@ -29,7 +29,6 @@
 #include <linux/pm_runtime.h>
 #include <linux/pci_hotplug.h>
 #include <linux/vmalloc.h>
-#include <asm/setup.h>
 #include <asm/dma.h>
 #include <linux/aer.h>
 #include "pci.h"
-- 
2.7.4

Re: [PATCH 0/3] PCI: Remove unnecessary headers include

[Bjorn Helgaas]

On Thu, Aug 13, 2020 at 03:51:50PM +0200, Gustavo Pimentel wrote:
> Coverity Scan has detected 3 headers include that don't provide
> any symbol required by the current code implemented on
> drivers/pci/pci.c, therefore this patchset aims to remove those
> entries.
> 
> Gustavo Pimentel (3):
>   PCI: Remove unnecessary header include (linux/of_pci.h)
>   PCI: Remove unnecessary header include (linux/pci-ats.h)
>   PCI: Remove unnecessary header include (asm/setup.h)

Applied to pci/misc for v5.10, thanks, Gustavo!

> 
>  drivers/pci/pci.c | 3 ---
>  1 file changed, 3 deletions(-)
> 
> -- 
> 2.7.4
>