]> www.infradead.org Git - users/willy/xarray.git/commitdiff
drm/i915/dp: Fix disabling training pattern at end of UHBR link training
authorImre Deak <imre.deak@intel.com>
Thu, 24 Jul 2025 18:29:00 +0000 (21:29 +0300)
committerImre Deak <imre.deak@intel.com>
Fri, 25 Jul 2025 14:41:50 +0000 (17:41 +0300)
The Fixed: commit below overlooked the fact that
intel_dp_link_train_all_phys() is only used for non-UHBR link rates, but
intel_dp_stop_link_train() is used for both non-UHBR and UHBR link
rates. Hence, after removing the disabling of the training pattern from
intel_dp_stop_link_train(), the commit missed adding this back to the
end of UHBR link training in intel_dp_128b132b_link_train(). This left
the sink in link training mode at the end of an UHBR rate link training.

Fix things by disabling the training pattern at the end of UHBR link
training as well.

Cc: Ville Syrjälä <ville.syrjala@linux.intel.com>
Fixes: 11fab5a2a1ad ("drm/i915/dp: Clear DPCD training pattern before transmitting the idle pattern")
Reviewed-by: Arun R Murthy <arun.r.murthy@intel.com>
Signed-off-by: Imre Deak <imre.deak@intel.com>
Link: https://lore.kernel.org/r/20250724182900.160891-1-imre.deak@intel.com
drivers/gpu/drm/i915/display/intel_dp_link_training.c

index 344b74109a83038ae0b74402ab42eb04447bfe5c..9d14b4e5954561a095fd36774768f2e4dca456d2 100644 (file)
@@ -1609,6 +1609,8 @@ out:
                intel_dp_program_link_training_pattern(intel_dp, crtc_state,
                                                       DP_PHY_DPRX, DP_TRAINING_PATTERN_2);
 
+       intel_dp_disable_dpcd_training_pattern(intel_dp, DP_PHY_DPRX);
+
        return passed;
 }