On Tue, Jun 09, 2020 at 05:39:48PM +0200, refactormyself@xxxxxxxxx wrote: > From: Bolarinwa Olayemi Saheed <refactormyself@xxxxxxxxx> > > pci_enable_pcie_error_reporting() return PCIBIOS_ error codes which were > passed on down the call heirarchy pcie capability accessors. > > PCIBIOS_ error codes have positive values. Passing on these values is > inconsistent with functions which return only a negative value on failure. > > Before passing on return value of pcie capability accessors, call > pcibios_err_to_errno() to convert any positive PCIBIOS_ error codes to > negative non-PCI generic error values. > > Signed-off-by: Bolarinwa Olayemi Saheed <refactormyself@xxxxxxxxx> > Suggested-by: Bjorn Helgaas <bjorn@xxxxxxxxxxx> > --- > drivers/pci/pcie/aer.c | 6 +++++- > 1 file changed, 5 insertions(+), 1 deletion(-) > > diff --git a/drivers/pci/pcie/aer.c b/drivers/pci/pcie/aer.c > index f4274d301235..95d480a52078 100644 > --- a/drivers/pci/pcie/aer.c > +++ b/drivers/pci/pcie/aer.c > @@ -349,13 +349,17 @@ bool aer_acpi_firmware_first(void) > > int pci_enable_pcie_error_reporting(struct pci_dev *dev) > { > + int rc; > + > if (pcie_aer_get_firmware_first(dev)) > return -EIO; > > if (!dev->aer_cap) > return -EIO; > > - return pcie_capability_set_word(dev, PCI_EXP_DEVCTL, PCI_EXP_AER_FLAGS); > + rc = pcie_capability_set_word(dev, PCI_EXP_DEVCTL, PCI_EXP_AER_FLAGS); > + > + return rc; I think this is missing the pcibios_err_to_errno()? > } > EXPORT_SYMBOL_GPL(pci_enable_pcie_error_reporting); > > -- > 2.18.2 >