[3/3] pci: rockchip: Drop legacy PHY driver

Message ID 20200709181103.89870-4-jagan@amarulasolutions.com
State New
Headers show
Series
  • phy: rockchip: Add PCIe phy driver
Related show

Commit Message

Jagan Teki July 9, 2020, 6:11 p.m. UTC
Drop the legacy PHY driver and it's associated code since
the PHY handling driver now part of Generic PHY framework.

Signed-off-by: Jagan Teki <jagan@amarulasolutions.com>
---
 drivers/pci/Makefile            |   2 +-
 drivers/pci/pcie_rockchip.c     |  75 +++++++++++-
 drivers/pci/pcie_rockchip.h     | 146 -----------------------
 drivers/pci/pcie_rockchip_phy.c | 205 --------------------------------
 4 files changed, 74 insertions(+), 354 deletions(-)
 delete mode 100644 drivers/pci/pcie_rockchip.h
 delete mode 100644 drivers/pci/pcie_rockchip_phy.c

Comments

Kever Yang July 18, 2020, noon UTC | #1
On 2020/7/10 上午2:11, Jagan Teki wrote:
> Drop the legacy PHY driver and it's associated code since
> the PHY handling driver now part of Generic PHY framework.
>
> Signed-off-by: Jagan Teki <jagan@amarulasolutions.com>
Reviewed-by: Kever Yang<kever.yang@rock-chips.com>

Thanks,
- Kever

> ---
>   drivers/pci/Makefile            |   2 +-
>   drivers/pci/pcie_rockchip.c     |  75 +++++++++++-
>   drivers/pci/pcie_rockchip.h     | 146 -----------------------
>   drivers/pci/pcie_rockchip_phy.c | 205 --------------------------------
>   4 files changed, 74 insertions(+), 354 deletions(-)
>   delete mode 100644 drivers/pci/pcie_rockchip.h
>   delete mode 100644 drivers/pci/pcie_rockchip_phy.c
>
> diff --git a/drivers/pci/Makefile b/drivers/pci/Makefile
> index 955351c5c2..493e9354dd 100644
> --- a/drivers/pci/Makefile
> +++ b/drivers/pci/Makefile
> @@ -43,4 +43,4 @@ obj-$(CONFIG_PCI_PHYTIUM) += pcie_phytium.o
>   obj-$(CONFIG_PCIE_INTEL_FPGA) += pcie_intel_fpga.o
>   obj-$(CONFIG_PCI_KEYSTONE) += pcie_dw_ti.o
>   obj-$(CONFIG_PCIE_MEDIATEK) += pcie_mediatek.o
> -obj-$(CONFIG_PCIE_ROCKCHIP) += pcie_rockchip.o pcie_rockchip_phy.o
> +obj-$(CONFIG_PCIE_ROCKCHIP) += pcie_rockchip.o
> diff --git a/drivers/pci/pcie_rockchip.c b/drivers/pci/pcie_rockchip.c
> index ce573aa4b4..04609f1296 100644
> --- a/drivers/pci/pcie_rockchip.c
> +++ b/drivers/pci/pcie_rockchip.c
> @@ -15,6 +15,7 @@
>   #include <clk.h>
>   #include <dm.h>
>   #include <dm/device_compat.h>
> +#include <generic-phy.h>
>   #include <pci.h>
>   #include <power-domain.h>
>   #include <power/regulator.h>
> @@ -25,10 +26,80 @@
>   #include <asm/arch-rockchip/clock.h>
>   #include <linux/iopoll.h>
>   
> -#include "pcie_rockchip.h"
> -
>   DECLARE_GLOBAL_DATA_PTR;
>   
> +#define HIWORD_UPDATE(mask, val)        (((mask) << 16) | (val))
> +#define HIWORD_UPDATE_BIT(val)          HIWORD_UPDATE(val, val)
> +
> +#define ENCODE_LANES(x)                 ((((x) >> 1) & 3) << 4)
> +#define PCIE_CLIENT_BASE                0x0
> +#define PCIE_CLIENT_CONFIG              (PCIE_CLIENT_BASE + 0x00)
> +#define PCIE_CLIENT_CONF_ENABLE         HIWORD_UPDATE_BIT(0x0001)
> +#define PCIE_CLIENT_LINK_TRAIN_ENABLE   HIWORD_UPDATE_BIT(0x0002)
> +#define PCIE_CLIENT_MODE_RC             HIWORD_UPDATE_BIT(0x0040)
> +#define PCIE_CLIENT_GEN_SEL_1           HIWORD_UPDATE(0x0080, 0)
> +#define PCIE_CLIENT_BASIC_STATUS1	0x0048
> +#define PCIE_CLIENT_LINK_STATUS_UP	GENMASK(21, 20)
> +#define PCIE_CLIENT_LINK_STATUS_MASK	GENMASK(21, 20)
> +#define PCIE_LINK_UP(x) \
> +	(((x) & PCIE_CLIENT_LINK_STATUS_MASK) == PCIE_CLIENT_LINK_STATUS_UP)
> +#define PCIE_RC_NORMAL_BASE		0x800000
> +#define PCIE_LM_BASE			0x900000
> +#define PCIE_LM_VENDOR_ID              (PCIE_LM_BASE + 0x44)
> +#define PCIE_LM_VENDOR_ROCKCHIP		0x1d87
> +#define PCIE_LM_RCBAR			(PCIE_LM_BASE + 0x300)
> +#define PCIE_LM_RCBARPIE		BIT(19)
> +#define PCIE_LM_RCBARPIS		BIT(20)
> +#define PCIE_RC_BASE			0xa00000
> +#define PCIE_RC_CONFIG_DCR		(PCIE_RC_BASE + 0x0c4)
> +#define PCIE_RC_CONFIG_DCR_CSPL_SHIFT	18
> +#define PCIE_RC_CONFIG_DCR_CPLS_SHIFT	26
> +#define PCIE_RC_PCIE_LCAP		(PCIE_RC_BASE + 0x0cc)
> +#define PCIE_RC_PCIE_LCAP_APMS_L0S	BIT(10)
> +#define PCIE_ATR_BASE			0xc00000
> +#define PCIE_ATR_OB_ADDR0(i)		(PCIE_ATR_BASE + 0x000 + (i) * 0x20)
> +#define PCIE_ATR_OB_ADDR1(i)		(PCIE_ATR_BASE + 0x004 + (i) * 0x20)
> +#define PCIE_ATR_OB_DESC0(i)		(PCIE_ATR_BASE + 0x008 + (i) * 0x20)
> +#define PCIE_ATR_OB_DESC1(i)		(PCIE_ATR_BASE + 0x00c + (i) * 0x20)
> +#define PCIE_ATR_IB_ADDR0(i)		(PCIE_ATR_BASE + 0x800 + (i) * 0x8)
> +#define PCIE_ATR_IB_ADDR1(i)		(PCIE_ATR_BASE + 0x804 + (i) * 0x8)
> +#define PCIE_ATR_HDR_MEM		0x2
> +#define PCIE_ATR_HDR_IO			0x6
> +#define PCIE_ATR_HDR_CFG_TYPE0		0xa
> +#define PCIE_ATR_HDR_CFG_TYPE1		0xb
> +#define PCIE_ATR_HDR_RID		BIT(23)
> +
> +#define PCIE_ATR_OB_REGION0_SIZE	(32 * 1024 * 1024)
> +#define PCIE_ATR_OB_REGION_SIZE		(1 * 1024 * 1024)
> +
> +struct rockchip_pcie {
> +	fdt_addr_t axi_base;
> +	fdt_addr_t apb_base;
> +	int first_busno;
> +	struct udevice *dev;
> +
> +	/* resets */
> +	struct reset_ctl core_rst;
> +	struct reset_ctl mgmt_rst;
> +	struct reset_ctl mgmt_sticky_rst;
> +	struct reset_ctl pipe_rst;
> +	struct reset_ctl pm_rst;
> +	struct reset_ctl pclk_rst;
> +	struct reset_ctl aclk_rst;
> +
> +	/* gpio */
> +	struct gpio_desc ep_gpio;
> +
> +	/* vpcie regulators */
> +	struct udevice *vpcie12v;
> +	struct udevice *vpcie3v3;
> +	struct udevice *vpcie1v8;
> +	struct udevice *vpcie0v9;
> +
> +	/* phy */
> +	struct phy pcie_phy;
> +};
> +
>   static int rockchip_pcie_off_conf(pci_dev_t bdf, uint offset)
>   {
>   	unsigned int bus = PCI_BUS(bdf);
> diff --git a/drivers/pci/pcie_rockchip.h b/drivers/pci/pcie_rockchip.h
> deleted file mode 100644
> index 6d20d5232d..0000000000
> --- a/drivers/pci/pcie_rockchip.h
> +++ /dev/null
> @@ -1,146 +0,0 @@
> -/* SPDX-License-Identifier: GPL-2.0+ */
> -/*
> - * Rockchip PCIe Headers
> - *
> - * Copyright (c) 2016 Rockchip, Inc.
> - * Copyright (c) 2020 Amarula Solutions(India)
> - * Copyright (c) 2020 Jagan Teki <jagan@amarulasolutions.com>
> - * Copyright (c) 2019 Patrick Wildt <patrick@blueri.se>
> - *
> - */
> -
> -#include <generic-phy.h>
> -
> -#define HIWORD_UPDATE(mask, val)        (((mask) << 16) | (val))
> -#define HIWORD_UPDATE_BIT(val)          HIWORD_UPDATE(val, val)
> -
> -#define ENCODE_LANES(x)                 ((((x) >> 1) & 3) << 4)
> -#define PCIE_CLIENT_BASE                0x0
> -#define PCIE_CLIENT_CONFIG              (PCIE_CLIENT_BASE + 0x00)
> -#define PCIE_CLIENT_CONF_ENABLE         HIWORD_UPDATE_BIT(0x0001)
> -#define PCIE_CLIENT_LINK_TRAIN_ENABLE   HIWORD_UPDATE_BIT(0x0002)
> -#define PCIE_CLIENT_MODE_RC             HIWORD_UPDATE_BIT(0x0040)
> -#define PCIE_CLIENT_GEN_SEL_1           HIWORD_UPDATE(0x0080, 0)
> -#define PCIE_CLIENT_BASIC_STATUS1	0x0048
> -#define PCIE_CLIENT_LINK_STATUS_UP	GENMASK(21, 20)
> -#define PCIE_CLIENT_LINK_STATUS_MASK	GENMASK(21, 20)
> -#define PCIE_LINK_UP(x) \
> -	(((x) & PCIE_CLIENT_LINK_STATUS_MASK) == PCIE_CLIENT_LINK_STATUS_UP)
> -#define PCIE_RC_NORMAL_BASE		0x800000
> -#define PCIE_LM_BASE			0x900000
> -#define PCIE_LM_VENDOR_ID              (PCIE_LM_BASE + 0x44)
> -#define PCIE_LM_VENDOR_ROCKCHIP		0x1d87
> -#define PCIE_LM_RCBAR			(PCIE_LM_BASE + 0x300)
> -#define PCIE_LM_RCBARPIE		BIT(19)
> -#define PCIE_LM_RCBARPIS		BIT(20)
> -#define PCIE_RC_BASE			0xa00000
> -#define PCIE_RC_CONFIG_DCR		(PCIE_RC_BASE + 0x0c4)
> -#define PCIE_RC_CONFIG_DCR_CSPL_SHIFT	18
> -#define PCIE_RC_CONFIG_DCR_CPLS_SHIFT	26
> -#define PCIE_RC_PCIE_LCAP		(PCIE_RC_BASE + 0x0cc)
> -#define PCIE_RC_PCIE_LCAP_APMS_L0S	BIT(10)
> -#define PCIE_ATR_BASE			0xc00000
> -#define PCIE_ATR_OB_ADDR0(i)		(PCIE_ATR_BASE + 0x000 + (i) * 0x20)
> -#define PCIE_ATR_OB_ADDR1(i)		(PCIE_ATR_BASE + 0x004 + (i) * 0x20)
> -#define PCIE_ATR_OB_DESC0(i)		(PCIE_ATR_BASE + 0x008 + (i) * 0x20)
> -#define PCIE_ATR_OB_DESC1(i)		(PCIE_ATR_BASE + 0x00c + (i) * 0x20)
> -#define PCIE_ATR_IB_ADDR0(i)		(PCIE_ATR_BASE + 0x800 + (i) * 0x8)
> -#define PCIE_ATR_IB_ADDR1(i)		(PCIE_ATR_BASE + 0x804 + (i) * 0x8)
> -#define PCIE_ATR_HDR_MEM		0x2
> -#define PCIE_ATR_HDR_IO			0x6
> -#define PCIE_ATR_HDR_CFG_TYPE0		0xa
> -#define PCIE_ATR_HDR_CFG_TYPE1		0xb
> -#define PCIE_ATR_HDR_RID		BIT(23)
> -
> -#define PCIE_ATR_OB_REGION0_SIZE	(32 * 1024 * 1024)
> -#define PCIE_ATR_OB_REGION_SIZE		(1 * 1024 * 1024)
> -
> -/*
> - * The higher 16-bit of this register is used for write protection
> - * only if BIT(x + 16) set to 1 the BIT(x) can be written.
> - */
> -#define HIWORD_UPDATE_MASK(val, mask, shift) \
> -		((val) << (shift) | (mask) << ((shift) + 16))
> -
> -#define PHY_CFG_DATA_SHIFT    7
> -#define PHY_CFG_ADDR_SHIFT    1
> -#define PHY_CFG_DATA_MASK     0xf
> -#define PHY_CFG_ADDR_MASK     0x3f
> -#define PHY_CFG_RD_MASK       0x3ff
> -#define PHY_CFG_WR_ENABLE     1
> -#define PHY_CFG_WR_DISABLE    1
> -#define PHY_CFG_WR_SHIFT      0
> -#define PHY_CFG_WR_MASK       1
> -#define PHY_CFG_PLL_LOCK      0x10
> -#define PHY_CFG_CLK_TEST      0x10
> -#define PHY_CFG_CLK_SCC       0x12
> -#define PHY_CFG_SEPE_RATE     BIT(3)
> -#define PHY_CFG_PLL_100M      BIT(3)
> -#define PHY_PLL_LOCKED        BIT(9)
> -#define PHY_PLL_OUTPUT        BIT(10)
> -#define PHY_LANE_IDLE_OFF     0x1
> -#define PHY_LANE_IDLE_MASK    0x1
> -#define PHY_LANE_IDLE_A_SHIFT 3
> -#define PHY_LANE_IDLE_B_SHIFT 4
> -#define PHY_LANE_IDLE_C_SHIFT 5
> -#define PHY_LANE_IDLE_D_SHIFT 6
> -
> -#define PCIE_PHY_CONF		0xe220
> -#define PCIE_PHY_STATUS		0xe2a4
> -#define PCIE_PHY_LANEOFF	0xe214
> -
> -struct rockchip_pcie_phy {
> -	void *reg_base;
> -	struct clk refclk;
> -	struct reset_ctl phy_rst;
> -	struct rockchip_pcie_phy_ops *ops;
> -};
> -
> -struct rockchip_pcie_phy_ops {
> -	int (*init)(struct rockchip_pcie_phy *phy);
> -	int (*exit)(struct rockchip_pcie_phy *phy);
> -	int (*power_on)(struct rockchip_pcie_phy *phy);
> -	int (*power_off)(struct rockchip_pcie_phy *phy);
> -};
> -
> -struct rockchip_pcie {
> -	fdt_addr_t axi_base;
> -	fdt_addr_t apb_base;
> -	int first_busno;
> -	struct udevice *dev;
> -	struct rockchip_pcie_phy rk_phy;
> -	struct rockchip_pcie_phy *phy;
> -
> -	/* resets */
> -	struct reset_ctl core_rst;
> -	struct reset_ctl mgmt_rst;
> -	struct reset_ctl mgmt_sticky_rst;
> -	struct reset_ctl pipe_rst;
> -	struct reset_ctl pm_rst;
> -	struct reset_ctl pclk_rst;
> -	struct reset_ctl aclk_rst;
> -
> -	/* gpio */
> -	struct gpio_desc ep_gpio;
> -
> -	/* vpcie regulators */
> -	struct udevice *vpcie12v;
> -	struct udevice *vpcie3v3;
> -	struct udevice *vpcie1v8;
> -	struct udevice *vpcie0v9;
> -
> -	/* phy */
> -	struct phy pcie_phy;
> -};
> -
> -int rockchip_pcie_phy_get(struct udevice *dev);
> -
> -static inline struct rockchip_pcie_phy *pcie_get_phy(struct rockchip_pcie *pcie)
> -{
> -	return pcie->phy;
> -}
> -
> -static inline struct rockchip_pcie_phy_ops *phy_get_ops(struct rockchip_pcie_phy *phy)
> -{
> -	return (struct rockchip_pcie_phy_ops *)phy->ops;
> -}
> diff --git a/drivers/pci/pcie_rockchip_phy.c b/drivers/pci/pcie_rockchip_phy.c
> deleted file mode 100644
> index 47f5d6c7e3..0000000000
> --- a/drivers/pci/pcie_rockchip_phy.c
> +++ /dev/null
> @@ -1,205 +0,0 @@
> -// SPDX-License-Identifier: GPL-2.0+
> -/*
> - * Rockchip PCIe PHY driver
> - *
> - * Copyright (c) 2016 Rockchip, Inc.
> - * Copyright (c) 2020 Amarula Solutions(India)
> - */
> -
> -#include <common.h>
> -#include <clk.h>
> -#include <dm.h>
> -#include <dm/device_compat.h>
> -#include <reset.h>
> -#include <syscon.h>
> -#include <asm/gpio.h>
> -#include <asm/io.h>
> -#include <linux/iopoll.h>
> -#include <asm/arch-rockchip/clock.h>
> -
> -#include "pcie_rockchip.h"
> -
> -DECLARE_GLOBAL_DATA_PTR;
> -
> -static void phy_wr_cfg(struct rockchip_pcie_phy *phy, u32 addr, u32 data)
> -{
> -	u32 reg;
> -
> -	reg = HIWORD_UPDATE_MASK(data, PHY_CFG_DATA_MASK, PHY_CFG_DATA_SHIFT);
> -	reg |= HIWORD_UPDATE_MASK(addr, PHY_CFG_ADDR_MASK, PHY_CFG_ADDR_SHIFT);
> -	writel(reg, phy->reg_base + PCIE_PHY_CONF);
> -
> -	udelay(1);
> -
> -	reg = HIWORD_UPDATE_MASK(PHY_CFG_WR_ENABLE,
> -				 PHY_CFG_WR_MASK,
> -				 PHY_CFG_WR_SHIFT);
> -	writel(reg, phy->reg_base + PCIE_PHY_CONF);
> -
> -	udelay(1);
> -
> -	reg = HIWORD_UPDATE_MASK(PHY_CFG_WR_DISABLE,
> -				 PHY_CFG_WR_MASK,
> -				 PHY_CFG_WR_SHIFT);
> -	writel(reg, phy->reg_base + PCIE_PHY_CONF);
> -}
> -
> -static int rockchip_pcie_phy_power_on(struct rockchip_pcie_phy *phy)
> -{
> -	int ret = 0;
> -	u32 reg, status;
> -
> -	ret = reset_deassert(&phy->phy_rst);
> -	if (ret) {
> -		dev_err(dev, "failed to assert phy reset\n");
> -		return ret;
> -	}
> -
> -	reg = HIWORD_UPDATE_MASK(PHY_CFG_PLL_LOCK,
> -				 PHY_CFG_ADDR_MASK,
> -				 PHY_CFG_ADDR_SHIFT);
> -	writel(reg, phy->reg_base + PCIE_PHY_CONF);
> -
> -	reg = HIWORD_UPDATE_MASK(!PHY_LANE_IDLE_OFF,
> -				 PHY_LANE_IDLE_MASK,
> -				 PHY_LANE_IDLE_A_SHIFT);
> -	writel(reg, phy->reg_base + PCIE_PHY_LANEOFF);
> -
> -	ret = -EINVAL;
> -	ret = readl_poll_sleep_timeout(phy->reg_base + PCIE_PHY_STATUS,
> -				       status,
> -				       status & PHY_PLL_LOCKED,
> -				       20 * 1000,
> -				       50);
> -	if (ret) {
> -		dev_err(&phy->dev, "pll lock timeout!\n");
> -		goto err_pll_lock;
> -	}
> -
> -	phy_wr_cfg(phy, PHY_CFG_CLK_TEST, PHY_CFG_SEPE_RATE);
> -	phy_wr_cfg(phy, PHY_CFG_CLK_SCC, PHY_CFG_PLL_100M);
> -
> -	ret = -ETIMEDOUT;
> -	ret = readl_poll_sleep_timeout(phy->reg_base + PCIE_PHY_STATUS,
> -				       status,
> -				       !(status & PHY_PLL_OUTPUT),
> -				       20 * 1000,
> -				       50);
> -	if (ret) {
> -		dev_err(&phy->dev, "pll output enable timeout!\n");
> -		goto err_pll_lock;
> -	}
> -
> -	reg = HIWORD_UPDATE_MASK(PHY_CFG_PLL_LOCK,
> -				 PHY_CFG_ADDR_MASK,
> -				 PHY_CFG_ADDR_SHIFT);
> -	writel(reg, phy->reg_base + PCIE_PHY_CONF);
> -
> -	ret = -EINVAL;
> -	ret = readl_poll_sleep_timeout(phy->reg_base + PCIE_PHY_STATUS,
> -				       status,
> -				       status & PHY_PLL_LOCKED,
> -				       20 * 1000,
> -				       50);
> -	if (ret) {
> -		dev_err(&phy->dev, "pll relock timeout!\n");
> -		goto err_pll_lock;
> -	}
> -
> -	return 0;
> -
> -err_pll_lock:
> -	reset_assert(&phy->phy_rst);
> -	return ret;
> -}
> -
> -static int rockchip_pcie_phy_power_off(struct rockchip_pcie_phy *phy)
> -{
> -	int ret;
> -	u32 reg;
> -
> -	reg = HIWORD_UPDATE_MASK(PHY_LANE_IDLE_OFF,
> -				 PHY_LANE_IDLE_MASK,
> -				 PHY_LANE_IDLE_A_SHIFT);
> -	writel(reg, phy->reg_base + PCIE_PHY_LANEOFF);
> -
> -	ret = reset_assert(&phy->phy_rst);
> -	if (ret) {
> -		dev_err(dev, "failed to assert phy reset\n");
> -		return ret;
> -	}
> -
> -	return 0;
> -}
> -
> -static int rockchip_pcie_phy_init(struct rockchip_pcie_phy *phy)
> -{
> -	int ret;
> -
> -	ret = clk_enable(&phy->refclk);
> -	if (ret) {
> -		dev_err(dev, "failed to enable refclk clock\n");
> -		return ret;
> -	}
> -
> -	ret = reset_assert(&phy->phy_rst);
> -	if (ret) {
> -		dev_err(dev, "failed to assert phy reset\n");
> -		goto err_reset;
> -	}
> -
> -	return 0;
> -
> -err_reset:
> -	clk_disable(&phy->refclk);
> -	return ret;
> -}
> -
> -static int rockchip_pcie_phy_exit(struct rockchip_pcie_phy *phy)
> -{
> -	clk_disable(&phy->refclk);
> -
> -	return 0;
> -}
> -
> -static struct rockchip_pcie_phy_ops pcie_phy_ops = {
> -	.init = rockchip_pcie_phy_init,
> -	.power_on = rockchip_pcie_phy_power_on,
> -	.power_off = rockchip_pcie_phy_power_off,
> -	.exit = rockchip_pcie_phy_exit,
> -};
> -
> -int rockchip_pcie_phy_get(struct udevice *dev)
> -{
> -	struct rockchip_pcie *priv = dev_get_priv(dev);
> -	struct rockchip_pcie_phy *phy_priv = &priv->rk_phy;
> -	ofnode phy_node;
> -	u32 phandle;
> -	int ret;
> -
> -	phandle = dev_read_u32_default(dev, "phys", 0);
> -	phy_node = ofnode_get_by_phandle(phandle);
> -	if (!ofnode_valid(phy_node)) {
> -		dev_err(dev, "failed to found pcie-phy\n");
> -		return -ENODEV;
> -	}
> -
> -	phy_priv->reg_base = syscon_get_first_range(ROCKCHIP_SYSCON_GRF);
> -
> -	ret = clk_get_by_index_nodev(phy_node, 0, &phy_priv->refclk);
> -	if (ret) {
> -		dev_err(dev, "failed to get refclk clock phandle\n");
> -		return ret;
> -	}
> -
> -	ret = reset_get_by_index_nodev(phy_node, 0, &phy_priv->phy_rst);
> -	if (ret) {
> -		dev_err(dev, "failed to get phy reset phandle\n");
> -		return ret;
> -	}
> -
> -	phy_priv->ops = &pcie_phy_ops;
> -	priv->phy = phy_priv;
> -
> -	return 0;
> -}

Patch

diff --git a/drivers/pci/Makefile b/drivers/pci/Makefile
index 955351c5c2..493e9354dd 100644
--- a/drivers/pci/Makefile
+++ b/drivers/pci/Makefile
@@ -43,4 +43,4 @@  obj-$(CONFIG_PCI_PHYTIUM) += pcie_phytium.o
 obj-$(CONFIG_PCIE_INTEL_FPGA) += pcie_intel_fpga.o
 obj-$(CONFIG_PCI_KEYSTONE) += pcie_dw_ti.o
 obj-$(CONFIG_PCIE_MEDIATEK) += pcie_mediatek.o
-obj-$(CONFIG_PCIE_ROCKCHIP) += pcie_rockchip.o pcie_rockchip_phy.o
+obj-$(CONFIG_PCIE_ROCKCHIP) += pcie_rockchip.o
diff --git a/drivers/pci/pcie_rockchip.c b/drivers/pci/pcie_rockchip.c
index ce573aa4b4..04609f1296 100644
--- a/drivers/pci/pcie_rockchip.c
+++ b/drivers/pci/pcie_rockchip.c
@@ -15,6 +15,7 @@ 
 #include <clk.h>
 #include <dm.h>
 #include <dm/device_compat.h>
+#include <generic-phy.h>
 #include <pci.h>
 #include <power-domain.h>
 #include <power/regulator.h>
@@ -25,10 +26,80 @@ 
 #include <asm/arch-rockchip/clock.h>
 #include <linux/iopoll.h>
 
-#include "pcie_rockchip.h"
-
 DECLARE_GLOBAL_DATA_PTR;
 
+#define HIWORD_UPDATE(mask, val)        (((mask) << 16) | (val))
+#define HIWORD_UPDATE_BIT(val)          HIWORD_UPDATE(val, val)
+
+#define ENCODE_LANES(x)                 ((((x) >> 1) & 3) << 4)
+#define PCIE_CLIENT_BASE                0x0
+#define PCIE_CLIENT_CONFIG              (PCIE_CLIENT_BASE + 0x00)
+#define PCIE_CLIENT_CONF_ENABLE         HIWORD_UPDATE_BIT(0x0001)
+#define PCIE_CLIENT_LINK_TRAIN_ENABLE   HIWORD_UPDATE_BIT(0x0002)
+#define PCIE_CLIENT_MODE_RC             HIWORD_UPDATE_BIT(0x0040)
+#define PCIE_CLIENT_GEN_SEL_1           HIWORD_UPDATE(0x0080, 0)
+#define PCIE_CLIENT_BASIC_STATUS1	0x0048
+#define PCIE_CLIENT_LINK_STATUS_UP	GENMASK(21, 20)
+#define PCIE_CLIENT_LINK_STATUS_MASK	GENMASK(21, 20)
+#define PCIE_LINK_UP(x) \
+	(((x) & PCIE_CLIENT_LINK_STATUS_MASK) == PCIE_CLIENT_LINK_STATUS_UP)
+#define PCIE_RC_NORMAL_BASE		0x800000
+#define PCIE_LM_BASE			0x900000
+#define PCIE_LM_VENDOR_ID              (PCIE_LM_BASE + 0x44)
+#define PCIE_LM_VENDOR_ROCKCHIP		0x1d87
+#define PCIE_LM_RCBAR			(PCIE_LM_BASE + 0x300)
+#define PCIE_LM_RCBARPIE		BIT(19)
+#define PCIE_LM_RCBARPIS		BIT(20)
+#define PCIE_RC_BASE			0xa00000
+#define PCIE_RC_CONFIG_DCR		(PCIE_RC_BASE + 0x0c4)
+#define PCIE_RC_CONFIG_DCR_CSPL_SHIFT	18
+#define PCIE_RC_CONFIG_DCR_CPLS_SHIFT	26
+#define PCIE_RC_PCIE_LCAP		(PCIE_RC_BASE + 0x0cc)
+#define PCIE_RC_PCIE_LCAP_APMS_L0S	BIT(10)
+#define PCIE_ATR_BASE			0xc00000
+#define PCIE_ATR_OB_ADDR0(i)		(PCIE_ATR_BASE + 0x000 + (i) * 0x20)
+#define PCIE_ATR_OB_ADDR1(i)		(PCIE_ATR_BASE + 0x004 + (i) * 0x20)
+#define PCIE_ATR_OB_DESC0(i)		(PCIE_ATR_BASE + 0x008 + (i) * 0x20)
+#define PCIE_ATR_OB_DESC1(i)		(PCIE_ATR_BASE + 0x00c + (i) * 0x20)
+#define PCIE_ATR_IB_ADDR0(i)		(PCIE_ATR_BASE + 0x800 + (i) * 0x8)
+#define PCIE_ATR_IB_ADDR1(i)		(PCIE_ATR_BASE + 0x804 + (i) * 0x8)
+#define PCIE_ATR_HDR_MEM		0x2
+#define PCIE_ATR_HDR_IO			0x6
+#define PCIE_ATR_HDR_CFG_TYPE0		0xa
+#define PCIE_ATR_HDR_CFG_TYPE1		0xb
+#define PCIE_ATR_HDR_RID		BIT(23)
+
+#define PCIE_ATR_OB_REGION0_SIZE	(32 * 1024 * 1024)
+#define PCIE_ATR_OB_REGION_SIZE		(1 * 1024 * 1024)
+
+struct rockchip_pcie {
+	fdt_addr_t axi_base;
+	fdt_addr_t apb_base;
+	int first_busno;
+	struct udevice *dev;
+
+	/* resets */
+	struct reset_ctl core_rst;
+	struct reset_ctl mgmt_rst;
+	struct reset_ctl mgmt_sticky_rst;
+	struct reset_ctl pipe_rst;
+	struct reset_ctl pm_rst;
+	struct reset_ctl pclk_rst;
+	struct reset_ctl aclk_rst;
+
+	/* gpio */
+	struct gpio_desc ep_gpio;
+
+	/* vpcie regulators */
+	struct udevice *vpcie12v;
+	struct udevice *vpcie3v3;
+	struct udevice *vpcie1v8;
+	struct udevice *vpcie0v9;
+
+	/* phy */
+	struct phy pcie_phy;
+};
+
 static int rockchip_pcie_off_conf(pci_dev_t bdf, uint offset)
 {
 	unsigned int bus = PCI_BUS(bdf);
diff --git a/drivers/pci/pcie_rockchip.h b/drivers/pci/pcie_rockchip.h
deleted file mode 100644
index 6d20d5232d..0000000000
--- a/drivers/pci/pcie_rockchip.h
+++ /dev/null
@@ -1,146 +0,0 @@ 
-/* SPDX-License-Identifier: GPL-2.0+ */
-/*
- * Rockchip PCIe Headers
- *
- * Copyright (c) 2016 Rockchip, Inc.
- * Copyright (c) 2020 Amarula Solutions(India)
- * Copyright (c) 2020 Jagan Teki <jagan@amarulasolutions.com>
- * Copyright (c) 2019 Patrick Wildt <patrick@blueri.se>
- *
- */
-
-#include <generic-phy.h>
-
-#define HIWORD_UPDATE(mask, val)        (((mask) << 16) | (val))
-#define HIWORD_UPDATE_BIT(val)          HIWORD_UPDATE(val, val)
-
-#define ENCODE_LANES(x)                 ((((x) >> 1) & 3) << 4)
-#define PCIE_CLIENT_BASE                0x0
-#define PCIE_CLIENT_CONFIG              (PCIE_CLIENT_BASE + 0x00)
-#define PCIE_CLIENT_CONF_ENABLE         HIWORD_UPDATE_BIT(0x0001)
-#define PCIE_CLIENT_LINK_TRAIN_ENABLE   HIWORD_UPDATE_BIT(0x0002)
-#define PCIE_CLIENT_MODE_RC             HIWORD_UPDATE_BIT(0x0040)
-#define PCIE_CLIENT_GEN_SEL_1           HIWORD_UPDATE(0x0080, 0)
-#define PCIE_CLIENT_BASIC_STATUS1	0x0048
-#define PCIE_CLIENT_LINK_STATUS_UP	GENMASK(21, 20)
-#define PCIE_CLIENT_LINK_STATUS_MASK	GENMASK(21, 20)
-#define PCIE_LINK_UP(x) \
-	(((x) & PCIE_CLIENT_LINK_STATUS_MASK) == PCIE_CLIENT_LINK_STATUS_UP)
-#define PCIE_RC_NORMAL_BASE		0x800000
-#define PCIE_LM_BASE			0x900000
-#define PCIE_LM_VENDOR_ID              (PCIE_LM_BASE + 0x44)
-#define PCIE_LM_VENDOR_ROCKCHIP		0x1d87
-#define PCIE_LM_RCBAR			(PCIE_LM_BASE + 0x300)
-#define PCIE_LM_RCBARPIE		BIT(19)
-#define PCIE_LM_RCBARPIS		BIT(20)
-#define PCIE_RC_BASE			0xa00000
-#define PCIE_RC_CONFIG_DCR		(PCIE_RC_BASE + 0x0c4)
-#define PCIE_RC_CONFIG_DCR_CSPL_SHIFT	18
-#define PCIE_RC_CONFIG_DCR_CPLS_SHIFT	26
-#define PCIE_RC_PCIE_LCAP		(PCIE_RC_BASE + 0x0cc)
-#define PCIE_RC_PCIE_LCAP_APMS_L0S	BIT(10)
-#define PCIE_ATR_BASE			0xc00000
-#define PCIE_ATR_OB_ADDR0(i)		(PCIE_ATR_BASE + 0x000 + (i) * 0x20)
-#define PCIE_ATR_OB_ADDR1(i)		(PCIE_ATR_BASE + 0x004 + (i) * 0x20)
-#define PCIE_ATR_OB_DESC0(i)		(PCIE_ATR_BASE + 0x008 + (i) * 0x20)
-#define PCIE_ATR_OB_DESC1(i)		(PCIE_ATR_BASE + 0x00c + (i) * 0x20)
-#define PCIE_ATR_IB_ADDR0(i)		(PCIE_ATR_BASE + 0x800 + (i) * 0x8)
-#define PCIE_ATR_IB_ADDR1(i)		(PCIE_ATR_BASE + 0x804 + (i) * 0x8)
-#define PCIE_ATR_HDR_MEM		0x2
-#define PCIE_ATR_HDR_IO			0x6
-#define PCIE_ATR_HDR_CFG_TYPE0		0xa
-#define PCIE_ATR_HDR_CFG_TYPE1		0xb
-#define PCIE_ATR_HDR_RID		BIT(23)
-
-#define PCIE_ATR_OB_REGION0_SIZE	(32 * 1024 * 1024)
-#define PCIE_ATR_OB_REGION_SIZE		(1 * 1024 * 1024)
-
-/*
- * The higher 16-bit of this register is used for write protection
- * only if BIT(x + 16) set to 1 the BIT(x) can be written.
- */
-#define HIWORD_UPDATE_MASK(val, mask, shift) \
-		((val) << (shift) | (mask) << ((shift) + 16))
-
-#define PHY_CFG_DATA_SHIFT    7
-#define PHY_CFG_ADDR_SHIFT    1
-#define PHY_CFG_DATA_MASK     0xf
-#define PHY_CFG_ADDR_MASK     0x3f
-#define PHY_CFG_RD_MASK       0x3ff
-#define PHY_CFG_WR_ENABLE     1
-#define PHY_CFG_WR_DISABLE    1
-#define PHY_CFG_WR_SHIFT      0
-#define PHY_CFG_WR_MASK       1
-#define PHY_CFG_PLL_LOCK      0x10
-#define PHY_CFG_CLK_TEST      0x10
-#define PHY_CFG_CLK_SCC       0x12
-#define PHY_CFG_SEPE_RATE     BIT(3)
-#define PHY_CFG_PLL_100M      BIT(3)
-#define PHY_PLL_LOCKED        BIT(9)
-#define PHY_PLL_OUTPUT        BIT(10)
-#define PHY_LANE_IDLE_OFF     0x1
-#define PHY_LANE_IDLE_MASK    0x1
-#define PHY_LANE_IDLE_A_SHIFT 3
-#define PHY_LANE_IDLE_B_SHIFT 4
-#define PHY_LANE_IDLE_C_SHIFT 5
-#define PHY_LANE_IDLE_D_SHIFT 6
-
-#define PCIE_PHY_CONF		0xe220
-#define PCIE_PHY_STATUS		0xe2a4
-#define PCIE_PHY_LANEOFF	0xe214
-
-struct rockchip_pcie_phy {
-	void *reg_base;
-	struct clk refclk;
-	struct reset_ctl phy_rst;
-	struct rockchip_pcie_phy_ops *ops;
-};
-
-struct rockchip_pcie_phy_ops {
-	int (*init)(struct rockchip_pcie_phy *phy);
-	int (*exit)(struct rockchip_pcie_phy *phy);
-	int (*power_on)(struct rockchip_pcie_phy *phy);
-	int (*power_off)(struct rockchip_pcie_phy *phy);
-};
-
-struct rockchip_pcie {
-	fdt_addr_t axi_base;
-	fdt_addr_t apb_base;
-	int first_busno;
-	struct udevice *dev;
-	struct rockchip_pcie_phy rk_phy;
-	struct rockchip_pcie_phy *phy;
-
-	/* resets */
-	struct reset_ctl core_rst;
-	struct reset_ctl mgmt_rst;
-	struct reset_ctl mgmt_sticky_rst;
-	struct reset_ctl pipe_rst;
-	struct reset_ctl pm_rst;
-	struct reset_ctl pclk_rst;
-	struct reset_ctl aclk_rst;
-
-	/* gpio */
-	struct gpio_desc ep_gpio;
-
-	/* vpcie regulators */
-	struct udevice *vpcie12v;
-	struct udevice *vpcie3v3;
-	struct udevice *vpcie1v8;
-	struct udevice *vpcie0v9;
-
-	/* phy */
-	struct phy pcie_phy;
-};
-
-int rockchip_pcie_phy_get(struct udevice *dev);
-
-static inline struct rockchip_pcie_phy *pcie_get_phy(struct rockchip_pcie *pcie)
-{
-	return pcie->phy;
-}
-
-static inline struct rockchip_pcie_phy_ops *phy_get_ops(struct rockchip_pcie_phy *phy)
-{
-	return (struct rockchip_pcie_phy_ops *)phy->ops;
-}
diff --git a/drivers/pci/pcie_rockchip_phy.c b/drivers/pci/pcie_rockchip_phy.c
deleted file mode 100644
index 47f5d6c7e3..0000000000
--- a/drivers/pci/pcie_rockchip_phy.c
+++ /dev/null
@@ -1,205 +0,0 @@ 
-// SPDX-License-Identifier: GPL-2.0+
-/*
- * Rockchip PCIe PHY driver
- *
- * Copyright (c) 2016 Rockchip, Inc.
- * Copyright (c) 2020 Amarula Solutions(India)
- */
-
-#include <common.h>
-#include <clk.h>
-#include <dm.h>
-#include <dm/device_compat.h>
-#include <reset.h>
-#include <syscon.h>
-#include <asm/gpio.h>
-#include <asm/io.h>
-#include <linux/iopoll.h>
-#include <asm/arch-rockchip/clock.h>
-
-#include "pcie_rockchip.h"
-
-DECLARE_GLOBAL_DATA_PTR;
-
-static void phy_wr_cfg(struct rockchip_pcie_phy *phy, u32 addr, u32 data)
-{
-	u32 reg;
-
-	reg = HIWORD_UPDATE_MASK(data, PHY_CFG_DATA_MASK, PHY_CFG_DATA_SHIFT);
-	reg |= HIWORD_UPDATE_MASK(addr, PHY_CFG_ADDR_MASK, PHY_CFG_ADDR_SHIFT);
-	writel(reg, phy->reg_base + PCIE_PHY_CONF);
-
-	udelay(1);
-
-	reg = HIWORD_UPDATE_MASK(PHY_CFG_WR_ENABLE,
-				 PHY_CFG_WR_MASK,
-				 PHY_CFG_WR_SHIFT);
-	writel(reg, phy->reg_base + PCIE_PHY_CONF);
-
-	udelay(1);
-
-	reg = HIWORD_UPDATE_MASK(PHY_CFG_WR_DISABLE,
-				 PHY_CFG_WR_MASK,
-				 PHY_CFG_WR_SHIFT);
-	writel(reg, phy->reg_base + PCIE_PHY_CONF);
-}
-
-static int rockchip_pcie_phy_power_on(struct rockchip_pcie_phy *phy)
-{
-	int ret = 0;
-	u32 reg, status;
-
-	ret = reset_deassert(&phy->phy_rst);
-	if (ret) {
-		dev_err(dev, "failed to assert phy reset\n");
-		return ret;
-	}
-
-	reg = HIWORD_UPDATE_MASK(PHY_CFG_PLL_LOCK,
-				 PHY_CFG_ADDR_MASK,
-				 PHY_CFG_ADDR_SHIFT);
-	writel(reg, phy->reg_base + PCIE_PHY_CONF);
-
-	reg = HIWORD_UPDATE_MASK(!PHY_LANE_IDLE_OFF,
-				 PHY_LANE_IDLE_MASK,
-				 PHY_LANE_IDLE_A_SHIFT);
-	writel(reg, phy->reg_base + PCIE_PHY_LANEOFF);
-
-	ret = -EINVAL;
-	ret = readl_poll_sleep_timeout(phy->reg_base + PCIE_PHY_STATUS,
-				       status,
-				       status & PHY_PLL_LOCKED,
-				       20 * 1000,
-				       50);
-	if (ret) {
-		dev_err(&phy->dev, "pll lock timeout!\n");
-		goto err_pll_lock;
-	}
-
-	phy_wr_cfg(phy, PHY_CFG_CLK_TEST, PHY_CFG_SEPE_RATE);
-	phy_wr_cfg(phy, PHY_CFG_CLK_SCC, PHY_CFG_PLL_100M);
-
-	ret = -ETIMEDOUT;
-	ret = readl_poll_sleep_timeout(phy->reg_base + PCIE_PHY_STATUS,
-				       status,
-				       !(status & PHY_PLL_OUTPUT),
-				       20 * 1000,
-				       50);
-	if (ret) {
-		dev_err(&phy->dev, "pll output enable timeout!\n");
-		goto err_pll_lock;
-	}
-
-	reg = HIWORD_UPDATE_MASK(PHY_CFG_PLL_LOCK,
-				 PHY_CFG_ADDR_MASK,
-				 PHY_CFG_ADDR_SHIFT);
-	writel(reg, phy->reg_base + PCIE_PHY_CONF);
-
-	ret = -EINVAL;
-	ret = readl_poll_sleep_timeout(phy->reg_base + PCIE_PHY_STATUS,
-				       status,
-				       status & PHY_PLL_LOCKED,
-				       20 * 1000,
-				       50);
-	if (ret) {
-		dev_err(&phy->dev, "pll relock timeout!\n");
-		goto err_pll_lock;
-	}
-
-	return 0;
-
-err_pll_lock:
-	reset_assert(&phy->phy_rst);
-	return ret;
-}
-
-static int rockchip_pcie_phy_power_off(struct rockchip_pcie_phy *phy)
-{
-	int ret;
-	u32 reg;
-
-	reg = HIWORD_UPDATE_MASK(PHY_LANE_IDLE_OFF,
-				 PHY_LANE_IDLE_MASK,
-				 PHY_LANE_IDLE_A_SHIFT);
-	writel(reg, phy->reg_base + PCIE_PHY_LANEOFF);
-
-	ret = reset_assert(&phy->phy_rst);
-	if (ret) {
-		dev_err(dev, "failed to assert phy reset\n");
-		return ret;
-	}
-
-	return 0;
-}
-
-static int rockchip_pcie_phy_init(struct rockchip_pcie_phy *phy)
-{
-	int ret;
-
-	ret = clk_enable(&phy->refclk);
-	if (ret) {
-		dev_err(dev, "failed to enable refclk clock\n");
-		return ret;
-	}
-
-	ret = reset_assert(&phy->phy_rst);
-	if (ret) {
-		dev_err(dev, "failed to assert phy reset\n");
-		goto err_reset;
-	}
-
-	return 0;
-
-err_reset:
-	clk_disable(&phy->refclk);
-	return ret;
-}
-
-static int rockchip_pcie_phy_exit(struct rockchip_pcie_phy *phy)
-{
-	clk_disable(&phy->refclk);
-
-	return 0;
-}
-
-static struct rockchip_pcie_phy_ops pcie_phy_ops = {
-	.init = rockchip_pcie_phy_init,
-	.power_on = rockchip_pcie_phy_power_on,
-	.power_off = rockchip_pcie_phy_power_off,
-	.exit = rockchip_pcie_phy_exit,
-};
-
-int rockchip_pcie_phy_get(struct udevice *dev)
-{
-	struct rockchip_pcie *priv = dev_get_priv(dev);
-	struct rockchip_pcie_phy *phy_priv = &priv->rk_phy;
-	ofnode phy_node;
-	u32 phandle;
-	int ret;
-
-	phandle = dev_read_u32_default(dev, "phys", 0);
-	phy_node = ofnode_get_by_phandle(phandle);
-	if (!ofnode_valid(phy_node)) {
-		dev_err(dev, "failed to found pcie-phy\n");
-		return -ENODEV;
-	}
-
-	phy_priv->reg_base = syscon_get_first_range(ROCKCHIP_SYSCON_GRF);
-
-	ret = clk_get_by_index_nodev(phy_node, 0, &phy_priv->refclk);
-	if (ret) {
-		dev_err(dev, "failed to get refclk clock phandle\n");
-		return ret;
-	}
-
-	ret = reset_get_by_index_nodev(phy_node, 0, &phy_priv->phy_rst);
-	if (ret) {
-		dev_err(dev, "failed to get phy reset phandle\n");
-		return ret;
-	}
-
-	phy_priv->ops = &pcie_phy_ops;
-	priv->phy = phy_priv;
-
-	return 0;
-}