Make use of the 'warm reset' register to ensure every peace of
hardware (core, phy, endpoint card) are in a known state before doing
the hardware setup.

The Aardvark IP can trigger a reset signal upon hot reset or link
failure that will only reach the components on the board without
affecting the entire device (eg. only the endpoint card). This line is
multiplexed on MPPs so if it is not used as PCI reset and multiplexed
for instance as a GPIO, the signals produced by the PCIe IP during the
warm reset operation won't affect the state of the line.

As usual, hardware designers can implement a card reset wired to a
GPIO. Support for such reset GPIO will be added in another patch.

Signed-off-by: Miquel Raynal <miquel.ray...@bootlin.com>
---
 drivers/pci/controller/pci-aardvark.c | 17 +++++++++++++++++
 1 file changed, 17 insertions(+)

diff --git a/drivers/pci/controller/pci-aardvark.c 
b/drivers/pci/controller/pci-aardvark.c
index 27ec79defa57..cfe48e553bca 100644
--- a/drivers/pci/controller/pci-aardvark.c
+++ b/drivers/pci/controller/pci-aardvark.c
@@ -139,6 +139,10 @@
 #define     CTRL_MODE_MASK                     0x1
 #define     PCIE_CORE_MODE_DIRECT              0x0
 #define     PCIE_CORE_MODE_COMMAND             0x1
+#define CTRL_WARM_RESET_REG                    (CTRL_CORE_BASE_ADDR + 0x4)
+#define     CTRL_PCIE_CORE_WARM_RESET          BIT(0)
+#define     CTRL_PHY_CORE_WARM_RESET           BIT(1)
+#define     CTRL_PERSTN_GPIO_EN                        BIT(3)
 
 /* PCIe Central Interrupts Registers */
 #define CENTRAL_INT_BASE_ADDR                  0x1b000
@@ -249,6 +253,19 @@ static void advk_pcie_setup_hw(struct advk_pcie *pcie)
 {
        u32 reg;
 
+       /* Warm reset */
+       reg = advk_readl(pcie, CTRL_WARM_RESET_REG);
+       reg |= CTRL_PCIE_CORE_WARM_RESET | CTRL_PHY_CORE_WARM_RESET |
+              CTRL_PERSTN_GPIO_EN;
+       advk_writel(pcie, reg, CTRL_WARM_RESET_REG);
+       reg = advk_readl(pcie, CTRL_WARM_RESET_REG);
+       mdelay(1);
+       reg &= ~(CTRL_PCIE_CORE_WARM_RESET | CTRL_PHY_CORE_WARM_RESET |
+                CTRL_PERSTN_GPIO_EN);
+       advk_writel(pcie, reg, CTRL_WARM_RESET_REG);
+       reg = advk_readl(pcie, CTRL_WARM_RESET_REG);
+       mdelay(10);
+
        /* Set HW Reference Clock Buffer Control */
        advk_writel(pcie, PCIE_PHY_REFCLK_BUF_CTRL, PCIE_PHY_REFCLK);
 
-- 
2.19.1

Reply via email to