* [PATCH 0/3] PCI: Remove unnecessary headers include
@ 2020-08-13 13:51 Gustavo Pimentel
2020-08-13 13:51 ` [PATCH 1/3] PCI: Remove unnecessary header include (linux/of_pci.h) Gustavo Pimentel
` (3 more replies)
0 siblings, 4 replies; 5+ messages in thread
From: Gustavo Pimentel @ 2020-08-13 13:51 UTC (permalink / raw)
To: bhelgaas, linux-pci; +Cc: 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
^ permalink raw reply [flat|nested] 5+ messages in thread
* [PATCH 1/3] PCI: Remove unnecessary header include (linux/of_pci.h)
2020-08-13 13:51 [PATCH 0/3] PCI: Remove unnecessary headers include Gustavo Pimentel
@ 2020-08-13 13:51 ` Gustavo Pimentel
2020-08-13 13:51 ` [PATCH 2/3] PCI: Remove unnecessary header include (linux/pci-ats.h) Gustavo Pimentel
` (2 subsequent siblings)
3 siblings, 0 replies; 5+ messages in thread
From: Gustavo Pimentel @ 2020-08-13 13:51 UTC (permalink / raw)
To: bhelgaas, linux-pci; +Cc: Gustavo Pimentel, Joao Pinto
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
^ permalink raw reply related [flat|nested] 5+ messages in thread
* [PATCH 2/3] PCI: Remove unnecessary header include (linux/pci-ats.h)
2020-08-13 13:51 [PATCH 0/3] PCI: Remove unnecessary headers include Gustavo Pimentel
2020-08-13 13:51 ` [PATCH 1/3] PCI: Remove unnecessary header include (linux/of_pci.h) Gustavo Pimentel
@ 2020-08-13 13:51 ` Gustavo Pimentel
2020-08-13 13:51 ` [PATCH 3/3] PCI: Remove unnecessary header include (asm/setup.h) Gustavo Pimentel
2020-08-20 20:58 ` [PATCH 0/3] PCI: Remove unnecessary headers include Bjorn Helgaas
3 siblings, 0 replies; 5+ messages in thread
From: Gustavo Pimentel @ 2020-08-13 13:51 UTC (permalink / raw)
To: bhelgaas, linux-pci; +Cc: Gustavo Pimentel, Joao Pinto
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
^ permalink raw reply related [flat|nested] 5+ messages in thread
* [PATCH 3/3] PCI: Remove unnecessary header include (asm/setup.h)
2020-08-13 13:51 [PATCH 0/3] PCI: Remove unnecessary headers include Gustavo Pimentel
2020-08-13 13:51 ` [PATCH 1/3] PCI: Remove unnecessary header include (linux/of_pci.h) Gustavo Pimentel
2020-08-13 13:51 ` [PATCH 2/3] PCI: Remove unnecessary header include (linux/pci-ats.h) Gustavo Pimentel
@ 2020-08-13 13:51 ` Gustavo Pimentel
2020-08-20 20:58 ` [PATCH 0/3] PCI: Remove unnecessary headers include Bjorn Helgaas
3 siblings, 0 replies; 5+ messages in thread
From: Gustavo Pimentel @ 2020-08-13 13:51 UTC (permalink / raw)
To: bhelgaas, linux-pci; +Cc: Gustavo Pimentel, Joao Pinto
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
^ permalink raw reply related [flat|nested] 5+ messages in thread
* Re: [PATCH 0/3] PCI: Remove unnecessary headers include
2020-08-13 13:51 [PATCH 0/3] PCI: Remove unnecessary headers include Gustavo Pimentel
` (2 preceding siblings ...)
2020-08-13 13:51 ` [PATCH 3/3] PCI: Remove unnecessary header include (asm/setup.h) Gustavo Pimentel
@ 2020-08-20 20:58 ` Bjorn Helgaas
3 siblings, 0 replies; 5+ messages in thread
From: Bjorn Helgaas @ 2020-08-20 20:58 UTC (permalink / raw)
To: Gustavo Pimentel; +Cc: bhelgaas, linux-pci
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
>
^ permalink raw reply [flat|nested] 5+ messages in thread
end of thread, other threads:[~2020-08-20 20:58 UTC | newest]
Thread overview: 5+ messages (download: mbox.gz / follow: Atom feed)
-- links below jump to the message on this page --
2020-08-13 13:51 [PATCH 0/3] PCI: Remove unnecessary headers include Gustavo Pimentel
2020-08-13 13:51 ` [PATCH 1/3] PCI: Remove unnecessary header include (linux/of_pci.h) Gustavo Pimentel
2020-08-13 13:51 ` [PATCH 2/3] PCI: Remove unnecessary header include (linux/pci-ats.h) Gustavo Pimentel
2020-08-13 13:51 ` [PATCH 3/3] PCI: Remove unnecessary header include (asm/setup.h) Gustavo Pimentel
2020-08-20 20:58 ` [PATCH 0/3] PCI: Remove unnecessary headers include Bjorn Helgaas
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).