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/pci.c | |
| parent | dfb117b3e50c52c7b3416db4a4569224b8db80bb (diff) | |
| download | linux-809a3bf9f34cb6d0c0383b31b3495fa1ed3508a6.tar.gz linux-809a3bf9f34cb6d0c0383b31b3495fa1ed3508a6.tar.bz2 linux-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/pci.c')
| -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; |