mbox series

[0/2] PCI/AER: Remove redundant Device Control Error Reporting Enable

Message ID 20230307192655.874008-1-helgaas@kernel.org
Headers
Series PCI/AER: Remove redundant Device Control Error Reporting Enable |

Message

Bjorn Helgaas March 7, 2023, 7:26 p.m. UTC 
  From: Bjorn Helgaas <bhelgaas@google.com>

Since f26e58bf6f54 ("PCI/AER: Enable error reporting when AER is native"),
which appeared in v6.0, the PCI core has enabled PCIe error reporting for
all devices during enumeration.

Remove driver code to do this and remove unnecessary includes of
<linux/aer.h>.

Bjorn Helgaas (2):
  dmaengine: ioat: Drop redundant pci_enable_pcie_error_reporting()
  dmaengine: idxd: Remove unnecessary aer.h include

 drivers/dma/idxd/init.c | 1 -
 drivers/dma/ioat/init.c | 6 ------
 2 files changed, 7 deletions(-)

Comments

Dave Jiang March 7, 2023, 8:06 p.m. UTC | #1 
On 3/7/23 12:26 PM, 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: Fenghua Yu <fenghua.yu@intel.com>
> Cc: Dave Jiang <dave.jiang@intel.com>

Acked-by: Dave Jiang <dave.jiang@intel.com>

> ---
>   drivers/dma/idxd/init.c | 1 -
>   1 file changed, 1 deletion(-)
> 
> diff --git a/drivers/dma/idxd/init.c b/drivers/dma/idxd/init.c
> index 640d3048368e..a85efcda7095 100644
> --- a/drivers/dma/idxd/init.c
> +++ b/drivers/dma/idxd/init.c
> @@ -9,7 +9,6 @@
>   #include <linux/delay.h>
>   #include <linux/dma-mapping.h>
>   #include <linux/workqueue.h>
> -#include <linux/aer.h>
>   #include <linux/fs.h>
>   #include <linux/io-64-nonatomic-lo-hi.h>
>   #include <linux/device.h>
Vinod Koul March 17, 2023, 5:46 p.m. UTC | #2 
On 07-03-23, 13:26, Bjorn Helgaas wrote:
> From: Bjorn Helgaas <bhelgaas@google.com>
> 
> Since f26e58bf6f54 ("PCI/AER: Enable error reporting when AER is native"),
> which appeared in v6.0, the PCI core has enabled PCIe error reporting for
> all devices during enumeration.
> 
> Remove driver code to do this and remove unnecessary includes of
> <linux/aer.h>.

Applied, thanks