diff options
| author |   Bjorn Helgaas <bhelgaas@google.com> | 2012-06-20 16:41:16 -0600 |
|---|---|---|
| committer | Bjorn Helgaas <bhelgaas@google.com> | 2012-06-20 17:28:53 -0600 |
| commit | 809a3bf9f34cb6d0c0383b31b3495fa1ed3508a6 (patch) | |
| tree | bf12b30a6c9c717cafe158d80e2df16b56004dce /drivers/pci | |
| parent | PCI: acpiphp: check whether _ADR evaluation succeeded (diff) | |
| download | linux-dev-809a3bf9f34cb6d0c0383b31b3495fa1ed3508a6.tar.xz linux-dev-809a3bf9f34cb6d0c0383b31b3495fa1ed3508a6.zip | |
PCI: remove useless pcix_set_mmrbc() dev->bus check
For a valid pci_dev, dev->bus != NULL always, so remove this
unnecessary test.
Found by Coverity (CID 101680).
Signed-off-by: Bjorn Helgaas <bhelgaas@google.com>
Diffstat (limited to 'drivers/pci')
| -rw-r--r-- | drivers/pci/pci.c | 3 |
1 files changed, 1 insertions, 2 deletions
diff --git a/drivers/pci/pci.c b/drivers/pci/pci.c index b9e93cf1eb40..7f1310e58534 100644 --- a/drivers/pci/pci.c +++ b/drivers/pci/pci.c @@ -3395,8 +3395,7 @@ int pcix_set_mmrbc(struct pci_dev *dev, int mmrbc) o = (cmd & PCI_X_CMD_MAX_READ) >> 2; if (o != v) { - if (v > o && dev->bus && - (dev->bus->bus_flags & PCI_BUS_FLAGS_NO_MMRBC)) + if (v > o && (dev->bus->bus_flags & PCI_BUS_FLAGS_NO_MMRBC)) return -EIO; cmd &= ~PCI_X_CMD_MAX_READ; |