Skip to content

Commit

Permalink
phy: qcom: qmp-combo: Fix register base for QSERDES_DP_PHY_MODE
Browse files Browse the repository at this point in the history
The register base that was used to write to the QSERDES_DP_PHY_MODE
register was 'dp_dp_phy' before commit 815891e ("phy:
qcom-qmp-combo: Introduce orientation variable"). There isn't any
explanation in the commit why this is changed, so I suspect it was an
oversight or happened while being extracted from some other series.
Oddly the value being 0x4c or 0x5c doesn't seem to matter for me, so I
suspect this is dead code, but that can be fixed in another patch. It's
not good to write to the wrong register space, and maybe some other
version of this phy relies on this.

Cc: Douglas Anderson <[email protected]>
Cc: Abhinav Kumar <[email protected]>
Cc: Dmitry Baryshkov <[email protected]>
Cc: Neil Armstrong <[email protected]>
Cc: Abel Vesa <[email protected]>
Cc: Steev Klimaszewski <[email protected]>
Cc: Johan Hovold <[email protected]>
Cc: Bjorn Andersson <[email protected]>
Cc: [email protected]      # 6.5
Fixes: 815891e ("phy: qcom-qmp-combo: Introduce orientation variable")
Signed-off-by: Stephen Boyd <[email protected]>
Reviewed-by: Abhinav Kumar <[email protected]>
Reviewed-by: Bjorn Andersson <[email protected]>
Reviewed-by: Dmitry Baryshkov <[email protected]>
Reviewed-by: Johan Hovold <[email protected]>
Link: https://lore.kernel.org/r/[email protected]
Signed-off-by: Vinod Koul <[email protected]>
  • Loading branch information
bebarino authored and vinodkoul committed Apr 6, 2024
1 parent 5abed58 commit ee13e1f
Showing 1 changed file with 2 additions and 2 deletions.
4 changes: 2 additions & 2 deletions drivers/phy/qualcomm/phy-qcom-qmp-combo.c
Original file line number Diff line number Diff line change
Expand Up @@ -2153,9 +2153,9 @@ static bool qmp_combo_configure_dp_mode(struct qmp_combo *qmp)
writel(val, qmp->dp_dp_phy + QSERDES_DP_PHY_PD_CTL);

if (reverse)
writel(0x4c, qmp->pcs + QSERDES_DP_PHY_MODE);
writel(0x4c, qmp->dp_dp_phy + QSERDES_DP_PHY_MODE);
else
writel(0x5c, qmp->pcs + QSERDES_DP_PHY_MODE);
writel(0x5c, qmp->dp_dp_phy + QSERDES_DP_PHY_MODE);

return reverse;
}
Expand Down

0 comments on commit ee13e1f

Please sign in to comment.