From: JeffyChen <jeffy.chen@rock-chips.com>
To: Shawn Lin <shawn.lin@rock-chips.com>,
Bjorn Helgaas <bhelgaas@google.com>,
Lorenzo Pieralisi <lorenzo.pieralisi@arm.com>
Cc: devicetree@vger.kernel.org, Heiko Stuebner <heiko@sntech.de>,
linux-pci@vger.kernel.org, linux-rockchip@lists.infradead.org,
Rob Herring <robh+dt@kernel.org>
Subject: Re: [v2,3/6] PCI: rockchip: Spilt out common function to init controller
Date: Sun, 25 Feb 2018 08:51:40 +0800 [thread overview]
Message-ID: <5A92089C.9090102@rock-chips.com> (raw)
In-Reply-To: <1519348606-243485-1-git-send-email-shawn.lin@rock-chips.com>
Hi Shawn,
tested on my chromebook kevin, with this series and enable
CONFIG_PCIE_ROCKCHIP_HOST, the pcie wifi(mrvl 8997) still works.
so, for the pcie host driver part:
Tested-by: Jeffy Chen <jeffy.chen@rock-chips.com>
On 02/23/2018 09:16 AM, Shawn Lin wrote:
> Most of the operations are used for both of host and EP driver,
> so this patch spilt them out to a new function, rockchip_pcie_init_port
> in pcie-rockchip.c and rename the original function to
> rockchip_pcie_host_init_port to avoid confusion. No functional
> changed intended.
>
> Signed-off-by: Shawn Lin <shawn.lin@rock-chips.com>
> ---
>
> Changes in v2: None
>
> drivers/pci/rockchip/pcie-rockchip-host.c | 131 ++-------------------------
> drivers/pci/rockchip/pcie-rockchip.c | 141 ++++++++++++++++++++++++++++++
> drivers/pci/rockchip/pcie-rockchip.h | 3 +
> 3 files changed, 151 insertions(+), 124 deletions(-)
>
> diff --git a/drivers/pci/rockchip/pcie-rockchip-host.c b/drivers/pci/rockchip/pcie-rockchip-host.c
> index 079949c..f5b0492 100644
> --- a/drivers/pci/rockchip/pcie-rockchip-host.c
> +++ b/drivers/pci/rockchip/pcie-rockchip-host.c
> @@ -294,134 +294,17 @@ static void rockchip_pcie_set_power_limit(struct rockchip_pcie *rockchip)
> * rockchip_pcie_init_port - Initialize hardware
> * @rockchip: PCIe port information
> */
> -static int rockchip_pcie_init_port(struct rockchip_pcie *rockchip)
> +static int rockchip_pcie_host_init_port(struct rockchip_pcie *rockchip)
> {
> struct device *dev = rockchip->dev;
> - int err, i;
> + int err, i = MAX_LANE_NUM;
> u32 status;
>
> gpiod_set_value_cansleep(rockchip->ep_gpio, 0);
>
> - err = reset_control_assert(rockchip->aclk_rst);
> - if (err) {
> - dev_err(dev, "assert aclk_rst err %d\n", err);
> - return err;
> - }
> -
> - err = reset_control_assert(rockchip->pclk_rst);
> - if (err) {
> - dev_err(dev, "assert pclk_rst err %d\n", err);
> - return err;
> - }
> -
> - err = reset_control_assert(rockchip->pm_rst);
> - if (err) {
> - dev_err(dev, "assert pm_rst err %d\n", err);
> + err = rockchip_pcie_init_port(rockchip);
> + if (err)
> return err;
> - }
> -
> - for (i = 0; i < MAX_LANE_NUM; i++) {
> - err = phy_init(rockchip->phys[i]);
> - if (err) {
> - dev_err(dev, "init phy%d err %d\n", i, err);
> - goto err_exit_phy;
> - }
> - }
> -
> - err = reset_control_assert(rockchip->core_rst);
> - if (err) {
> - dev_err(dev, "assert core_rst err %d\n", err);
> - goto err_exit_phy;
> - }
> -
> - err = reset_control_assert(rockchip->mgmt_rst);
> - if (err) {
> - dev_err(dev, "assert mgmt_rst err %d\n", err);
> - goto err_exit_phy;
> - }
> -
> - err = reset_control_assert(rockchip->mgmt_sticky_rst);
> - if (err) {
> - dev_err(dev, "assert mgmt_sticky_rst err %d\n", err);
> - goto err_exit_phy;
> - }
> -
> - err = reset_control_assert(rockchip->pipe_rst);
> - if (err) {
> - dev_err(dev, "assert pipe_rst err %d\n", err);
> - goto err_exit_phy;
> - }
> -
> - udelay(10);
> -
> - err = reset_control_deassert(rockchip->pm_rst);
> - if (err) {
> - dev_err(dev, "deassert pm_rst err %d\n", err);
> - goto err_exit_phy;
> - }
> -
> - err = reset_control_deassert(rockchip->aclk_rst);
> - if (err) {
> - dev_err(dev, "deassert aclk_rst err %d\n", err);
> - goto err_exit_phy;
> - }
> -
> - err = reset_control_deassert(rockchip->pclk_rst);
> - if (err) {
> - dev_err(dev, "deassert pclk_rst err %d\n", err);
> - goto err_exit_phy;
> - }
> -
> - if (rockchip->link_gen == 2)
> - rockchip_pcie_write(rockchip, PCIE_CLIENT_GEN_SEL_2,
> - PCIE_CLIENT_CONFIG);
> - else
> - rockchip_pcie_write(rockchip, PCIE_CLIENT_GEN_SEL_1,
> - PCIE_CLIENT_CONFIG);
> -
> - rockchip_pcie_write(rockchip,
> - PCIE_CLIENT_CONF_ENABLE |
> - PCIE_CLIENT_LINK_TRAIN_ENABLE |
> - PCIE_CLIENT_ARI_ENABLE |
> - PCIE_CLIENT_CONF_LANE_NUM(rockchip->lanes) |
> - PCIE_CLIENT_MODE_RC,
> - PCIE_CLIENT_CONFIG);
> -
> - for (i = 0; i < MAX_LANE_NUM; i++) {
> - err = phy_power_on(rockchip->phys[i]);
> - if (err) {
> - dev_err(dev, "power on phy%d err %d\n", i, err);
> - goto err_power_off_phy;
> - }
> - }
> -
> - /*
> - * Please don't reorder the deassert sequence of the following
> - * four reset pins.
> - */
> - err = reset_control_deassert(rockchip->mgmt_sticky_rst);
> - if (err) {
> - dev_err(dev, "deassert mgmt_sticky_rst err %d\n", err);
> - goto err_power_off_phy;
> - }
> -
> - err = reset_control_deassert(rockchip->core_rst);
> - if (err) {
> - dev_err(dev, "deassert core_rst err %d\n", err);
> - goto err_power_off_phy;
> - }
> -
> - err = reset_control_deassert(rockchip->mgmt_rst);
> - if (err) {
> - dev_err(dev, "deassert mgmt_rst err %d\n", err);
> - goto err_power_off_phy;
> - }
> -
> - err = reset_control_deassert(rockchip->pipe_rst);
> - if (err) {
> - dev_err(dev, "deassert pipe_rst err %d\n", err);
> - goto err_power_off_phy;
> - }
>
> /* Fix the transmitted FTS count desired to exit from L0s. */
> status = rockchip_pcie_read(rockchip, PCIE_CORE_CTRL_PLC1);
> @@ -514,8 +397,8 @@ static int rockchip_pcie_init_port(struct rockchip_pcie *rockchip)
> err_power_off_phy:
> while (i--)
> phy_power_off(rockchip->phys[i]);
> +
> i = MAX_LANE_NUM;
> -err_exit_phy:
> while (i--)
> phy_exit(rockchip->phys[i]);
> return err;
> @@ -1034,7 +917,7 @@ static int __maybe_unused rockchip_pcie_resume_noirq(struct device *dev)
> if (err)
> goto err_disable_0v9;
>
> - err = rockchip_pcie_init_port(rockchip);
> + err = rockchip_pcie_host_init_port(rockchip);
> if (err)
> goto err_pcie_resume;
>
> @@ -1099,7 +982,7 @@ static int rockchip_pcie_probe(struct platform_device *pdev)
> goto err_set_vpcie;
> }
>
> - err = rockchip_pcie_init_port(rockchip);
> + err = rockchip_pcie_host_init_port(rockchip);
> if (err)
> goto err_vpcie;
>
> diff --git a/drivers/pci/rockchip/pcie-rockchip.c b/drivers/pci/rockchip/pcie-rockchip.c
> index 6c0063d..d03508c 100644
> --- a/drivers/pci/rockchip/pcie-rockchip.c
> +++ b/drivers/pci/rockchip/pcie-rockchip.c
> @@ -12,6 +12,7 @@
> */
>
> #include <linux/clk.h>
> +#include <linux/delay.h>
> #include <linux/gpio/consumer.h>
> #include <linux/of_pci.h>
> #include <linux/phy/phy.h>
> @@ -144,6 +145,146 @@ int rockchip_pcie_parse_dt(struct rockchip_pcie *rockchip)
> return 0;
> }
>
> +int rockchip_pcie_init_port(struct rockchip_pcie *rockchip)
> +{
> + struct device *dev = rockchip->dev;
> + int err, i;
> + u32 regs;
> +
> + err = reset_control_assert(rockchip->aclk_rst);
> + if (err) {
> + dev_err(dev, "assert aclk_rst err %d\n", err);
> + return err;
> + }
> +
> + err = reset_control_assert(rockchip->pclk_rst);
> + if (err) {
> + dev_err(dev, "assert pclk_rst err %d\n", err);
> + return err;
> + }
> +
> + err = reset_control_assert(rockchip->pm_rst);
> + if (err) {
> + dev_err(dev, "assert pm_rst err %d\n", err);
> + return err;
> + }
> +
> + for (i = 0; i < MAX_LANE_NUM; i++) {
> + err = phy_init(rockchip->phys[i]);
> + if (err) {
> + dev_err(dev, "init phy%d err %d\n", i, err);
> + goto err_exit_phy;
> + }
> + }
> +
> + err = reset_control_assert(rockchip->core_rst);
> + if (err) {
> + dev_err(dev, "assert core_rst err %d\n", err);
> + goto err_exit_phy;
> + }
> +
> + err = reset_control_assert(rockchip->mgmt_rst);
> + if (err) {
> + dev_err(dev, "assert mgmt_rst err %d\n", err);
> + goto err_exit_phy;
> + }
> +
> + err = reset_control_assert(rockchip->mgmt_sticky_rst);
> + if (err) {
> + dev_err(dev, "assert mgmt_sticky_rst err %d\n", err);
> + goto err_exit_phy;
> + }
> +
> + err = reset_control_assert(rockchip->pipe_rst);
> + if (err) {
> + dev_err(dev, "assert pipe_rst err %d\n", err);
> + goto err_exit_phy;
> + }
> +
> + udelay(10);
> +
> + err = reset_control_deassert(rockchip->pm_rst);
> + if (err) {
> + dev_err(dev, "deassert pm_rst err %d\n", err);
> + goto err_exit_phy;
> + }
> +
> + err = reset_control_deassert(rockchip->aclk_rst);
> + if (err) {
> + dev_err(dev, "deassert aclk_rst err %d\n", err);
> + goto err_exit_phy;
> + }
> +
> + err = reset_control_deassert(rockchip->pclk_rst);
> + if (err) {
> + dev_err(dev, "deassert pclk_rst err %d\n", err);
> + goto err_exit_phy;
> + }
> +
> + if (rockchip->link_gen == 2)
> + rockchip_pcie_write(rockchip, PCIE_CLIENT_GEN_SEL_2,
> + PCIE_CLIENT_CONFIG);
> + else
> + rockchip_pcie_write(rockchip, PCIE_CLIENT_GEN_SEL_1,
> + PCIE_CLIENT_CONFIG);
> +
> + regs = PCIE_CLIENT_LINK_TRAIN_ENABLE | PCIE_CLIENT_ARI_ENABLE |
> + PCIE_CLIENT_CONF_LANE_NUM(rockchip->lanes);
> +
> + if (rockchip->is_rc)
> + regs |= PCIE_CLIENT_CONF_ENABLE | PCIE_CLIENT_MODE_RC;
> + else
> + regs |= PCIE_CLIENT_CONF_DISABLE | PCIE_CLIENT_MODE_EP;
> +
> + rockchip_pcie_write(rockchip, regs, PCIE_CLIENT_CONFIG);
> +
> + for (i = 0; i < MAX_LANE_NUM; i++) {
> + err = phy_power_on(rockchip->phys[i]);
> + if (err) {
> + dev_err(dev, "power on phy%d err %d\n", i, err);
> + goto err_power_off_phy;
> + }
> + }
> +
> + /*
> + * Please don't reorder the deassert sequence of the following
> + * four reset pins.
> + */
> + err = reset_control_deassert(rockchip->mgmt_sticky_rst);
> + if (err) {
> + dev_err(dev, "deassert mgmt_sticky_rst err %d\n", err);
> + goto err_power_off_phy;
> + }
> +
> + err = reset_control_deassert(rockchip->core_rst);
> + if (err) {
> + dev_err(dev, "deassert core_rst err %d\n", err);
> + goto err_power_off_phy;
> + }
> +
> + err = reset_control_deassert(rockchip->mgmt_rst);
> + if (err) {
> + dev_err(dev, "deassert mgmt_rst err %d\n", err);
> + goto err_power_off_phy;
> + }
> +
> + err = reset_control_deassert(rockchip->pipe_rst);
> + if (err) {
> + dev_err(dev, "deassert pipe_rst err %d\n", err);
> + goto err_power_off_phy;
> + }
> +
> + return 0;
> +err_power_off_phy:
> + while (i--)
> + phy_power_off(rockchip->phys[i]);
> + i = MAX_LANE_NUM;
> +err_exit_phy:
> + while (i--)
> + phy_exit(rockchip->phys[i]);
> + return err;
> +}
> +
> int rockchip_pcie_get_phys(struct rockchip_pcie *rockchip)
> {
> struct device *dev = rockchip->dev;
> diff --git a/drivers/pci/rockchip/pcie-rockchip.h b/drivers/pci/rockchip/pcie-rockchip.h
> index c0f84d9..af3e74f 100644
> --- a/drivers/pci/rockchip/pcie-rockchip.h
> +++ b/drivers/pci/rockchip/pcie-rockchip.h
> @@ -27,10 +27,12 @@
> #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_CONF_DISABLE HIWORD_UPDATE(0x0001, 0)
> #define PCIE_CLIENT_LINK_TRAIN_ENABLE HIWORD_UPDATE_BIT(0x0002)
> #define PCIE_CLIENT_ARI_ENABLE HIWORD_UPDATE_BIT(0x0008)
> #define PCIE_CLIENT_CONF_LANE_NUM(x) HIWORD_UPDATE(0x0030, ENCODE_LANES(x))
> #define PCIE_CLIENT_MODE_RC HIWORD_UPDATE_BIT(0x0040)
> +#define PCIE_CLIENT_MODE_EP HIWORD_UPDATE(0x0040, 0)
> #define PCIE_CLIENT_GEN_SEL_1 HIWORD_UPDATE(0x0080, 0)
> #define PCIE_CLIENT_GEN_SEL_2 HIWORD_UPDATE_BIT(0x0080)
> #define PCIE_CLIENT_DEBUG_OUT_0 (PCIE_CLIENT_BASE + 0x3c)
> @@ -238,6 +240,7 @@ static void rockchip_pcie_write(struct rockchip_pcie *rockchip, u32 val,
> }
>
> int rockchip_pcie_parse_dt(struct rockchip_pcie *rockchip);
> +int rockchip_pcie_init_port(struct rockchip_pcie *rockchip);
> int rockchip_pcie_get_phys(struct rockchip_pcie *rockchip);
> void rockchip_pcie_deinit_phys(struct rockchip_pcie *rockchip);
> int rockchip_pcie_enable_clocks(struct rockchip_pcie *rockchip);
>
WARNING: multiple messages have this Message-ID (diff)
From: JeffyChen <jeffy.chen-TNX95d0MmH7DzftRWevZcw@public.gmane.org>
To: Shawn Lin <shawn.lin-TNX95d0MmH7DzftRWevZcw@public.gmane.org>,
Bjorn Helgaas <bhelgaas-hpIqsD4AKlfQT0dZR+AlfA@public.gmane.org>,
Lorenzo Pieralisi
<lorenzo.pieralisi-5wv7dgnIgG8@public.gmane.org>
Cc: devicetree-u79uwXL29TY76Z2rM5mHXA@public.gmane.org,
linux-rockchip-IAPFreCvJWM7uuMidbF8XUB+6BGkLq7r@public.gmane.org,
Rob Herring <robh+dt-DgEjT+Ai2ygdnm+yROfE0A@public.gmane.org>,
Heiko Stuebner <heiko-4mtYJXux2i+zQB+pC5nmwQ@public.gmane.org>,
linux-pci-u79uwXL29TY76Z2rM5mHXA@public.gmane.org
Subject: Re: [v2, 3/6] PCI: rockchip: Spilt out common function to init controller
Date: Sun, 25 Feb 2018 08:51:40 +0800 [thread overview]
Message-ID: <5A92089C.9090102@rock-chips.com> (raw)
In-Reply-To: <1519348606-243485-1-git-send-email-shawn.lin-TNX95d0MmH7DzftRWevZcw@public.gmane.org>
Hi Shawn,
tested on my chromebook kevin, with this series and enable
CONFIG_PCIE_ROCKCHIP_HOST, the pcie wifi(mrvl 8997) still works.
so, for the pcie host driver part:
Tested-by: Jeffy Chen <jeffy.chen-TNX95d0MmH7DzftRWevZcw@public.gmane.org>
On 02/23/2018 09:16 AM, Shawn Lin wrote:
> Most of the operations are used for both of host and EP driver,
> so this patch spilt them out to a new function, rockchip_pcie_init_port
> in pcie-rockchip.c and rename the original function to
> rockchip_pcie_host_init_port to avoid confusion. No functional
> changed intended.
>
> Signed-off-by: Shawn Lin <shawn.lin-TNX95d0MmH7DzftRWevZcw@public.gmane.org>
> ---
>
> Changes in v2: None
>
> drivers/pci/rockchip/pcie-rockchip-host.c | 131 ++-------------------------
> drivers/pci/rockchip/pcie-rockchip.c | 141 ++++++++++++++++++++++++++++++
> drivers/pci/rockchip/pcie-rockchip.h | 3 +
> 3 files changed, 151 insertions(+), 124 deletions(-)
>
> diff --git a/drivers/pci/rockchip/pcie-rockchip-host.c b/drivers/pci/rockchip/pcie-rockchip-host.c
> index 079949c..f5b0492 100644
> --- a/drivers/pci/rockchip/pcie-rockchip-host.c
> +++ b/drivers/pci/rockchip/pcie-rockchip-host.c
> @@ -294,134 +294,17 @@ static void rockchip_pcie_set_power_limit(struct rockchip_pcie *rockchip)
> * rockchip_pcie_init_port - Initialize hardware
> * @rockchip: PCIe port information
> */
> -static int rockchip_pcie_init_port(struct rockchip_pcie *rockchip)
> +static int rockchip_pcie_host_init_port(struct rockchip_pcie *rockchip)
> {
> struct device *dev = rockchip->dev;
> - int err, i;
> + int err, i = MAX_LANE_NUM;
> u32 status;
>
> gpiod_set_value_cansleep(rockchip->ep_gpio, 0);
>
> - err = reset_control_assert(rockchip->aclk_rst);
> - if (err) {
> - dev_err(dev, "assert aclk_rst err %d\n", err);
> - return err;
> - }
> -
> - err = reset_control_assert(rockchip->pclk_rst);
> - if (err) {
> - dev_err(dev, "assert pclk_rst err %d\n", err);
> - return err;
> - }
> -
> - err = reset_control_assert(rockchip->pm_rst);
> - if (err) {
> - dev_err(dev, "assert pm_rst err %d\n", err);
> + err = rockchip_pcie_init_port(rockchip);
> + if (err)
> return err;
> - }
> -
> - for (i = 0; i < MAX_LANE_NUM; i++) {
> - err = phy_init(rockchip->phys[i]);
> - if (err) {
> - dev_err(dev, "init phy%d err %d\n", i, err);
> - goto err_exit_phy;
> - }
> - }
> -
> - err = reset_control_assert(rockchip->core_rst);
> - if (err) {
> - dev_err(dev, "assert core_rst err %d\n", err);
> - goto err_exit_phy;
> - }
> -
> - err = reset_control_assert(rockchip->mgmt_rst);
> - if (err) {
> - dev_err(dev, "assert mgmt_rst err %d\n", err);
> - goto err_exit_phy;
> - }
> -
> - err = reset_control_assert(rockchip->mgmt_sticky_rst);
> - if (err) {
> - dev_err(dev, "assert mgmt_sticky_rst err %d\n", err);
> - goto err_exit_phy;
> - }
> -
> - err = reset_control_assert(rockchip->pipe_rst);
> - if (err) {
> - dev_err(dev, "assert pipe_rst err %d\n", err);
> - goto err_exit_phy;
> - }
> -
> - udelay(10);
> -
> - err = reset_control_deassert(rockchip->pm_rst);
> - if (err) {
> - dev_err(dev, "deassert pm_rst err %d\n", err);
> - goto err_exit_phy;
> - }
> -
> - err = reset_control_deassert(rockchip->aclk_rst);
> - if (err) {
> - dev_err(dev, "deassert aclk_rst err %d\n", err);
> - goto err_exit_phy;
> - }
> -
> - err = reset_control_deassert(rockchip->pclk_rst);
> - if (err) {
> - dev_err(dev, "deassert pclk_rst err %d\n", err);
> - goto err_exit_phy;
> - }
> -
> - if (rockchip->link_gen == 2)
> - rockchip_pcie_write(rockchip, PCIE_CLIENT_GEN_SEL_2,
> - PCIE_CLIENT_CONFIG);
> - else
> - rockchip_pcie_write(rockchip, PCIE_CLIENT_GEN_SEL_1,
> - PCIE_CLIENT_CONFIG);
> -
> - rockchip_pcie_write(rockchip,
> - PCIE_CLIENT_CONF_ENABLE |
> - PCIE_CLIENT_LINK_TRAIN_ENABLE |
> - PCIE_CLIENT_ARI_ENABLE |
> - PCIE_CLIENT_CONF_LANE_NUM(rockchip->lanes) |
> - PCIE_CLIENT_MODE_RC,
> - PCIE_CLIENT_CONFIG);
> -
> - for (i = 0; i < MAX_LANE_NUM; i++) {
> - err = phy_power_on(rockchip->phys[i]);
> - if (err) {
> - dev_err(dev, "power on phy%d err %d\n", i, err);
> - goto err_power_off_phy;
> - }
> - }
> -
> - /*
> - * Please don't reorder the deassert sequence of the following
> - * four reset pins.
> - */
> - err = reset_control_deassert(rockchip->mgmt_sticky_rst);
> - if (err) {
> - dev_err(dev, "deassert mgmt_sticky_rst err %d\n", err);
> - goto err_power_off_phy;
> - }
> -
> - err = reset_control_deassert(rockchip->core_rst);
> - if (err) {
> - dev_err(dev, "deassert core_rst err %d\n", err);
> - goto err_power_off_phy;
> - }
> -
> - err = reset_control_deassert(rockchip->mgmt_rst);
> - if (err) {
> - dev_err(dev, "deassert mgmt_rst err %d\n", err);
> - goto err_power_off_phy;
> - }
> -
> - err = reset_control_deassert(rockchip->pipe_rst);
> - if (err) {
> - dev_err(dev, "deassert pipe_rst err %d\n", err);
> - goto err_power_off_phy;
> - }
>
> /* Fix the transmitted FTS count desired to exit from L0s. */
> status = rockchip_pcie_read(rockchip, PCIE_CORE_CTRL_PLC1);
> @@ -514,8 +397,8 @@ static int rockchip_pcie_init_port(struct rockchip_pcie *rockchip)
> err_power_off_phy:
> while (i--)
> phy_power_off(rockchip->phys[i]);
> +
> i = MAX_LANE_NUM;
> -err_exit_phy:
> while (i--)
> phy_exit(rockchip->phys[i]);
> return err;
> @@ -1034,7 +917,7 @@ static int __maybe_unused rockchip_pcie_resume_noirq(struct device *dev)
> if (err)
> goto err_disable_0v9;
>
> - err = rockchip_pcie_init_port(rockchip);
> + err = rockchip_pcie_host_init_port(rockchip);
> if (err)
> goto err_pcie_resume;
>
> @@ -1099,7 +982,7 @@ static int rockchip_pcie_probe(struct platform_device *pdev)
> goto err_set_vpcie;
> }
>
> - err = rockchip_pcie_init_port(rockchip);
> + err = rockchip_pcie_host_init_port(rockchip);
> if (err)
> goto err_vpcie;
>
> diff --git a/drivers/pci/rockchip/pcie-rockchip.c b/drivers/pci/rockchip/pcie-rockchip.c
> index 6c0063d..d03508c 100644
> --- a/drivers/pci/rockchip/pcie-rockchip.c
> +++ b/drivers/pci/rockchip/pcie-rockchip.c
> @@ -12,6 +12,7 @@
> */
>
> #include <linux/clk.h>
> +#include <linux/delay.h>
> #include <linux/gpio/consumer.h>
> #include <linux/of_pci.h>
> #include <linux/phy/phy.h>
> @@ -144,6 +145,146 @@ int rockchip_pcie_parse_dt(struct rockchip_pcie *rockchip)
> return 0;
> }
>
> +int rockchip_pcie_init_port(struct rockchip_pcie *rockchip)
> +{
> + struct device *dev = rockchip->dev;
> + int err, i;
> + u32 regs;
> +
> + err = reset_control_assert(rockchip->aclk_rst);
> + if (err) {
> + dev_err(dev, "assert aclk_rst err %d\n", err);
> + return err;
> + }
> +
> + err = reset_control_assert(rockchip->pclk_rst);
> + if (err) {
> + dev_err(dev, "assert pclk_rst err %d\n", err);
> + return err;
> + }
> +
> + err = reset_control_assert(rockchip->pm_rst);
> + if (err) {
> + dev_err(dev, "assert pm_rst err %d\n", err);
> + return err;
> + }
> +
> + for (i = 0; i < MAX_LANE_NUM; i++) {
> + err = phy_init(rockchip->phys[i]);
> + if (err) {
> + dev_err(dev, "init phy%d err %d\n", i, err);
> + goto err_exit_phy;
> + }
> + }
> +
> + err = reset_control_assert(rockchip->core_rst);
> + if (err) {
> + dev_err(dev, "assert core_rst err %d\n", err);
> + goto err_exit_phy;
> + }
> +
> + err = reset_control_assert(rockchip->mgmt_rst);
> + if (err) {
> + dev_err(dev, "assert mgmt_rst err %d\n", err);
> + goto err_exit_phy;
> + }
> +
> + err = reset_control_assert(rockchip->mgmt_sticky_rst);
> + if (err) {
> + dev_err(dev, "assert mgmt_sticky_rst err %d\n", err);
> + goto err_exit_phy;
> + }
> +
> + err = reset_control_assert(rockchip->pipe_rst);
> + if (err) {
> + dev_err(dev, "assert pipe_rst err %d\n", err);
> + goto err_exit_phy;
> + }
> +
> + udelay(10);
> +
> + err = reset_control_deassert(rockchip->pm_rst);
> + if (err) {
> + dev_err(dev, "deassert pm_rst err %d\n", err);
> + goto err_exit_phy;
> + }
> +
> + err = reset_control_deassert(rockchip->aclk_rst);
> + if (err) {
> + dev_err(dev, "deassert aclk_rst err %d\n", err);
> + goto err_exit_phy;
> + }
> +
> + err = reset_control_deassert(rockchip->pclk_rst);
> + if (err) {
> + dev_err(dev, "deassert pclk_rst err %d\n", err);
> + goto err_exit_phy;
> + }
> +
> + if (rockchip->link_gen == 2)
> + rockchip_pcie_write(rockchip, PCIE_CLIENT_GEN_SEL_2,
> + PCIE_CLIENT_CONFIG);
> + else
> + rockchip_pcie_write(rockchip, PCIE_CLIENT_GEN_SEL_1,
> + PCIE_CLIENT_CONFIG);
> +
> + regs = PCIE_CLIENT_LINK_TRAIN_ENABLE | PCIE_CLIENT_ARI_ENABLE |
> + PCIE_CLIENT_CONF_LANE_NUM(rockchip->lanes);
> +
> + if (rockchip->is_rc)
> + regs |= PCIE_CLIENT_CONF_ENABLE | PCIE_CLIENT_MODE_RC;
> + else
> + regs |= PCIE_CLIENT_CONF_DISABLE | PCIE_CLIENT_MODE_EP;
> +
> + rockchip_pcie_write(rockchip, regs, PCIE_CLIENT_CONFIG);
> +
> + for (i = 0; i < MAX_LANE_NUM; i++) {
> + err = phy_power_on(rockchip->phys[i]);
> + if (err) {
> + dev_err(dev, "power on phy%d err %d\n", i, err);
> + goto err_power_off_phy;
> + }
> + }
> +
> + /*
> + * Please don't reorder the deassert sequence of the following
> + * four reset pins.
> + */
> + err = reset_control_deassert(rockchip->mgmt_sticky_rst);
> + if (err) {
> + dev_err(dev, "deassert mgmt_sticky_rst err %d\n", err);
> + goto err_power_off_phy;
> + }
> +
> + err = reset_control_deassert(rockchip->core_rst);
> + if (err) {
> + dev_err(dev, "deassert core_rst err %d\n", err);
> + goto err_power_off_phy;
> + }
> +
> + err = reset_control_deassert(rockchip->mgmt_rst);
> + if (err) {
> + dev_err(dev, "deassert mgmt_rst err %d\n", err);
> + goto err_power_off_phy;
> + }
> +
> + err = reset_control_deassert(rockchip->pipe_rst);
> + if (err) {
> + dev_err(dev, "deassert pipe_rst err %d\n", err);
> + goto err_power_off_phy;
> + }
> +
> + return 0;
> +err_power_off_phy:
> + while (i--)
> + phy_power_off(rockchip->phys[i]);
> + i = MAX_LANE_NUM;
> +err_exit_phy:
> + while (i--)
> + phy_exit(rockchip->phys[i]);
> + return err;
> +}
> +
> int rockchip_pcie_get_phys(struct rockchip_pcie *rockchip)
> {
> struct device *dev = rockchip->dev;
> diff --git a/drivers/pci/rockchip/pcie-rockchip.h b/drivers/pci/rockchip/pcie-rockchip.h
> index c0f84d9..af3e74f 100644
> --- a/drivers/pci/rockchip/pcie-rockchip.h
> +++ b/drivers/pci/rockchip/pcie-rockchip.h
> @@ -27,10 +27,12 @@
> #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_CONF_DISABLE HIWORD_UPDATE(0x0001, 0)
> #define PCIE_CLIENT_LINK_TRAIN_ENABLE HIWORD_UPDATE_BIT(0x0002)
> #define PCIE_CLIENT_ARI_ENABLE HIWORD_UPDATE_BIT(0x0008)
> #define PCIE_CLIENT_CONF_LANE_NUM(x) HIWORD_UPDATE(0x0030, ENCODE_LANES(x))
> #define PCIE_CLIENT_MODE_RC HIWORD_UPDATE_BIT(0x0040)
> +#define PCIE_CLIENT_MODE_EP HIWORD_UPDATE(0x0040, 0)
> #define PCIE_CLIENT_GEN_SEL_1 HIWORD_UPDATE(0x0080, 0)
> #define PCIE_CLIENT_GEN_SEL_2 HIWORD_UPDATE_BIT(0x0080)
> #define PCIE_CLIENT_DEBUG_OUT_0 (PCIE_CLIENT_BASE + 0x3c)
> @@ -238,6 +240,7 @@ static void rockchip_pcie_write(struct rockchip_pcie *rockchip, u32 val,
> }
>
> int rockchip_pcie_parse_dt(struct rockchip_pcie *rockchip);
> +int rockchip_pcie_init_port(struct rockchip_pcie *rockchip);
> int rockchip_pcie_get_phys(struct rockchip_pcie *rockchip);
> void rockchip_pcie_deinit_phys(struct rockchip_pcie *rockchip);
> int rockchip_pcie_enable_clocks(struct rockchip_pcie *rockchip);
>
next prev parent reply other threads:[~2018-02-25 0:58 UTC|newest]
Thread overview: 30+ messages / expand[flat|nested] mbox.gz Atom feed top
2018-02-23 1:15 [PATCH v2 0/6] Add endpoint driver for Rockchip PCIe controller Shawn Lin
2018-02-23 1:15 ` Shawn Lin
2018-02-23 1:15 ` [PATCH v2 1/6] PCI: rockchip: Create individual folder for rockchip drivers Shawn Lin
2018-02-23 1:15 ` Shawn Lin
2018-02-23 1:16 ` [PATCH v2 2/6] PCI: rockchip: Split out common function to parse DT Shawn Lin
2018-02-23 1:16 ` Shawn Lin
2018-02-23 1:16 ` [PATCH v2 3/6] PCI: rockchip: Spilt out common function to init controller Shawn Lin
2018-02-23 1:16 ` Shawn Lin
2018-02-25 0:51 ` JeffyChen [this message]
2018-02-25 0:51 ` [v2, " JeffyChen
2018-02-27 11:29 ` [PATCH v2 " Lorenzo Pieralisi
2018-02-27 11:29 ` Lorenzo Pieralisi
2018-02-23 1:17 ` [PATCH v2 4/6] dt-bindings: PCI: rockchip: Rename rockchip-pcie.txt to rockchip-pcie-host.txt Shawn Lin
2018-02-23 1:17 ` Shawn Lin
2018-03-02 15:10 ` Rob Herring
2018-03-02 15:10 ` Rob Herring
2018-02-23 1:17 ` [PATCH v2 5/6] PCI: rockchip: Add Endpoint controller driver for Rockchip PCIe controller Shawn Lin
2018-02-23 1:17 ` Shawn Lin
2018-02-27 15:32 ` Lorenzo Pieralisi
2018-02-27 15:32 ` Lorenzo Pieralisi
2018-02-28 1:40 ` Shawn Lin
2018-02-28 1:40 ` Shawn Lin
2018-02-28 11:22 ` Lorenzo Pieralisi
2018-02-28 11:22 ` Lorenzo Pieralisi
2018-02-28 12:11 ` Lorenzo Pieralisi
2018-02-28 12:11 ` Lorenzo Pieralisi
2018-03-01 3:45 ` Shawn Lin
2018-03-01 3:45 ` Shawn Lin
2018-02-23 1:17 ` [PATCH v2 6/6] dt-bindings: PCI: rockchip: Add DT bindings for Rockchip PCIe endpoint controller Shawn Lin
2018-02-23 1:17 ` Shawn Lin
Reply instructions:
You may reply publicly to this message via plain-text email
using any one of the following methods:
* Save the following mbox file, import it into your mail client,
and reply-to-all from there: mbox
Avoid top-posting and favor interleaved quoting:
https://en.wikipedia.org/wiki/Posting_style#Interleaved_style
* Reply using the --to, --cc, and --in-reply-to
switches of git-send-email(1):
git send-email \
--in-reply-to=5A92089C.9090102@rock-chips.com \
--to=jeffy.chen@rock-chips.com \
--cc=bhelgaas@google.com \
--cc=devicetree@vger.kernel.org \
--cc=heiko@sntech.de \
--cc=linux-pci@vger.kernel.org \
--cc=linux-rockchip@lists.infradead.org \
--cc=lorenzo.pieralisi@arm.com \
--cc=robh+dt@kernel.org \
--cc=shawn.lin@rock-chips.com \
/path/to/YOUR_REPLY
https://kernel.org/pub/software/scm/git/docs/git-send-email.html
* If your mail client supports setting the In-Reply-To header
via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line
before the message body.
This is an external index of several public inboxes,
see mirroring instructions on how to clone and mirror
all data and code used by this external index.