Browse Source

PCI: mvebu: Split port parsing and resource claiming from port setup

Split the PCIe port DT parsing and resource claiming from setting up the
actual ports.  This allows us to gather all the resources first, before
touching the hardware.  This is important as some of these resources (such
as the GPIO for the PCIe reset) may defer probing.

Tested-by: Willy Tarreau <w@1wt.eu> (Iomega iConnect Kirkwood, MiraBox Armada 370)
Tested-by: Andrew Lunn <andrew@lunn.ch> (D-Link DIR664 Kirkwood)
Tested-by: Thomas Petazzoni <thomas.petazzoni@free-electrons.com> (Armada XP GP)
Signed-off-by: Russell King <rmk+kernel@arm.linux.org.uk>
Signed-off-by: Bjorn Helgaas <bhelgaas@google.com>
Reviewed-by: Thomas Petazzoni <thomas.petazzoni@free-electrons.com>
Russell King 10 years ago
parent
commit
3884d846a4
1 changed files with 12 additions and 3 deletions
  1. 12 3
      drivers/pci/host/pci-mvebu.c

+ 12 - 3
drivers/pci/host/pci-mvebu.c

@@ -1093,6 +1093,18 @@ static int mvebu_pcie_probe(struct platform_device *pdev)
 			continue;
 			continue;
 		}
 		}
 
 
+		port->dn = child;
+		i++;
+	}
+	pcie->nports = i;
+
+	for (i = 0; i < pcie->nports; i++) {
+		struct mvebu_pcie_port *port = &pcie->ports[i];
+
+		child = port->dn;
+		if (!child)
+			continue;
+
 		if (gpio_is_valid(port->reset_gpio)) {
 		if (gpio_is_valid(port->reset_gpio)) {
 			u32 reset_udelay = 20000;
 			u32 reset_udelay = 20000;
 
 
@@ -1118,10 +1130,7 @@ static int mvebu_pcie_probe(struct platform_device *pdev)
 		}
 		}
 
 
 		mvebu_pcie_set_local_dev_nr(port, 1);
 		mvebu_pcie_set_local_dev_nr(port, 1);
-
-		port->dn = child;
 		mvebu_sw_pci_bridge_init(port);
 		mvebu_sw_pci_bridge_init(port);
-		i++;
 	}
 	}
 
 
 	pcie->nports = i;
 	pcie->nports = i;