PCI: Remove unnecessary check for pcie_get_mps() failure
authorYijing Wang <wangyijing@huawei.com>
Thu, 22 Aug 2013 03:24:45 +0000 (11:24 +0800)
committerBjorn Helgaas <bhelgaas@google.com>
Thu, 22 Aug 2013 16:47:03 +0000 (10:47 -0600)
After 59875ae489 ("PCI/core: Use PCI Express Capability accessors"),
pcie_get_mps() never returns an error, so don't bother to check for it.

No functional change.

[bhelgaas: changelog, fix pcie_get_mps() doc]
Signed-off-by: Yijing Wang <wangyijing@huawei.com>
Signed-off-by: Bjorn Helgaas <bhelgaas@google.com>
drivers/pci/pci.c

index e37fea6..5bb97ee 100644 (file)
@@ -3525,8 +3525,6 @@ int pcie_set_readrq(struct pci_dev *dev, int rq)
        if (pcie_bus_config == PCIE_BUS_PERFORMANCE) {
                int mps = pcie_get_mps(dev);
 
-               if (mps < 0)
-                       return mps;
                if (mps < rq)
                        rq = mps;
        }
@@ -3543,7 +3541,6 @@ EXPORT_SYMBOL(pcie_set_readrq);
  * @dev: PCI device to query
  *
  * Returns maximum payload size in bytes
- *    or appropriate error value.
  */
 int pcie_get_mps(struct pci_dev *dev)
 {