Re: [PATCH v3] phy: qcom-qmp: add hbr3_hbr2 voltage and premphasis swing table

[Vinod Koul]

On 16-03-21, 10:12, Kuogee Hsieh wrote:
> Add hbr3_hbr2 voltage and premphasis swing table to support
> HBR3 link rate.
> 
> Changes in V2:
> -- replaced upper case with lower case at hbr3_hbr2 table
> 
> Changes in v3:
> -- replace space with tab at hbr3_hbr2 table

This should be after the --- line below
> 
> Signed-off-by: Kuogee Hsieh 

No need of blank line here
> 
> Reviewed-by: Stephen Boyd 

I took the liberty of fixing these two, applying the fixes tag and
applied..


-- 
~Vinod

Re: [PATCH v3] phy: qcom-qmp: add hbr3_hbr2 voltage and premphasis swing table

[Stephen Boyd]

Quoting Kuogee Hsieh (2021-03-16 10:12:41)
> Add hbr3_hbr2 voltage and premphasis swing table to support
> HBR3 link rate.
> 
> Changes in V2:
> -- replaced upper case with lower case at hbr3_hbr2 table
> 
> Changes in v3:
> -- replace space with tab at hbr3_hbr2 table
> 
> Signed-off-by: Kuogee Hsieh 
> 
> Reviewed-by: Stephen Boyd 

Fixes: 52e013d0bffa ("phy: qcom-qmp: Add support for DP in USB3+DP combo phy")

[PATCH v3] phy: qcom-qmp: add hbr3_hbr2 voltage and premphasis swing table

[Kuogee Hsieh]

Add hbr3_hbr2 voltage and premphasis swing table to support
HBR3 link rate.

Changes in V2:
-- replaced upper case with lower case at hbr3_hbr2 table

Changes in v3:
-- replace space with tab at hbr3_hbr2 table

Signed-off-by: Kuogee Hsieh 

Reviewed-by: Stephen Boyd 
---
 drivers/phy/qualcomm/phy-qcom-qmp.c | 23 +--
 1 file changed, 21 insertions(+), 2 deletions(-)

diff --git a/drivers/phy/qualcomm/phy-qcom-qmp.c 
b/drivers/phy/qualcomm/phy-qcom-qmp.c
index 0939a9e..6aaa59c 100644
--- a/drivers/phy/qualcomm/phy-qcom-qmp.c
+++ b/drivers/phy/qualcomm/phy-qcom-qmp.c
@@ -2965,6 +2965,20 @@ static void qcom_qmp_phy_dp_aux_init(struct qmp_phy 
*qphy)
   qphy->pcs + QSERDES_V3_DP_PHY_AUX_INTERRUPT_MASK);
 }
 
+static const u8 qmp_dp_v3_pre_emphasis_hbr3_hbr2[4][4] = {
+   { 0x00, 0x0c, 0x15, 0x1a },
+   { 0x02, 0x0e, 0x16, 0xff },
+   { 0x02, 0x11, 0xff, 0xff },
+   { 0x04, 0xff, 0xff, 0xff }
+};
+
+static const u8 qmp_dp_v3_voltage_swing_hbr3_hbr2[4][4] = {
+   { 0x02, 0x12, 0x16, 0x1a },
+   { 0x09, 0x19, 0x1f, 0xff },
+   { 0x10, 0x1f, 0xff, 0xff },
+   { 0x1f, 0xff, 0xff, 0xff }
+};
+
 static const u8 qmp_dp_v3_pre_emphasis_hbr_rbr[4][4] = {
{ 0x00, 0x0c, 0x14, 0x19 },
{ 0x00, 0x0b, 0x12, 0xff },
@@ -3000,8 +3014,13 @@ static void qcom_qmp_phy_configure_dp_tx(struct qmp_phy 
*qphy)
drvr_en = 0x10;
}
 
-   voltage_swing_cfg = qmp_dp_v3_voltage_swing_hbr_rbr[v_level][p_level];
-   pre_emphasis_cfg = qmp_dp_v3_pre_emphasis_hbr_rbr[v_level][p_level];
+   if (dp_opts->link_rate <= 2700) {
+   voltage_swing_cfg = 
qmp_dp_v3_voltage_swing_hbr_rbr[v_level][p_level];
+   pre_emphasis_cfg = 
qmp_dp_v3_pre_emphasis_hbr_rbr[v_level][p_level];
+   } else {
+   voltage_swing_cfg = 
qmp_dp_v3_voltage_swing_hbr3_hbr2[v_level][p_level];
+   pre_emphasis_cfg = 
qmp_dp_v3_pre_emphasis_hbr3_hbr2[v_level][p_level];
+   }
 
/* TODO: Move check to config check */
if (voltage_swing_cfg == 0xFF && pre_emphasis_cfg == 0xFF)
-- 
The Qualcomm Innovation Center, Inc. is a member of the Code Aurora Forum,
a Linux Foundation Collaborative Project