drm/msms/dp: fixed link clock divider bits be over written in BPC unknown case
authorKuogee Hsieh <quic_khsieh@quicinc.com>
Wed, 10 Jan 2024 20:18:51 +0000 (12:18 -0800)
committerAbhinav Kumar <quic_abhinavk@quicinc.com>
Wed, 24 Jan 2024 22:06:24 +0000 (14:06 -0800)
commit77e8aad5519e04f6c1e132aaec1c5f8faf41844f
treead8e14a4e248c9bb5e0c573ac7b4b164c9bbf438
parent1793ce9f205efe07ca2992ef129b86dab2c329f5
drm/msms/dp: fixed link clock divider bits be over written in BPC unknown case

Since the value of DP_TEST_BIT_DEPTH_8 is already left shifted, in the
BPC unknown case, the additional shift causes spill over to the other
bits of the [DP_CONFIGURATION_CTRL] register.
Fix this by changing the return value of dp_link_get_test_bits_depth()
in the BPC unknown case to (DP_TEST_BIT_DEPTH_8 >> DP_TEST_BIT_DEPTH_SHIFT).

Fixes: c943b4948b58 ("drm/msm/dp: add displayPort driver support")
Signed-off-by: Kuogee Hsieh <quic_khsieh@quicinc.com>
Reviewed-by: Abhinav Kumar <quic_abhinavk@quicinc.com>
Reviewed-by: Dmitry Baryshkov <dmitry.baryshkov@linaro.org>
Patchwork: https://patchwork.freedesktop.org/patch/573989/
Link: https://lore.kernel.org/r/1704917931-30133-1-git-send-email-quic_khsieh@quicinc.com
[quic_abhinavk@quicinc.com: fix minor checkpatch warning to align with opening braces]
Signed-off-by: Abhinav Kumar <quic_abhinavk@quicinc.com>
drivers/gpu/drm/msm/dp/dp_ctrl.c
drivers/gpu/drm/msm/dp/dp_link.c