SDM845 DPU hardware is rev 4.0.0 per hardware documents.
Original patch to enable wide_bus operation did not take into account
the SDM845 and it got carried over by accident.

- Incorrect setting caused inoperable DisplayPort.
- Corrected by separating SDM845 into its own descriptor.

Fixes: c7c412202623 ("drm/msm/dp: enable widebus on all relevant chipsets")
Signed-off-by: James A. MacInnes <james.a.macin...@gmail.com>
---
 drivers/gpu/drm/msm/dp/dp_display.c | 7 ++++++-
 1 file changed, 6 insertions(+), 1 deletion(-)

diff --git a/drivers/gpu/drm/msm/dp/dp_display.c 
b/drivers/gpu/drm/msm/dp/dp_display.c
index aff51bb973eb..e30cccd63910 100644
--- a/drivers/gpu/drm/msm/dp/dp_display.c
+++ b/drivers/gpu/drm/msm/dp/dp_display.c
@@ -126,6 +126,11 @@ static const struct msm_dp_desc msm_dp_desc_sa8775p[] = {
        {}
 };
 
+static const struct msm_dp_desc msm_dp_desc_sdm845[] = {
+       { .io_start = 0x0ae90000, .id = MSM_DP_CONTROLLER_0 },
+       {}
+};
+
 static const struct msm_dp_desc msm_dp_desc_sc7180[] = {
        { .io_start = 0x0ae90000, .id = MSM_DP_CONTROLLER_0, 
.wide_bus_supported = true },
        {}
@@ -178,7 +183,7 @@ static const struct of_device_id msm_dp_dt_match[] = {
        { .compatible = "qcom,sc8180x-edp", .data = &msm_dp_desc_sc8180x },
        { .compatible = "qcom,sc8280xp-dp", .data = &msm_dp_desc_sc8280xp },
        { .compatible = "qcom,sc8280xp-edp", .data = &msm_dp_desc_sc8280xp },
-       { .compatible = "qcom,sdm845-dp", .data = &msm_dp_desc_sc7180 },
+       { .compatible = "qcom,sdm845-dp", .data = &msm_dp_desc_sdm845 },
        { .compatible = "qcom,sm8350-dp", .data = &msm_dp_desc_sc7180 },
        { .compatible = "qcom,sm8650-dp", .data = &msm_dp_desc_sm8650 },
        { .compatible = "qcom,x1e80100-dp", .data = &msm_dp_desc_x1e80100 },

-- 
2.43.0

Reply via email to