[PATCH 2/3] pci: rockchip: Switch to generic-phy

[Kever Yang]

On 2020/7/10 上午2:11, Jagan Teki wrote:
> Now, we have a PCIe PHY driver as part of the Generic
> PHY framework. Let's use it instead of legacy PHY driver.
>
> Signed-off-by: Jagan Teki <jagan at amarulasolutions.com>
Reviewed-by: Kever Yang<kever.yang at rock-chips.com>

Thanks,
- Kever
> ---
>   drivers/pci/Kconfig         |  1 +
>   drivers/pci/pcie_rockchip.c | 20 ++++++++++----------
>   drivers/pci/pcie_rockchip.h |  5 +++++
>   3 files changed, 16 insertions(+), 10 deletions(-)
>
> diff --git a/drivers/pci/Kconfig b/drivers/pci/Kconfig
> index 6d8c22aacf..58ca673e4b 100644
> --- a/drivers/pci/Kconfig
> +++ b/drivers/pci/Kconfig
> @@ -200,6 +200,7 @@ config PCIE_MEDIATEK
>   config PCIE_ROCKCHIP
>   	bool "Enable Rockchip PCIe driver"
>   	select DM_PCI
> +	select PHY_ROCKCHIP_PCIE
>   	default y if ROCKCHIP_RK3399
>   	help
>   	  Say Y here if you want to enable PCIe controller support on
> diff --git a/drivers/pci/pcie_rockchip.c b/drivers/pci/pcie_rockchip.c
> index 0edc2464a8..ce573aa4b4 100644
> --- a/drivers/pci/pcie_rockchip.c
> +++ b/drivers/pci/pcie_rockchip.c
> @@ -159,8 +159,6 @@ static int rockchip_pcie_atr_init(struct rockchip_pcie *priv)
>   static int rockchip_pcie_init_port(struct udevice *dev)
>   {
>   	struct rockchip_pcie *priv = dev_get_priv(dev);
> -	struct rockchip_pcie_phy *phy = pcie_get_phy(priv);
> -	struct rockchip_pcie_phy_ops *ops = phy_get_ops(phy);
>   	u32 cr, val, status;
>   	int ret;
>   
> @@ -185,7 +183,7 @@ static int rockchip_pcie_init_port(struct udevice *dev)
>   		return ret;
>   	}
>   
> -	ret = ops->init(phy);
> +	ret = generic_phy_init(&priv->pcie_phy);
>   	if (ret) {
>   		dev_err(dev, "failed to init phy (ret=%d)\n", ret);
>   		goto err_exit_phy;
> @@ -242,7 +240,7 @@ static int rockchip_pcie_init_port(struct udevice *dev)
>   	cr |= PCIE_CLIENT_CONF_ENABLE | PCIE_CLIENT_MODE_RC;
>   	writel(cr, priv->apb_base + PCIE_CLIENT_CONFIG);
>   
> -	ret = ops->power_on(phy);
> +	ret = generic_phy_power_on(&priv->pcie_phy);
>   	if (ret) {
>   		dev_err(dev, "failed to power on phy (ret=%d)\n", ret);
>   		goto err_power_off_phy;
> @@ -311,9 +309,9 @@ static int rockchip_pcie_init_port(struct udevice *dev)
>   	return 0;
>   
>   err_power_off_phy:
> -	ops->power_off(phy);
> +	generic_phy_power_off(&priv->pcie_phy);
>   err_exit_phy:
> -	ops->exit(phy);
> +	generic_phy_exit(&priv->pcie_phy);
>   	return ret;
>   }
>   
> @@ -443,6 +441,12 @@ static int rockchip_pcie_parse_dt(struct udevice *dev)
>   		return ret;
>   	}
>   
> +	ret = generic_phy_get_by_index(dev, 0, &priv->pcie_phy);
> +	if (ret) {
> +		dev_err(dev, "failed to get pcie-phy (ret=%d)\n", ret);
> +		return ret;
> +	}
> +
>   	return 0;
>   }
>   
> @@ -460,10 +464,6 @@ static int rockchip_pcie_probe(struct udevice *dev)
>   	if (ret)
>   		return ret;
>   
> -	ret = rockchip_pcie_phy_get(dev);
> -	if (ret)
> -		return ret;
> -
>   	ret = rockchip_pcie_set_vpcie(dev);
>   	if (ret)
>   		return ret;
> diff --git a/drivers/pci/pcie_rockchip.h b/drivers/pci/pcie_rockchip.h
> index 845d5059e1..6d20d5232d 100644
> --- a/drivers/pci/pcie_rockchip.h
> +++ b/drivers/pci/pcie_rockchip.h
> @@ -9,6 +9,8 @@
>    *
>    */
>   
> +#include <generic-phy.h>
> +
>   #define HIWORD_UPDATE(mask, val)        (((mask) << 16) | (val))
>   #define HIWORD_UPDATE_BIT(val)          HIWORD_UPDATE(val, val)
>   
> @@ -126,6 +128,9 @@ struct rockchip_pcie {
>   	struct udevice *vpcie3v3;
>   	struct udevice *vpcie1v8;
>   	struct udevice *vpcie0v9;
> +
> +	/* phy */
> +	struct phy pcie_phy;
>   };
>   
>   int rockchip_pcie_phy_get(struct udevice *dev);