commit d34d6feaf4a76833effcec0b148b65946b04cde8 upstream.

Change the AUX DPCD probe address to DP_TRAINING_PATTERN_SET. Using
DP_DPCD_REV for this is not compliant with the DP Standard and it leads
to link training failures at least on a DP2.0 docking station when using
UHBR link rates.

This patch is a revert of commit 944e732be9c3 ("drm/dp: Change AUX DPCD
probe address from DPCD_REV to LANE0_1_STATUS") and the corresponding
fix for commit 05981233cf2e ("Revert "drm/dp: Change AUX DPCD probe
address from DPCD_REV to LANE0_1_STATUS") in the v6.16.y tree.

This change is only meant to be applied in the v6.16.y tree, not in
earlier stable trees.

Cc: Greg Kroah-Hartman <gre...@linuxfoundation.org>
Cc: Sasha Levin <sas...@kernel.org>
Cc: dri-de...@lists.freedesktop.org
Cc: sta...@vger.kernel.org # v6.16
Signed-off-by: Imre Deak <imre.d...@intel.com>
---
 drivers/gpu/drm/display/drm_dp_helper.c | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

diff --git a/drivers/gpu/drm/display/drm_dp_helper.c 
b/drivers/gpu/drm/display/drm_dp_helper.c
index f2a6559a27100..ea78c6c8ca7a6 100644
--- a/drivers/gpu/drm/display/drm_dp_helper.c
+++ b/drivers/gpu/drm/display/drm_dp_helper.c
@@ -725,7 +725,7 @@ ssize_t drm_dp_dpcd_read(struct drm_dp_aux *aux, unsigned 
int offset,
         * monitor doesn't power down exactly after the throw away read.
         */
        if (!aux->is_remote) {
-               ret = drm_dp_dpcd_probe(aux, DP_DPCD_REV);
+               ret = drm_dp_dpcd_probe(aux, DP_TRAINING_PATTERN_SET);
                if (ret < 0)
                        return ret;
        }
-- 
2.49.1

Reply via email to