[<prev] [next>] [<thread-prev] [thread-next>] [day] [month] [year] [list]
Message-ID: <1625024288.20084.9.camel@mhfsdcap03>
Date: Wed, 30 Jun 2021 11:38:08 +0800
From: Qizhong Cheng <qizhong.cheng@...iatek.com>
To: Jianjun Wang <jianjun.wang@...iatek.com>
CC: Bjorn Helgaas <bhelgaas@...gle.com>,
Rob Herring <robh+dt@...nel.org>,
Lorenzo Pieralisi <lorenzo.pieralisi@....com>,
Ryder Lee <ryder.lee@...iatek.com>,
Matthias Brugger <matthias.bgg@...il.com>,
<linux-pci@...r.kernel.org>, <linux-mediatek@...ts.infradead.org>,
<devicetree@...r.kernel.org>, <linux-kernel@...r.kernel.org>,
<linux-arm-kernel@...ts.infradead.org>, <youlin.pei@...iatek.com>,
<chuanjia.liu@...iatek.com>, <ot_jieyang@...iatek.com>,
<drinkcat@...omium.org>, <Rex-BC.Chen@...iatek.com>,
Krzysztof Wilczyski <kw@...ux.com>, <Ryan-JH.Yu@...iatek.com>
Subject: Re: [PATCH v3 2/2] PCI: mediatek-gen3: Add support for disable
dvfsrc voltage request
Reviewed-by: Qizhong Cheng <qizhong.cheng@...iatek.com>
Tested-by: Qizhong Cheng <qizhong.cheng@...iatek.com>
On Wed, 2021-06-30 at 10:49 +0800, Jianjun Wang wrote:
> PCIe Gen3 PHY layer cannot work properly when the requested voltage
> is lower than a specific level(e.g. 0.55V, it's depends on
> the chip manufacturing process).
>
> When the dvfsrc feature is implemented, the requested voltage
> may be reduced to a lower level in suspend mode, hence that
> the MAC layer will assert a HW signal to request the dvfsrc
> to raise voltage to normal mode, and it will wait the voltage
> ready signal from dvfsrc to decide if the LTSSM can start normally.
>
> When the dvfsrc feature is not implemented, the MAC layer still
> assert the voltage request to dvfsrc when exit suspend mode,
> but will not receive the voltage ready signal, in this case,
> the LTSSM cannot start normally, and the PCIe link will be failed.
>
> Add support for disable dvfsrc voltage request, if the property of
> "disable-dvfsrc-vlt-req" is presented in device node, we assume that
> the requested voltage is always higher enough to keep the PCIe Gen3
> PHY active, and the voltage request to dvfsrc should be disabled.
>
> Signed-off-by: Jianjun Wang <jianjun.wang@...iatek.com>
> ---
> drivers/pci/controller/pcie-mediatek-gen3.c | 31 +++++++++++++++++++++
> 1 file changed, 31 insertions(+)
>
> diff --git a/drivers/pci/controller/pcie-mediatek-gen3.c b/drivers/pci/controller/pcie-mediatek-gen3.c
> index 3c5b97716d40..028014707588 100644
> --- a/drivers/pci/controller/pcie-mediatek-gen3.c
> +++ b/drivers/pci/controller/pcie-mediatek-gen3.c
> @@ -79,6 +79,9 @@
> #define PCIE_ICMD_PM_REG 0x198
> #define PCIE_TURN_OFF_LINK BIT(4)
>
> +#define PCIE_MISC_CTRL_REG 0x348
> +#define PCIE_DISABLE_DVFSRC_VLT_REQ BIT(1)
> +
> #define PCIE_TRANS_TABLE_BASE_REG 0x800
> #define PCIE_ATR_SRC_ADDR_MSB_OFFSET 0x4
> #define PCIE_ATR_TRSL_ADDR_LSB_OFFSET 0x8
> @@ -297,6 +300,34 @@ static int mtk_pcie_startup_port(struct mtk_pcie_port *port)
> val &= ~PCIE_INTX_ENABLE;
> writel_relaxed(val, port->base + PCIE_INT_ENABLE_REG);
>
> + /*
> + * PCIe Gen3 PHY layer can not work properly when the requested voltage
> + * is lower than a specific level(e.g. 0.55V, it's depends on
> + * the chip manufacturing process).
> + *
> + * When the dvfsrc feature is implemented, the requested voltage
> + * might be reduced to a lower level in suspend mode, hence that
> + * the MAC layer will assert a HW signal to request the dvfsrc
> + * to raise voltage to normal mode, and it will wait the voltage
> + * ready signal from dvfsrc to start the LTSSM normally.
> + *
> + * When the dvfsrc feature is not implemented, the MAC layer still
> + * assert the voltage request to dvfsrc when exit suspend mode,
> + * but will not get the voltage ready signal, in this case, the LTSSM
> + * cannot start normally, and the PCIe link will be failed.
> + *
> + * If the property of "disable-dvfsrc-vlt-req" is presented
> + * in device node, we assume that the requested voltage is always
> + * higher enough to keep the PCIe Gen3 PHY active, and the voltage
> + * request to dvfsrc should be disabled.
> + */
> + val = readl_relaxed(port->base + PCIE_MISC_CTRL_REG);
> + val &= ~PCIE_DISABLE_DVFSRC_VLT_REQ;
> + if (of_property_read_bool(port->dev->of_node, "disable-dvfsrc-vlt-req"))
> + val |= PCIE_DISABLE_DVFSRC_VLT_REQ;
> +
> + writel_relaxed(val, port->base + PCIE_MISC_CTRL_REG);
> +
> /* Assert all reset signals */
> val = readl_relaxed(port->base + PCIE_RST_CTRL_REG);
> val |= PCIE_MAC_RSTB | PCIE_PHY_RSTB | PCIE_BRG_RSTB | PCIE_PE_RSTB;
Powered by blists - more mailing lists