Re: [PATCH 17/28] sfc/siena: Drop redundant pci_enable_pcie_error_reporting()

From: Edward Cree
Date: Wed Mar 08 2023 - 11:13:55 EST


On 07/03/2023 18:19, Bjorn Helgaas wrote:
> From: Bjorn Helgaas <bhelgaas@xxxxxxxxxx>
>
> pci_enable_pcie_error_reporting() enables the device to send ERR_*
> Messages. Since f26e58bf6f54 ("PCI/AER: Enable error reporting when AER is
> native"), the PCI core does this for all devices during enumeration, so the
> driver doesn't need to do it itself.
>
> Remove the redundant pci_enable_pcie_error_reporting() call from the
> driver. Also remove the corresponding pci_disable_pcie_error_reporting()
> from the driver .remove() path.
>
> Note that this only controls ERR_* Messages from the device. An ERR_*
> Message may cause the Root Port to generate an interrupt, depending on the
> AER Root Error Command register managed by the AER service driver.
>
> Signed-off-by: Bjorn Helgaas <bhelgaas@xxxxxxxxxx>
> Cc: Edward Cree <ecree.xilinx@xxxxxxxxx>
> Cc: Martin Habets <habetsm.xilinx@xxxxxxxxx>
> ---

Acked-by: Edward Cree <ecree.xilinx@xxxxxxxxx>