* [PATCH v4 0/3] Add support for dphy tx on j721e
@ 2022-06-23 12:54 Rahul T R
2022-06-23 12:54 ` [PATCH v4 1/3] phy: dt-bindings: cdns,dphy: Add compatible for dphy " Rahul T R
` (3 more replies)
0 siblings, 4 replies; 8+ messages in thread
From: Rahul T R @ 2022-06-23 12:54 UTC (permalink / raw)
To: linux-phy, kishon, vkoul, robh+dt, krzysztof.kozlowski+dt
Cc: p.yadav, tomi.valkeinen, laurent.pinchart, linux-kernel, jpawar,
sjakhade, mparab, devicetree, vigneshr, lee.jones, Rahul T R
Following series of patches adds support for dphy tx on ti's j721e
soc. new compatible is added and required cdns dphy ops are implemented.
the series also adds band ctrl configuration required for dphy tx
v4:
-Remove the additional Kconfig for DPHY_TX_J721E support
-Add comments wherever absolute values are used in the code
-Accumulated the acks
v3:
-Fixed the header files order
-Store bands as array of unsigned int, since min[n] = max[n-1]
-Remove unreachable code warning
v2:
-fix a build error reported by kernel test robot <lkp@intel.com>
which uses clang compiler. did not get the error with gnu toolchain
9.2-2019.12
Rahul T R (3):
phy: dt-bindings: cdns,dphy: Add compatible for dphy on j721e
phy: cdns-dphy: Add band config for dphy tx
phy: cdns-dphy: Add support for DPHY TX on J721e
.../devicetree/bindings/phy/cdns,dphy.yaml | 5 +-
drivers/phy/cadence/cdns-dphy.c | 101 +++++++++++++++++-
2 files changed, 103 insertions(+), 3 deletions(-)
--
2.36.1
^ permalink raw reply [flat|nested] 8+ messages in thread
* [PATCH v4 1/3] phy: dt-bindings: cdns,dphy: Add compatible for dphy on j721e
2022-06-23 12:54 [PATCH v4 0/3] Add support for dphy tx on j721e Rahul T R
@ 2022-06-23 12:54 ` Rahul T R
2022-06-23 12:54 ` [PATCH v4 2/3] phy: cdns-dphy: Add band config for dphy tx Rahul T R
` (2 subsequent siblings)
3 siblings, 0 replies; 8+ messages in thread
From: Rahul T R @ 2022-06-23 12:54 UTC (permalink / raw)
To: linux-phy, kishon, vkoul, robh+dt, krzysztof.kozlowski+dt
Cc: p.yadav, tomi.valkeinen, laurent.pinchart, linux-kernel, jpawar,
sjakhade, mparab, devicetree, vigneshr, lee.jones, Rahul T R,
Krzysztof Kozlowski
Add compatible to support dphy tx on j721e
Signed-off-by: Rahul T R <r-ravikumar@ti.com>
Reviewed-by: Laurent Pinchart <laurent.pinchart@ideasonboard.com>
Reviewed-by: Krzysztof Kozlowski <krzysztof.kozlowski@linaro.org>
---
Documentation/devicetree/bindings/phy/cdns,dphy.yaml | 5 +++--
1 file changed, 3 insertions(+), 2 deletions(-)
diff --git a/Documentation/devicetree/bindings/phy/cdns,dphy.yaml b/Documentation/devicetree/bindings/phy/cdns,dphy.yaml
index c50629bd1b51..f0e9ca8427bb 100644
--- a/Documentation/devicetree/bindings/phy/cdns,dphy.yaml
+++ b/Documentation/devicetree/bindings/phy/cdns,dphy.yaml
@@ -11,8 +11,9 @@ maintainers:
properties:
compatible:
- items:
- - const: cdns,dphy
+ enum:
+ - cdns,dphy
+ - ti,j721e-dphy
reg:
maxItems: 1
--
2.36.1
^ permalink raw reply related [flat|nested] 8+ messages in thread
* [PATCH v4 2/3] phy: cdns-dphy: Add band config for dphy tx
2022-06-23 12:54 [PATCH v4 0/3] Add support for dphy tx on j721e Rahul T R
2022-06-23 12:54 ` [PATCH v4 1/3] phy: dt-bindings: cdns,dphy: Add compatible for dphy " Rahul T R
@ 2022-06-23 12:54 ` Rahul T R
2022-06-23 12:54 ` [PATCH v4 3/3] phy: cdns-dphy: Add support for DPHY TX on J721e Rahul T R
2022-07-05 6:53 ` Vinod Koul
3 siblings, 0 replies; 8+ messages in thread
From: Rahul T R @ 2022-06-23 12:54 UTC (permalink / raw)
To: linux-phy, kishon, vkoul, robh+dt, krzysztof.kozlowski+dt
Cc: p.yadav, tomi.valkeinen, laurent.pinchart, linux-kernel, jpawar,
sjakhade, mparab, devicetree, vigneshr, lee.jones, Rahul T R
Add support for band ctrl config for dphy tx.
Signed-off-by: Rahul T R <r-ravikumar@ti.com>
Reviewed-by: Pratyush Yadav <p.yadav@ti.com>
---
drivers/phy/cadence/cdns-dphy.c | 40 ++++++++++++++++++++++++++++++++-
1 file changed, 39 insertions(+), 1 deletion(-)
diff --git a/drivers/phy/cadence/cdns-dphy.c b/drivers/phy/cadence/cdns-dphy.c
index ba042e39cfaf..14f951013b4f 100644
--- a/drivers/phy/cadence/cdns-dphy.c
+++ b/drivers/phy/cadence/cdns-dphy.c
@@ -3,6 +3,7 @@
* Copyright: 2017-2018 Cadence Design Systems, Inc.
*/
+#include <linux/bitfield.h>
#include <linux/bitops.h>
#include <linux/clk.h>
#include <linux/io.h>
@@ -45,6 +46,10 @@
#define DPHY_CMN_OPDIV_FROM_REG BIT(6)
#define DPHY_CMN_OPDIV(x) ((x) << 7)
+#define DPHY_BAND_CFG DPHY_PCS(0x0)
+#define DPHY_BAND_CFG_LEFT_BAND GENMASK(4, 0)
+#define DPHY_BAND_CFG_RIGHT_BAND GENMASK(9, 5)
+
#define DPHY_PSM_CFG DPHY_PCS(0x4)
#define DPHY_PSM_CFG_FROM_REG BIT(0)
#define DPHY_PSM_CLK_DIV(x) ((x) << 1)
@@ -92,6 +97,12 @@ struct cdns_dphy {
struct phy *phy;
};
+/* Order of bands is important since the index is the band number. */
+static const unsigned int tx_bands[] = {
+ 80, 100, 120, 160, 200, 240, 320, 390, 450, 510, 560, 640, 690, 770,
+ 870, 950, 1000, 1200, 1400, 1600, 1800, 2000, 2200, 2500
+};
+
static int cdns_dsi_get_dphy_pll_cfg(struct cdns_dphy *dphy,
struct cdns_dphy_cfg *cfg,
struct phy_configure_opts_mipi_dphy *opts,
@@ -232,6 +243,24 @@ static int cdns_dphy_config_from_opts(struct phy *phy,
return 0;
}
+static int cdns_dphy_tx_get_band_ctrl(unsigned long hs_clk_rate)
+{
+ unsigned int rate;
+ int i;
+
+ rate = hs_clk_rate / 1000000UL;
+
+ if (rate < tx_bands[0])
+ return -EOPNOTSUPP;
+
+ for (i = 0; i < ARRAY_SIZE(tx_bands) - 1; i++) {
+ if (rate >= tx_bands[i] && rate < tx_bands[i + 1])
+ return i;
+ }
+
+ return -EOPNOTSUPP;
+}
+
static int cdns_dphy_validate(struct phy *phy, enum phy_mode mode, int submode,
union phy_configure_opts *opts)
{
@@ -247,7 +276,8 @@ static int cdns_dphy_configure(struct phy *phy, union phy_configure_opts *opts)
{
struct cdns_dphy *dphy = phy_get_drvdata(phy);
struct cdns_dphy_cfg cfg = { 0 };
- int ret;
+ int ret, band_ctrl;
+ unsigned int reg;
ret = cdns_dphy_config_from_opts(phy, &opts->mipi_dphy, &cfg);
if (ret)
@@ -276,6 +306,14 @@ static int cdns_dphy_configure(struct phy *phy, union phy_configure_opts *opts)
*/
cdns_dphy_set_pll_cfg(dphy, &cfg);
+ band_ctrl = cdns_dphy_tx_get_band_ctrl(opts->mipi_dphy.hs_clk_rate);
+ if (band_ctrl < 0)
+ return band_ctrl;
+
+ reg = FIELD_PREP(DPHY_BAND_CFG_LEFT_BAND, band_ctrl) |
+ FIELD_PREP(DPHY_BAND_CFG_RIGHT_BAND, band_ctrl);
+ writel(reg, dphy->regs + DPHY_BAND_CFG);
+
return 0;
}
--
2.36.1
^ permalink raw reply related [flat|nested] 8+ messages in thread
* [PATCH v4 3/3] phy: cdns-dphy: Add support for DPHY TX on J721e
2022-06-23 12:54 [PATCH v4 0/3] Add support for dphy tx on j721e Rahul T R
2022-06-23 12:54 ` [PATCH v4 1/3] phy: dt-bindings: cdns,dphy: Add compatible for dphy " Rahul T R
2022-06-23 12:54 ` [PATCH v4 2/3] phy: cdns-dphy: Add band config for dphy tx Rahul T R
@ 2022-06-23 12:54 ` Rahul T R
2022-06-24 9:45 ` Pratyush Yadav
2022-07-05 6:53 ` Vinod Koul
3 siblings, 1 reply; 8+ messages in thread
From: Rahul T R @ 2022-06-23 12:54 UTC (permalink / raw)
To: linux-phy, kishon, vkoul, robh+dt, krzysztof.kozlowski+dt
Cc: p.yadav, tomi.valkeinen, laurent.pinchart, linux-kernel, jpawar,
sjakhade, mparab, devicetree, vigneshr, lee.jones, Rahul T R
Add support new compatible for dphy-tx on j721e
and implement dphy ops required.
Signed-off-by: Rahul T R <r-ravikumar@ti.com>
---
drivers/phy/cadence/cdns-dphy.c | 61 +++++++++++++++++++++++++++++++++
1 file changed, 61 insertions(+)
diff --git a/drivers/phy/cadence/cdns-dphy.c b/drivers/phy/cadence/cdns-dphy.c
index 14f951013b4f..3dfdfb33cd0a 100644
--- a/drivers/phy/cadence/cdns-dphy.c
+++ b/drivers/phy/cadence/cdns-dphy.c
@@ -7,6 +7,7 @@
#include <linux/bitops.h>
#include <linux/clk.h>
#include <linux/io.h>
+#include <linux/iopoll.h>
#include <linux/module.h>
#include <linux/of_address.h>
#include <linux/of_device.h>
@@ -18,6 +19,7 @@
#define REG_WAKEUP_TIME_NS 800
#define DPHY_PLL_RATE_HZ 108000000
+#define POLL_TIMEOUT_US 1000
/* DPHY registers */
#define DPHY_PMA_CMN(reg) (reg)
@@ -62,6 +64,18 @@
#define DSI_NULL_FRAME_OVERHEAD 6
#define DSI_EOT_PKT_SIZE 4
+#define DPHY_TX_J721E_WIZ_PLL_CTRL 0xF04
+#define DPHY_TX_J721E_WIZ_STATUS 0xF08
+#define DPHY_TX_J721E_WIZ_RST_CTRL 0xF0C
+#define DPHY_TX_J721E_WIZ_PSM_FREQ 0xF10
+
+#define DPHY_TX_J721E_WIZ_IPDIV GENMASK(4, 0)
+#define DPHY_TX_J721E_WIZ_OPDIV GENMASK(13, 8)
+#define DPHY_TX_J721E_WIZ_FBDIV GENMASK(25, 16)
+#define DPHY_TX_J721E_WIZ_LANE_RSTB BIT(31)
+#define DPHY_TX_WIZ_PLL_LOCK BIT(31)
+#define DPHY_TX_WIZ_O_CMN_READY BIT(31)
+
struct cdns_dphy_cfg {
u8 pll_ipdiv;
u8 pll_opdiv;
@@ -210,6 +224,46 @@ static void cdns_dphy_ref_set_psm_div(struct cdns_dphy *dphy, u8 div)
dphy->regs + DPHY_PSM_CFG);
}
+static unsigned long cdns_dphy_j721e_get_wakeup_time_ns(struct cdns_dphy *dphy)
+{
+ /* Minimum wakeup time as per MIPI D-PHY spec v1.2 */
+ return 1000000;
+}
+
+static void cdns_dphy_j721e_set_pll_cfg(struct cdns_dphy *dphy,
+ const struct cdns_dphy_cfg *cfg)
+{
+ u32 status;
+
+ /*
+ * set the PWM and PLL Byteclk divider settings to recommended values
+ * which is same as that of in ref ops
+ */
+ writel(DPHY_CMN_PWM_HIGH(6) | DPHY_CMN_PWM_LOW(0x101) |
+ DPHY_CMN_PWM_DIV(0x8),
+ dphy->regs + DPHY_CMN_PWM);
+
+ writel((FIELD_PREP(DPHY_TX_J721E_WIZ_IPDIV, cfg->pll_ipdiv) |
+ FIELD_PREP(DPHY_TX_J721E_WIZ_OPDIV, cfg->pll_opdiv) |
+ FIELD_PREP(DPHY_TX_J721E_WIZ_FBDIV, cfg->pll_fbdiv)),
+ dphy->regs + DPHY_TX_J721E_WIZ_PLL_CTRL);
+
+ writel(DPHY_TX_J721E_WIZ_LANE_RSTB,
+ dphy->regs + DPHY_TX_J721E_WIZ_RST_CTRL);
+
+ readl_poll_timeout(dphy->regs + DPHY_TX_J721E_WIZ_PLL_CTRL, status,
+ (status & DPHY_TX_WIZ_PLL_LOCK), 0, POLL_TIMEOUT_US);
+
+ readl_poll_timeout(dphy->regs + DPHY_TX_J721E_WIZ_STATUS, status,
+ (status & DPHY_TX_WIZ_O_CMN_READY), 0,
+ POLL_TIMEOUT_US);
+}
+
+static void cdns_dphy_j721e_set_psm_div(struct cdns_dphy *dphy, u8 div)
+{
+ writel(div, dphy->regs + DPHY_TX_J721E_WIZ_PSM_FREQ);
+}
+
/*
* This is the reference implementation of DPHY hooks. Specific integration of
* this IP may have to re-implement some of them depending on how they decided
@@ -221,6 +275,12 @@ static const struct cdns_dphy_ops ref_dphy_ops = {
.set_psm_div = cdns_dphy_ref_set_psm_div,
};
+static const struct cdns_dphy_ops j721e_dphy_ops = {
+ .get_wakeup_time_ns = cdns_dphy_j721e_get_wakeup_time_ns,
+ .set_pll_cfg = cdns_dphy_j721e_set_pll_cfg,
+ .set_psm_div = cdns_dphy_j721e_set_psm_div,
+};
+
static int cdns_dphy_config_from_opts(struct phy *phy,
struct phy_configure_opts_mipi_dphy *opts,
struct cdns_dphy_cfg *cfg)
@@ -408,6 +468,7 @@ static int cdns_dphy_remove(struct platform_device *pdev)
static const struct of_device_id cdns_dphy_of_match[] = {
{ .compatible = "cdns,dphy", .data = &ref_dphy_ops },
+ { .compatible = "ti,j721e-dphy", .data = &j721e_dphy_ops },
{ /* sentinel */ },
};
MODULE_DEVICE_TABLE(of, cdns_dphy_of_match);
--
2.36.1
^ permalink raw reply related [flat|nested] 8+ messages in thread
* Re: [PATCH v4 3/3] phy: cdns-dphy: Add support for DPHY TX on J721e
2022-06-23 12:54 ` [PATCH v4 3/3] phy: cdns-dphy: Add support for DPHY TX on J721e Rahul T R
@ 2022-06-24 9:45 ` Pratyush Yadav
0 siblings, 0 replies; 8+ messages in thread
From: Pratyush Yadav @ 2022-06-24 9:45 UTC (permalink / raw)
To: Rahul T R
Cc: linux-phy, kishon, vkoul, robh+dt, krzysztof.kozlowski+dt,
tomi.valkeinen, laurent.pinchart, linux-kernel, jpawar, sjakhade,
mparab, devicetree, vigneshr, lee.jones
On 23/06/22 06:24PM, Rahul T R wrote:
> Add support new compatible for dphy-tx on j721e
> and implement dphy ops required.
>
> Signed-off-by: Rahul T R <r-ravikumar@ti.com>
Reviewed-by: Pratyush Yadav <p.yadav@ti.com>
--
Regards,
Pratyush Yadav
Texas Instruments Inc.
^ permalink raw reply [flat|nested] 8+ messages in thread
* Re: [PATCH v4 3/3] phy: cdns-dphy: Add support for DPHY TX on J721e
@ 2022-06-24 9:45 ` Pratyush Yadav
0 siblings, 0 replies; 8+ messages in thread
From: Pratyush Yadav @ 2022-06-24 9:45 UTC (permalink / raw)
To: Rahul T R
Cc: linux-phy, kishon, vkoul, robh+dt, krzysztof.kozlowski+dt,
tomi.valkeinen, laurent.pinchart, linux-kernel, jpawar, sjakhade,
mparab, devicetree, vigneshr, lee.jones
On 23/06/22 06:24PM, Rahul T R wrote:
> Add support new compatible for dphy-tx on j721e
> and implement dphy ops required.
>
> Signed-off-by: Rahul T R <r-ravikumar@ti.com>
Reviewed-by: Pratyush Yadav <p.yadav@ti.com>
--
Regards,
Pratyush Yadav
Texas Instruments Inc.
--
linux-phy mailing list
linux-phy@lists.infradead.org
https://lists.infradead.org/mailman/listinfo/linux-phy
^ permalink raw reply [flat|nested] 8+ messages in thread
* Re: [PATCH v4 0/3] Add support for dphy tx on j721e
2022-06-23 12:54 [PATCH v4 0/3] Add support for dphy tx on j721e Rahul T R
@ 2022-07-05 6:53 ` Vinod Koul
2022-06-23 12:54 ` [PATCH v4 2/3] phy: cdns-dphy: Add band config for dphy tx Rahul T R
` (2 subsequent siblings)
3 siblings, 0 replies; 8+ messages in thread
From: Vinod Koul @ 2022-07-05 6:53 UTC (permalink / raw)
To: Rahul T R
Cc: linux-phy, kishon, robh+dt, krzysztof.kozlowski+dt, p.yadav,
tomi.valkeinen, laurent.pinchart, linux-kernel, jpawar, sjakhade,
mparab, devicetree, vigneshr, lee.jones
On 23-06-22, 18:24, Rahul T R wrote:
> Following series of patches adds support for dphy tx on ti's j721e
> soc. new compatible is added and required cdns dphy ops are implemented.
> the series also adds band ctrl configuration required for dphy tx
Applied, thanks
--
~Vinod
^ permalink raw reply [flat|nested] 8+ messages in thread
* Re: [PATCH v4 0/3] Add support for dphy tx on j721e
@ 2022-07-05 6:53 ` Vinod Koul
0 siblings, 0 replies; 8+ messages in thread
From: Vinod Koul @ 2022-07-05 6:53 UTC (permalink / raw)
To: Rahul T R
Cc: linux-phy, kishon, robh+dt, krzysztof.kozlowski+dt, p.yadav,
tomi.valkeinen, laurent.pinchart, linux-kernel, jpawar, sjakhade,
mparab, devicetree, vigneshr, lee.jones
On 23-06-22, 18:24, Rahul T R wrote:
> Following series of patches adds support for dphy tx on ti's j721e
> soc. new compatible is added and required cdns dphy ops are implemented.
> the series also adds band ctrl configuration required for dphy tx
Applied, thanks
--
~Vinod
--
linux-phy mailing list
linux-phy@lists.infradead.org
https://lists.infradead.org/mailman/listinfo/linux-phy
^ permalink raw reply [flat|nested] 8+ messages in thread
end of thread, other threads:[~2022-07-05 6:54 UTC | newest]
Thread overview: 8+ messages (download: mbox.gz / follow: Atom feed)
-- links below jump to the message on this page --
2022-06-23 12:54 [PATCH v4 0/3] Add support for dphy tx on j721e Rahul T R
2022-06-23 12:54 ` [PATCH v4 1/3] phy: dt-bindings: cdns,dphy: Add compatible for dphy " Rahul T R
2022-06-23 12:54 ` [PATCH v4 2/3] phy: cdns-dphy: Add band config for dphy tx Rahul T R
2022-06-23 12:54 ` [PATCH v4 3/3] phy: cdns-dphy: Add support for DPHY TX on J721e Rahul T R
2022-06-24 9:45 ` Pratyush Yadav
2022-06-24 9:45 ` Pratyush Yadav
2022-07-05 6:53 ` [PATCH v4 0/3] Add support for dphy tx on j721e Vinod Koul
2022-07-05 6:53 ` Vinod Koul
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.