summaryrefslogtreecommitdiffstats
diff options
context:
space:
mode:
authorBjorn Helgaas <bhelgaas@google.com>2013-08-22 11:24:46 +0800
committerBjorn Helgaas <bhelgaas@google.com>2013-08-22 10:47:04 -0600
commitc2996948ac36a9082f27b9ad94dac4c821a9c33d (patch)
treeda7eacb8af784ca903787e01dd2ed4700c70654c
parentf67577118d1154154cf3c1d96f870c4da13846b4 (diff)
downloadop-kernel-dev-c2996948ac36a9082f27b9ad94dac4c821a9c33d.zip
op-kernel-dev-c2996948ac36a9082f27b9ad94dac4c821a9c33d.tar.gz
PCI: Simplify MPS test for Downstream Port
PCIe hotplug bridges are always either Root Ports or Downstream Ports. No other device type can have a PCIe link leading downstream to a slot. Root Ports don't have an upstream bridge, so "dev->is_hotplug_bridge && dev->bus->self" is true if and only if "dev" is a Downstream Port. That means we can simplify this by looking at the type of "dev" itself, without looking upstream at all. No functional change. Signed-off-by: Bjorn Helgaas <bhelgaas@google.com>
-rw-r--r--drivers/pci/probe.c3
1 files changed, 1 insertions, 2 deletions
diff --git a/drivers/pci/probe.c b/drivers/pci/probe.c
index ecae7f2..0591b08 100644
--- a/drivers/pci/probe.c
+++ b/drivers/pci/probe.c
@@ -1507,8 +1507,7 @@ static int pcie_find_smpss(struct pci_dev *dev, void *data)
* will occur as normal.
*/
if (dev->is_hotplug_bridge && (!list_is_singular(&dev->bus->devices) ||
- (dev->bus->self &&
- pci_pcie_type(dev->bus->self) != PCI_EXP_TYPE_ROOT_PORT)))
+ pci_pcie_type(dev) != PCI_EXP_TYPE_ROOT_PORT))
*smpss = 0;
if (*smpss > dev->pcie_mpss)
OpenPOWER on IntegriCloud