* [PATCH v3] phy: qcom-qmp: add hbr3_hbr2 voltage and premphasis swing table
@ 2021-03-16 17:12 Kuogee Hsieh
2021-03-17 0:54 ` Stephen Boyd
` (2 more replies)
0 siblings, 3 replies; 4+ messages in thread
From: Kuogee Hsieh @ 2021-03-16 17:12 UTC (permalink / raw)
To: robdclark, sean, swboyd, vkoul, agross, bjorn.andersson
Cc: tanmay, abhinavk, aravindh, khsieh, linux-arm-msm, linux-kernel
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(-)
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
^ permalink raw reply related [flat|nested] 4+ messages in thread
* Re: [PATCH v3] phy: qcom-qmp: add hbr3_hbr2 voltage and premphasis swing table
2021-03-16 17:12 [PATCH v3] phy: qcom-qmp: add hbr3_hbr2 voltage and premphasis swing table Kuogee Hsieh
@ 2021-03-17 0:54 ` Stephen Boyd
2021-03-17 6:37 ` Vinod Koul
2021-05-26 19:03 ` patchwork-bot+linux-arm-msm
2 siblings, 0 replies; 4+ messages in thread
From: Stephen Boyd @ 2021-03-17 0:54 UTC (permalink / raw)
To: Kuogee Hsieh, agross, bjorn.andersson, robdclark, sean, vkoul
Cc: tanmay, abhinavk, aravindh, khsieh, linux-arm-msm, linux-kernel
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")
^ permalink raw reply [flat|nested] 4+ messages in thread
* Re: [PATCH v3] phy: qcom-qmp: add hbr3_hbr2 voltage and premphasis swing table
2021-03-16 17:12 [PATCH v3] phy: qcom-qmp: add hbr3_hbr2 voltage and premphasis swing table Kuogee Hsieh
2021-03-17 0:54 ` Stephen Boyd
@ 2021-03-17 6:37 ` Vinod Koul
2021-05-26 19:03 ` patchwork-bot+linux-arm-msm
2 siblings, 0 replies; 4+ messages in thread
From: Vinod Koul @ 2021-03-17 6:37 UTC (permalink / raw)
To: Kuogee Hsieh
Cc: robdclark, sean, swboyd, agross, bjorn.andersson, tanmay,
abhinavk, aravindh, linux-arm-msm, linux-kernel
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
^ permalink raw reply [flat|nested] 4+ messages in thread
* Re: [PATCH v3] phy: qcom-qmp: add hbr3_hbr2 voltage and premphasis swing table
2021-03-16 17:12 [PATCH v3] phy: qcom-qmp: add hbr3_hbr2 voltage and premphasis swing table Kuogee Hsieh
2021-03-17 0:54 ` Stephen Boyd
2021-03-17 6:37 ` Vinod Koul
@ 2021-05-26 19:03 ` patchwork-bot+linux-arm-msm
2 siblings, 0 replies; 4+ messages in thread
From: patchwork-bot+linux-arm-msm @ 2021-05-26 19:03 UTC (permalink / raw)
To: Kuogee Hsieh; +Cc: linux-arm-msm
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
^ permalink raw reply [flat|nested] 4+ messages in thread
end of thread, other threads:[~2021-05-26 19:03 UTC | newest]
Thread overview: 4+ messages (download: mbox.gz / follow: Atom feed)
-- links below jump to the message on this page --
2021-03-16 17:12 [PATCH v3] phy: qcom-qmp: add hbr3_hbr2 voltage and premphasis swing table Kuogee Hsieh
2021-03-17 0:54 ` Stephen Boyd
2021-03-17 6:37 ` Vinod Koul
2021-05-26 19:03 ` patchwork-bot+linux-arm-msm
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox;
as well as URLs for NNTP newsgroup(s).