diff mbox series

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

Message ID 1615914761-12300-1-git-send-email-khsieh@codeaurora.org
State Accepted
Commit 3f2ec77c954c6707a7caab1be35afe0ccf57135e
Headers show
Series [v3] phy: qcom-qmp: add hbr3_hbr2 voltage and premphasis swing table | expand

Commit Message

Kuogee Hsieh March 16, 2021, 5:12 p.m. UTC
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 <khsieh@codeaurora.org>

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

Comments

Stephen Boyd March 17, 2021, 12:54 a.m. UTC | #1
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 <khsieh@codeaurora.org>

> 

> Reviewed-by: Stephen Boyd <swboyd@chromium.org>


Fixes: 52e013d0bffa ("phy: qcom-qmp: Add support for DP in USB3+DP combo phy")
Vinod Koul March 17, 2021, 6:37 a.m. UTC | #2
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 <khsieh@codeaurora.org>


No need of blank line here
> 

> Reviewed-by: Stephen Boyd <swboyd@chromium.org>


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


-- 
~Vinod
patchwork-bot+linux-arm-msm@kernel.org May 26, 2021, 7:03 p.m. UTC | #3
Hello:

This patch was applied to qcom/linux.git (refs/heads/for-next):

On Tue, 16 Mar 2021 10:12:41 -0700 you 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

> 

> [...]


Here is the summary with links:
  - [v3] phy: qcom-qmp: add hbr3_hbr2 voltage and premphasis swing table
    https://git.kernel.org/qcom/c/3f2ec77c954c

You are awesome, thank you!
--
Deet-doot-dot, I am a bot.
https://korg.docs.kernel.org/patchwork/pwbot.html
diff mbox series

Patch

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)