aboutsummaryrefslogtreecommitdiffstats
diff options
context:
space:
mode:
authorKamal Mostafa <kamal@canonical.com>2014-10-20 15:08:10 -0400
committerBjorn Helgaas <bhelgaas@google.com>2014-10-27 13:42:43 -0400
commitc302d35eac32bbd99ed9e3dc5b03d58073c78e86 (patch)
treed3315530254113e7f082a51b2c6709a7e9d47e80
parentf114040e3ea6e07372334ade75d1ee0775c355e1 (diff)
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>
-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)
262 goto err_out_none; 262 goto err_out_none;
263 } 263 }
264 264
265 if (!dev->port->subordinate) {
266 /* Can happen if we run out of bus numbers during probe */
267 dev_err(&dev->device,
268 "Hotplug bridge without secondary bus, ignoring\n");
269 goto err_out_none;
270 }
271
272 ctrl = pcie_init(dev); 265 ctrl = pcie_init(dev);
273 if (!ctrl) { 266 if (!ctrl) {
274 dev_err(&dev->device, "Controller initialization failed\n"); 267 dev_err(&dev->device, "Controller initialization failed\n");