pcie-brcmstb: move the unilateral disable of CLKREQ# before link-up

If a platform has a pull-up on CLKREQ# with no endpoint control, then
this could cause link failure in the idle time between PERST# deassert
and brcm_config_clkreq().

Signed-off-by: Jonathan Bell <jonathan@raspberrypi.com>

Author: P33M

drivers/pci/controller/pcie-brcmstb.c

@@ -1644,10 +1644,6 @@ static void brcm_config_clkreq(struct brcm_pcie *pcie)
 		mode = "safe";
 	}
 
-	/* Start out assuming safe mode (both mode bits cleared) */
-	clkreq_cntl = readl(pcie->base + HARD_DEBUG(pcie));
-	clkreq_cntl &= ~PCIE_CLKREQ_MASK;
-
 	if (strcmp(mode, "no-l1ss") == 0) {
 		/*
 		 * "no-l1ss" -- Provides Clock Power Management, L0s, and
@@ -1844,6 +1840,11 @@ static int brcm_pcie_start_link(struct brcm_pcie *pcie)
 
 	brcm_pcie_stats_trigger(pcie, 0);
 
+	/* Disable CLKREQ# input prior to link-up */
+	tmp = readl(pcie->base + HARD_DEBUG(pcie));
+	tmp &= ~PCIE_CLKREQ_MASK;
+	writel(tmp, pcie->base + HARD_DEBUG(pcie));
+
 	/* Unassert the fundamental reset */
 	if (pcie->tperst_clk_ms) {
 		/*