From: Michael Strauss <michael.stra...@amd.com>

[WHY]
Needed for PHY patterns as well to perform electrical compliance.
Also need to increase wait time from 30ms to 50ms to resolve very
intermittent UHBR20 link training failures.

Reviewed-by: Wenjing Liu <wenjing....@amd.com>
Acked-by: Zaeem Mohamed <zaeem.moha...@amd.com>
Signed-off-by: Michael Strauss <michael.stra...@amd.com>
---
 .../dc/link/hwss/link_hwss_hpo_fixed_vs_pe_retimer_dp.c     | 6 +++---
 1 file changed, 3 insertions(+), 3 deletions(-)

diff --git 
a/drivers/gpu/drm/amd/display/dc/link/hwss/link_hwss_hpo_fixed_vs_pe_retimer_dp.c
 
b/drivers/gpu/drm/amd/display/dc/link/hwss/link_hwss_hpo_fixed_vs_pe_retimer_dp.c
index 5302d2c9c760..116ff37126e7 100644
--- 
a/drivers/gpu/drm/amd/display/dc/link/hwss/link_hwss_hpo_fixed_vs_pe_retimer_dp.c
+++ 
b/drivers/gpu/drm/amd/display/dc/link/hwss/link_hwss_hpo_fixed_vs_pe_retimer_dp.c
@@ -168,9 +168,9 @@ static void 
set_hpo_fixed_vs_pe_retimer_dp_link_test_pattern(struct dc_link *lin
 
        link->dc->link_srv->dp_trace_source_sequence(link, 
DPCD_SOURCE_SEQ_AFTER_SET_SOURCE_PATTERN);
 
-       // Give retimer extra time to lock before updating 
DP_TRAINING_PATTERN_SET to TPS1
-       if (tp_params->dp_phy_pattern == 
DP_TEST_PATTERN_128b_132b_TPS1_TRAINING_MODE)
-               msleep(30);
+       // Give retimer extra time to lock before updating 
DP_TRAINING_PATTERN_SET to TPS1 or phy pattern
+       if (tp_params->dp_phy_pattern != 
DP_TEST_PATTERN_128b_132b_TPS2_TRAINING_MODE)
+               msleep(50);
 }
 
 static void set_hpo_fixed_vs_pe_retimer_dp_lane_settings(struct dc_link *link,
-- 
2.34.1

Reply via email to