* [PATCH] cxl/pci: Remove unnecessary aer.h include
@ 2023-07-10 23:14 Bjorn Helgaas
2023-07-11 8:15 ` Jonathan Cameron
2023-07-12 16:02 ` Dave Jiang
0 siblings, 2 replies; 5+ messages in thread
From: Bjorn Helgaas @ 2023-07-10 23:14 UTC (permalink / raw)
To: Alison Schofield, Vishal Verma, Ira Weiny, Ben Widawsky,
Dan Williams
Cc: linux-cxl, linux-pci, linux-kernel, Bjorn Helgaas
From: Bjorn Helgaas <bhelgaas@google.com>
<linux/aer.h> is unused, so remove it.
Signed-off-by: Bjorn Helgaas <bhelgaas@google.com>
Reviewed-by: Ira Weiny <ira.weiny@intel.com>
Cc: Alison Schofield <alison.schofield@intel.com>
Cc: Vishal Verma <vishal.l.verma@intel.com>
Cc: Ben Widawsky <bwidawsk@kernel.org>
Cc: Dan Williams <dan.j.williams@intel.com>
---
drivers/cxl/pci.c | 1 -
1 file changed, 1 deletion(-)
diff --git a/drivers/cxl/pci.c b/drivers/cxl/pci.c
index 1cb1494c28fe..8314cf2ac16a 100644
--- a/drivers/cxl/pci.c
+++ b/drivers/cxl/pci.c
@@ -8,7 +8,6 @@
#include <linux/mutex.h>
#include <linux/list.h>
#include <linux/pci.h>
-#include <linux/aer.h>
#include <linux/io.h>
#include "cxlmem.h"
#include "cxlpci.h"
--
2.34.1
^ permalink raw reply related [flat|nested] 5+ messages in thread
* Re: [PATCH] cxl/pci: Remove unnecessary aer.h include
2023-07-10 23:14 [PATCH] cxl/pci: Remove unnecessary aer.h include Bjorn Helgaas
@ 2023-07-11 8:15 ` Jonathan Cameron
2023-07-12 16:02 ` Dave Jiang
1 sibling, 0 replies; 5+ messages in thread
From: Jonathan Cameron @ 2023-07-11 8:15 UTC (permalink / raw)
To: Bjorn Helgaas
Cc: Alison Schofield, Vishal Verma, Ira Weiny, Ben Widawsky,
Dan Williams, linux-cxl, linux-pci, linux-kernel, Bjorn Helgaas
On Mon, 10 Jul 2023 18:14:16 -0500
Bjorn Helgaas <helgaas@kernel.org> wrote:
> From: Bjorn Helgaas <bhelgaas@google.com>
>
> <linux/aer.h> is unused, so remove it.
>
> Signed-off-by: Bjorn Helgaas <bhelgaas@google.com>
> Reviewed-by: Ira Weiny <ira.weiny@intel.com>
> Cc: Alison Schofield <alison.schofield@intel.com>
> Cc: Vishal Verma <vishal.l.verma@intel.com>
> Cc: Ben Widawsky <bwidawsk@kernel.org>
> Cc: Dan Williams <dan.j.williams@intel.com>
Was just working on splitting this file up as part of forward porting
the switch-cci and noticed this as well!
Lot of churn in this file in the recent past and I'll make it worse,
so good to keep on top of these includes.
Reviewed-by: Jonathan Cameron <Jonathan.Cameron@huawei.com>
> ---
> drivers/cxl/pci.c | 1 -
> 1 file changed, 1 deletion(-)
>
> diff --git a/drivers/cxl/pci.c b/drivers/cxl/pci.c
> index 1cb1494c28fe..8314cf2ac16a 100644
> --- a/drivers/cxl/pci.c
> +++ b/drivers/cxl/pci.c
> @@ -8,7 +8,6 @@
> #include <linux/mutex.h>
> #include <linux/list.h>
> #include <linux/pci.h>
> -#include <linux/aer.h>
> #include <linux/io.h>
> #include "cxlmem.h"
> #include "cxlpci.h"
^ permalink raw reply [flat|nested] 5+ messages in thread
* Re: [PATCH] cxl/pci: Remove unnecessary aer.h include
2023-07-10 23:14 [PATCH] cxl/pci: Remove unnecessary aer.h include Bjorn Helgaas
2023-07-11 8:15 ` Jonathan Cameron
@ 2023-07-12 16:02 ` Dave Jiang
1 sibling, 0 replies; 5+ messages in thread
From: Dave Jiang @ 2023-07-12 16:02 UTC (permalink / raw)
To: Bjorn Helgaas, Alison Schofield, Vishal Verma, Ira Weiny,
Ben Widawsky, Dan Williams
Cc: linux-cxl, linux-pci, linux-kernel, Bjorn Helgaas
On 7/10/23 16:14, Bjorn Helgaas wrote:
> From: Bjorn Helgaas <bhelgaas@google.com>
>
> <linux/aer.h> is unused, so remove it.
>
> Signed-off-by: Bjorn Helgaas <bhelgaas@google.com>
> Reviewed-by: Ira Weiny <ira.weiny@intel.com>
> Cc: Alison Schofield <alison.schofield@intel.com>
> Cc: Vishal Verma <vishal.l.verma@intel.com>
> Cc: Ben Widawsky <bwidawsk@kernel.org>
> Cc: Dan Williams <dan.j.williams@intel.com>
Reviewed-by: Dave Jiang <dave.jiang@intel.com>
> ---
> drivers/cxl/pci.c | 1 -
> 1 file changed, 1 deletion(-)
>
> diff --git a/drivers/cxl/pci.c b/drivers/cxl/pci.c
> index 1cb1494c28fe..8314cf2ac16a 100644
> --- a/drivers/cxl/pci.c
> +++ b/drivers/cxl/pci.c
> @@ -8,7 +8,6 @@
> #include <linux/mutex.h>
> #include <linux/list.h>
> #include <linux/pci.h>
> -#include <linux/aer.h>
> #include <linux/io.h>
> #include "cxlmem.h"
> #include "cxlpci.h"
^ permalink raw reply [flat|nested] 5+ messages in thread
* [PATCH] cxl/pci: Remove unnecessary aer.h include
@ 2023-03-07 16:24 Bjorn Helgaas
2023-03-07 20:46 ` Ira Weiny
0 siblings, 1 reply; 5+ messages in thread
From: Bjorn Helgaas @ 2023-03-07 16:24 UTC (permalink / raw)
To: Alison Schofield, Vishal Verma, Ira Weiny, Ben Widawsky,
Dan Williams
Cc: linux-cxl, linux-kernel, Bjorn Helgaas
From: Bjorn Helgaas <bhelgaas@google.com>
<linux/aer.h> is unused, so remove it.
Signed-off-by: Bjorn Helgaas <bhelgaas@google.com>
Cc: Alison Schofield <alison.schofield@intel.com>
Cc: Vishal Verma <vishal.l.verma@intel.com>
Cc: Ira Weiny <ira.weiny@intel.com>
Cc: Ben Widawsky <bwidawsk@kernel.org>
Cc: Dan Williams <dan.j.williams@intel.com>
Cc: linux-cxl@vger.kernel.org
---
drivers/cxl/pci.c | 1 -
1 file changed, 1 deletion(-)
diff --git a/drivers/cxl/pci.c b/drivers/cxl/pci.c
index 60b23624d167..d3bc450717ce 100644
--- a/drivers/cxl/pci.c
+++ b/drivers/cxl/pci.c
@@ -9,7 +9,6 @@
#include <linux/list.h>
#include <linux/pci.h>
#include <linux/pci-doe.h>
-#include <linux/aer.h>
#include <linux/io.h>
#include "cxlmem.h"
#include "cxlpci.h"
--
2.25.1
^ permalink raw reply related [flat|nested] 5+ messages in thread
* Re: [PATCH] cxl/pci: Remove unnecessary aer.h include
2023-03-07 16:24 Bjorn Helgaas
@ 2023-03-07 20:46 ` Ira Weiny
0 siblings, 0 replies; 5+ messages in thread
From: Ira Weiny @ 2023-03-07 20:46 UTC (permalink / raw)
To: Bjorn Helgaas, Alison Schofield, Vishal Verma, Ira Weiny,
Ben Widawsky, Dan Williams
Cc: linux-cxl, linux-kernel, Bjorn Helgaas
Bjorn Helgaas wrote:
> From: Bjorn Helgaas <bhelgaas@google.com>
>
> <linux/aer.h> is unused, so remove it.
>
> Signed-off-by: Bjorn Helgaas <bhelgaas@google.com>
> Cc: Alison Schofield <alison.schofield@intel.com>
> Cc: Vishal Verma <vishal.l.verma@intel.com>
> Cc: Ira Weiny <ira.weiny@intel.com>
Reviewed-by: Ira Weiny <ira.weiny@intel.com>
> Cc: Ben Widawsky <bwidawsk@kernel.org>
> Cc: Dan Williams <dan.j.williams@intel.com>
> Cc: linux-cxl@vger.kernel.org
> ---
> drivers/cxl/pci.c | 1 -
> 1 file changed, 1 deletion(-)
>
> diff --git a/drivers/cxl/pci.c b/drivers/cxl/pci.c
> index 60b23624d167..d3bc450717ce 100644
> --- a/drivers/cxl/pci.c
> +++ b/drivers/cxl/pci.c
> @@ -9,7 +9,6 @@
> #include <linux/list.h>
> #include <linux/pci.h>
> #include <linux/pci-doe.h>
> -#include <linux/aer.h>
> #include <linux/io.h>
> #include "cxlmem.h"
> #include "cxlpci.h"
> --
> 2.25.1
>
^ permalink raw reply [flat|nested] 5+ messages in thread
end of thread, other threads:[~2023-07-12 16:03 UTC | newest]
Thread overview: 5+ messages (download: mbox.gz follow: Atom feed
-- links below jump to the message on this page --
2023-07-10 23:14 [PATCH] cxl/pci: Remove unnecessary aer.h include Bjorn Helgaas
2023-07-11 8:15 ` Jonathan Cameron
2023-07-12 16:02 ` Dave Jiang
-- strict thread matches above, loose matches on Subject: below --
2023-03-07 16:24 Bjorn Helgaas
2023-03-07 20:46 ` Ira Weiny
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox