diff options
| author |   Heiner Kallweit <hkallweit1@gmail.com> | 2020-02-29 23:24:23 +0100 |
|---|---|---|
| committer |   David S. Miller <davem@davemloft.net> | 2020-03-04 14:21:00 -0800 |
| commit | ec5d9e87842a43be3a10ada0d5f560bbd3f31d5d (patch) | |
| tree | e340fc0b992a0dfc1e44f696f033d1419b4e0666 /drivers/pci/pci.c | |
| parent | PCI: Add constant PCI_STATUS_ERROR_BITS (diff) | |
| download | wireguard-linux-ec5d9e87842a43be3a10ada0d5f560bbd3f31d5d.tar.xz wireguard-linux-ec5d9e87842a43be3a10ada0d5f560bbd3f31d5d.zip | |
PCI: Add pci_status_get_and_clear_errors
Several drivers use the following code sequence:
1. Read PCI_STATUS
2. Mask out non-error bits
3. Action based on error bits set
4. Write back set error bits to clear them
As this is a repeated pattern, add a helper to the PCI core.
Signed-off-by: Heiner Kallweit <hkallweit1@gmail.com>
Acked-by: Bjorn Helgaas <bhelgaas@google.com>
Signed-off-by: David S. Miller <davem@davemloft.net>
Diffstat (limited to 'drivers/pci/pci.c')
| -rw-r--r-- | drivers/pci/pci.c | 23 |
1 files changed, 23 insertions, 0 deletions
diff --git a/drivers/pci/pci.c b/drivers/pci/pci.c index d828ca835a98..c16b0ba2a895 100644 --- a/drivers/pci/pci.c +++ b/drivers/pci/pci.c @@ -173,6 +173,29 @@ unsigned char pci_bus_max_busnr(struct pci_bus *bus) } EXPORT_SYMBOL_GPL(pci_bus_max_busnr); +/** + * pci_status_get_and_clear_errors - return and clear error bits in PCI_STATUS + * @pdev: the PCI device + * + * Returns error bits set in PCI_STATUS and clears them. + */ +int pci_status_get_and_clear_errors(struct pci_dev *pdev) +{ + u16 status; + int ret; + + ret = pci_read_config_word(pdev, PCI_STATUS, &status); + if (ret != PCIBIOS_SUCCESSFUL) + return -EIO; + + status &= PCI_STATUS_ERROR_BITS; + if (status) + pci_write_config_word(pdev, PCI_STATUS, status); + + return status; +} +EXPORT_SYMBOL_GPL(pci_status_get_and_clear_errors); + #ifdef CONFIG_HAS_IOMEM void __iomem *pci_ioremap_bar(struct pci_dev *pdev, int bar) { |