summary refs log tree commit diff
path: root/drivers/pci
diff options
context:
space:
mode:
authorKamal Mostafa <kamal@canonical.com>2014-10-20 12:08:10 -0700
committerBjorn Helgaas <bhelgaas@google.com>2014-10-27 11:42:43 -0600
commitc302d35eac32bbd99ed9e3dc5b03d58073c78e86 (patch)
treed3315530254113e7f082a51b2c6709a7e9d47e80 /drivers/pci
parentf114040e3ea6e07372334ade75d1ee0775c355e1 (diff)
downloadlinux-c302d35eac32bbd99ed9e3dc5b03d58073c78e86.tar.gz
Revert duplicate "PCI: pciehp: Prevent NULL dereference during probe"
This reverts bceee4a97eb5 ("PCI: pciehp: Prevent NULL dereference during
probe") because it was accidentally applied twice:

  62e4492c3063 ("PCI: Prevent NULL dereference during pciehp probe")
  bceee4a97eb5 ("PCI: pciehp: Prevent NULL dereference during probe")

Revert the latter to dispose of the duplicated code block.

[bhelgaas: tidy changelog, drop stable tag]
Signed-off-by: Kamal Mostafa <kamal@canonical.com>
Signed-off-by: Bjorn Helgaas <bhelgaas@google.com>
CC: Andreas Noever <andreas.noever@gmail.com>
Diffstat (limited to 'drivers/pci')
-rw-r--r--drivers/pci/hotplug/pciehp_core.c7
1 files changed, 0 insertions, 7 deletions
diff --git a/drivers/pci/hotplug/pciehp_core.c b/drivers/pci/hotplug/pciehp_core.c
index 3a5e7e28b874..07aa722bb12c 100644
--- a/drivers/pci/hotplug/pciehp_core.c
+++ b/drivers/pci/hotplug/pciehp_core.c
@@ -262,13 +262,6 @@ static int pciehp_probe(struct pcie_device *dev)
 		goto err_out_none;
 	}
 
-	if (!dev->port->subordinate) {
-		/* Can happen if we run out of bus numbers during probe */
-		dev_err(&dev->device,
-			"Hotplug bridge without secondary bus, ignoring\n");
-		goto err_out_none;
-	}
-
 	ctrl = pcie_init(dev);
 	if (!ctrl) {
 		dev_err(&dev->device, "Controller initialization failed\n");