2023-03-14 07:32:00

by Vaishnav Achath

[permalink] [raw]
Subject: [PATCH] phy: cadence: cdns-dphy-rx: Add common module reset support

From: Sinthu Raja <[email protected]>

DPHY RX module has a common module reset (RSTB_CMN) which is expected
to be released during configuration. In J721E SR1.0 the RSTB_CMN is
internally tied to CSI_RX_RST and is hardware controlled, for all
other newer platforms the common module reset is software controlled.
Add support to control common module reset during configuration and
also skip common module reset based on soc_device_match() for J721E SR1.0.

Signed-off-by: Sinthu Raja <[email protected]>
Co-developed-by: Vaishnav Achath <[email protected]>
Signed-off-by: Vaishnav Achath <[email protected]>
---

Tested on J721E SR1.0 and SR 1.1 by CSI2RX streaming,
without this changes CSI2RX streaming fails on J721E SR1.1
and all other newer TI platforms (J721S2, J784S4, AM62X).

Logs: https://gist.github.com/vaishnavachath/3ecda7de0e63b13c6f765ae2c4f9c5ff

drivers/phy/cadence/cdns-dphy-rx.c | 32 ++++++++++++++++++++++++++++++
1 file changed, 32 insertions(+)

diff --git a/drivers/phy/cadence/cdns-dphy-rx.c b/drivers/phy/cadence/cdns-dphy-rx.c
index 572c70089a94..c05b043893a9 100644
--- a/drivers/phy/cadence/cdns-dphy-rx.c
+++ b/drivers/phy/cadence/cdns-dphy-rx.c
@@ -11,10 +11,12 @@
#include <linux/phy/phy.h>
#include <linux/phy/phy-mipi-dphy.h>
#include <linux/platform_device.h>
+#include <linux/sys_soc.h>

#define DPHY_PMA_CMN(reg) (reg)
#define DPHY_PCS(reg) (0xb00 + (reg))
#define DPHY_ISO(reg) (0xc00 + (reg))
+#define DPHY_WRAP(reg) (0x1000 + (reg))

#define DPHY_CMN_SSM DPHY_PMA_CMN(0x20)
#define DPHY_CMN_RX_MODE_EN BIT(10)
@@ -33,6 +35,9 @@
#define DPHY_POWER_ISLAND_EN_CLK DPHY_PCS(0xc)
#define DPHY_POWER_ISLAND_EN_CLK_VAL 0xaa

+#define DPHY_LANE DPHY_WRAP(0x0)
+#define DPHY_LANE_RESET_CMN_EN BIT(23)
+
#define DPHY_ISO_CL_CTRL_L DPHY_ISO(0x10)
#define DPHY_ISO_DL_CTRL_L0 DPHY_ISO(0x14)
#define DPHY_ISO_DL_CTRL_L1 DPHY_ISO(0x20)
@@ -57,6 +62,10 @@ struct cdns_dphy_rx_band {
unsigned int max_rate;
};

+struct cdns_dphy_soc_data {
+ bool has_hw_cmn_rstb;
+};
+
/* Order of bands is important since the index is the band number. */
static const struct cdns_dphy_rx_band bands[] = {
{ 80, 100 }, { 100, 120 }, { 120, 160 }, { 160, 200 }, { 200, 240 },
@@ -142,13 +151,36 @@ static int cdns_dphy_rx_wait_lane_ready(struct cdns_dphy_rx *dphy,
return 0;
}

+static struct cdns_dphy_soc_data j721e_soc_data = {
+ .has_hw_cmn_rstb = true,
+};
+
+static const struct soc_device_attribute cdns_dphy_socinfo[] = {
+ {
+ .family = "J721E",
+ .revision = "SR1.0",
+ .data = &j721e_soc_data,
+ },
+ {/* sentinel */}
+};
+
static int cdns_dphy_rx_configure(struct phy *phy,
union phy_configure_opts *opts)
{
struct cdns_dphy_rx *dphy = phy_get_drvdata(phy);
unsigned int reg, lanes = opts->mipi_dphy.lanes;
+ const struct cdns_dphy_soc_data *soc_data = NULL;
+ const struct soc_device_attribute *soc;
int band_ctrl, ret;

+ soc = soc_device_match(cdns_dphy_socinfo);
+ if (soc && soc->data)
+ soc_data = soc->data;
+ if (!soc || (soc_data && !soc_data->has_hw_cmn_rstb)) {
+ reg = DPHY_LANE_RESET_CMN_EN;
+ writel(reg, dphy->regs + DPHY_LANE);
+ }
+
/* Data lanes. Minimum one lane is mandatory. */
if (lanes < DPHY_LANES_MIN || lanes > DPHY_LANES_MAX)
return -EINVAL;
--
2.17.1



2023-03-20 06:33:55

by Jai Luthra

[permalink] [raw]
Subject: Re: [PATCH] phy: cadence: cdns-dphy-rx: Add common module reset support

Hi,

On Mar 14, 2023 at 13:01:37 +0530, Vaishnav Achath wrote:
> From: Sinthu Raja <[email protected]>
>
> DPHY RX module has a common module reset (RSTB_CMN) which is expected
> to be released during configuration. In J721E SR1.0 the RSTB_CMN is
> internally tied to CSI_RX_RST and is hardware controlled, for all
> other newer platforms the common module reset is software controlled.
> Add support to control common module reset during configuration and
> also skip common module reset based on soc_device_match() for J721E SR1.0.
>
> Signed-off-by: Sinthu Raja <[email protected]>
> Co-developed-by: Vaishnav Achath <[email protected]>
> Signed-off-by: Vaishnav Achath <[email protected]>

Reviewed-by: Jai Luthra <[email protected]>

> ---
>
> Tested on J721E SR1.0 and SR 1.1 by CSI2RX streaming,
> without this changes CSI2RX streaming fails on J721E SR1.1
> and all other newer TI platforms (J721S2, J784S4, AM62X).
>
> Logs: https://gist.github.com/vaishnavachath/3ecda7de0e63b13c6f765ae2c4f9c5ff
>
> drivers/phy/cadence/cdns-dphy-rx.c | 32 ++++++++++++++++++++++++++++++
> 1 file changed, 32 insertions(+)
>
> diff --git a/drivers/phy/cadence/cdns-dphy-rx.c b/drivers/phy/cadence/cdns-dphy-rx.c
> index 572c70089a94..c05b043893a9 100644
> --- a/drivers/phy/cadence/cdns-dphy-rx.c
> +++ b/drivers/phy/cadence/cdns-dphy-rx.c
> @@ -11,10 +11,12 @@
> #include <linux/phy/phy.h>
> #include <linux/phy/phy-mipi-dphy.h>
> #include <linux/platform_device.h>
> +#include <linux/sys_soc.h>
>
> #define DPHY_PMA_CMN(reg) (reg)
> #define DPHY_PCS(reg) (0xb00 + (reg))
> #define DPHY_ISO(reg) (0xc00 + (reg))
> +#define DPHY_WRAP(reg) (0x1000 + (reg))
>
> #define DPHY_CMN_SSM DPHY_PMA_CMN(0x20)
> #define DPHY_CMN_RX_MODE_EN BIT(10)
> @@ -33,6 +35,9 @@
> #define DPHY_POWER_ISLAND_EN_CLK DPHY_PCS(0xc)
> #define DPHY_POWER_ISLAND_EN_CLK_VAL 0xaa
>
> +#define DPHY_LANE DPHY_WRAP(0x0)
> +#define DPHY_LANE_RESET_CMN_EN BIT(23)
> +
> #define DPHY_ISO_CL_CTRL_L DPHY_ISO(0x10)
> #define DPHY_ISO_DL_CTRL_L0 DPHY_ISO(0x14)
> #define DPHY_ISO_DL_CTRL_L1 DPHY_ISO(0x20)
> @@ -57,6 +62,10 @@ struct cdns_dphy_rx_band {
> unsigned int max_rate;
> };
>
> +struct cdns_dphy_soc_data {
> + bool has_hw_cmn_rstb;
> +};
> +
> /* Order of bands is important since the index is the band number. */
> static const struct cdns_dphy_rx_band bands[] = {
> { 80, 100 }, { 100, 120 }, { 120, 160 }, { 160, 200 }, { 200, 240 },
> @@ -142,13 +151,36 @@ static int cdns_dphy_rx_wait_lane_ready(struct cdns_dphy_rx *dphy,
> return 0;
> }
>
> +static struct cdns_dphy_soc_data j721e_soc_data = {
> + .has_hw_cmn_rstb = true,
> +};
> +
> +static const struct soc_device_attribute cdns_dphy_socinfo[] = {
> + {
> + .family = "J721E",
> + .revision = "SR1.0",
> + .data = &j721e_soc_data,
> + },
> + {/* sentinel */}
> +};
> +
> static int cdns_dphy_rx_configure(struct phy *phy,
> union phy_configure_opts *opts)
> {
> struct cdns_dphy_rx *dphy = phy_get_drvdata(phy);
> unsigned int reg, lanes = opts->mipi_dphy.lanes;
> + const struct cdns_dphy_soc_data *soc_data = NULL;
> + const struct soc_device_attribute *soc;
> int band_ctrl, ret;
>
> + soc = soc_device_match(cdns_dphy_socinfo);
> + if (soc && soc->data)
> + soc_data = soc->data;
> + if (!soc || (soc_data && !soc_data->has_hw_cmn_rstb)) {
> + reg = DPHY_LANE_RESET_CMN_EN;
> + writel(reg, dphy->regs + DPHY_LANE);
> + }
> +
> /* Data lanes. Minimum one lane is mandatory. */
> if (lanes < DPHY_LANES_MIN || lanes > DPHY_LANES_MAX)
> return -EINVAL;
> --
> 2.17.1
>


Attachments:
(No filename) (3.50 kB)
signature.asc (833.00 B)
Download all attachments

2023-04-12 17:07:20

by Vinod Koul

[permalink] [raw]
Subject: Re: [PATCH] phy: cadence: cdns-dphy-rx: Add common module reset support

On 14-03-23, 13:01, Vaishnav Achath wrote:
> From: Sinthu Raja <[email protected]>
>
> DPHY RX module has a common module reset (RSTB_CMN) which is expected
> to be released during configuration. In J721E SR1.0 the RSTB_CMN is
> internally tied to CSI_RX_RST and is hardware controlled, for all
> other newer platforms the common module reset is software controlled.
> Add support to control common module reset during configuration and
> also skip common module reset based on soc_device_match() for J721E SR1.0.

Applied, thanks

--
~Vinod