lkml.org 
[lkml]   [2022]   [Jun]   [7]   [last100]   RSS Feed
Views: [wrap][no wrap]   [headers]  [forward] 
 
Messages in this thread
    Patch in this message
    /
    From
    Subject[PATCH 5.15 452/667] PCI: imx6: Fix PERST# start-up sequence
    Date
    From: Francesco Dolcini <francesco.dolcini@toradex.com>

    [ Upstream commit a6809941c1f17f455db2cf4ca19c6d8c8746ec25 ]

    According to the PCIe standard the PERST# signal (reset-gpio in
    fsl,imx* compatible dts) should be kept asserted for at least 100 usec
    before the PCIe refclock is stable, should be kept asserted for at
    least 100 msec after the power rails are stable and the host should wait
    at least 100 msec after it is de-asserted before accessing the
    configuration space of any attached device.

    >From PCIe CEM r2.0, sec 2.6.2

    T-PVPERL: Power stable to PERST# inactive - 100 msec
    T-PERST-CLK: REFCLK stable before PERST# inactive - 100 usec.

    >From PCIe r5.0, sec 6.6.1

    With a Downstream Port that does not support Link speeds greater than
    5.0 GT/s, software must wait a minimum of 100 ms before sending a
    Configuration Request to the device immediately below that Port.

    Failure to do so could prevent PCIe devices to be working correctly,
    and this was experienced with real devices.

    Move reset assert to imx6_pcie_assert_core_reset(), this way we ensure
    that PERST# is asserted before enabling any clock, move de-assert to the
    end of imx6_pcie_deassert_core_reset() after the clock is enabled and
    deemed stable and add a new delay of 100 msec just afterward.

    Link: https://lore.kernel.org/all/20220211152550.286821-1-francesco.dolcini@toradex.com
    Link: https://lore.kernel.org/r/20220404081509.94356-1-francesco.dolcini@toradex.com
    Fixes: bb38919ec56e ("PCI: imx6: Add support for i.MX6 PCIe controller")
    Signed-off-by: Francesco Dolcini <francesco.dolcini@toradex.com>
    Signed-off-by: Lorenzo Pieralisi <lorenzo.pieralisi@arm.com>
    Reviewed-by: Lucas Stach <l.stach@pengutronix.de>
    Acked-by: Richard Zhu <hongxing.zhu@nxp.com>
    Signed-off-by: Sasha Levin <sashal@kernel.org>
    ---
    drivers/pci/controller/dwc/pci-imx6.c | 23 ++++++++++++++---------
    1 file changed, 14 insertions(+), 9 deletions(-)

    diff --git a/drivers/pci/controller/dwc/pci-imx6.c b/drivers/pci/controller/dwc/pci-imx6.c
    index 2c05f2f7d1c0..67dbf9d88d22 100644
    --- a/drivers/pci/controller/dwc/pci-imx6.c
    +++ b/drivers/pci/controller/dwc/pci-imx6.c
    @@ -403,6 +403,11 @@ static void imx6_pcie_assert_core_reset(struct imx6_pcie *imx6_pcie)
    dev_err(dev, "failed to disable vpcie regulator: %d\n",
    ret);
    }
    +
    + /* Some boards don't have PCIe reset GPIO. */
    + if (gpio_is_valid(imx6_pcie->reset_gpio))
    + gpio_set_value_cansleep(imx6_pcie->reset_gpio,
    + imx6_pcie->gpio_active_high);
    }

    static unsigned int imx6_pcie_grp_offset(const struct imx6_pcie *imx6_pcie)
    @@ -525,15 +530,6 @@ static void imx6_pcie_deassert_core_reset(struct imx6_pcie *imx6_pcie)
    /* allow the clocks to stabilize */
    usleep_range(200, 500);

    - /* Some boards don't have PCIe reset GPIO. */
    - if (gpio_is_valid(imx6_pcie->reset_gpio)) {
    - gpio_set_value_cansleep(imx6_pcie->reset_gpio,
    - imx6_pcie->gpio_active_high);
    - msleep(100);
    - gpio_set_value_cansleep(imx6_pcie->reset_gpio,
    - !imx6_pcie->gpio_active_high);
    - }
    -
    switch (imx6_pcie->drvdata->variant) {
    case IMX8MQ:
    reset_control_deassert(imx6_pcie->pciephy_reset);
    @@ -576,6 +572,15 @@ static void imx6_pcie_deassert_core_reset(struct imx6_pcie *imx6_pcie)
    break;
    }

    + /* Some boards don't have PCIe reset GPIO. */
    + if (gpio_is_valid(imx6_pcie->reset_gpio)) {
    + msleep(100);
    + gpio_set_value_cansleep(imx6_pcie->reset_gpio,
    + !imx6_pcie->gpio_active_high);
    + /* Wait for 100ms after PERST# deassertion (PCIe r5.0, 6.6.1) */
    + msleep(100);
    + }
    +
    return;

    err_ref_clk:
    --
    2.35.1


    \
     
     \ /
      Last update: 2022-06-07 21:37    [W:2.707 / U:0.140 seconds]
    ©2003-2020 Jasper Spaans|hosted at Digital Ocean and TransIP|Read the blog|Advertise on this site