Commit 2e7ef37c authored by Michael Strauss's avatar Michael Strauss Committed by Alex Deucher

drm/amd/display: Remove Legacy FIXED_VS Transparent LT Sequence

The New sequence has been in use in DCN314 with no regressions
introduced. Therefore, it is safe to enable this sequence for all
devices using FIXED_VS retimers. So, remove the legacy codepath and its
associated config flag.
Reviewed-by: default avatarOvidiu Bunea <ovidiu.bunea@amd.com>
Acked-by: default avatarHamza Mahfooz <hamza.mahfooz@amd.com>
Signed-off-by: default avatarMichael Strauss <michael.strauss@amd.com>
Signed-off-by: default avatarAlex Deucher <alexander.deucher@amd.com>
parent 63d0b872
...@@ -429,7 +429,6 @@ struct dc_config { ...@@ -429,7 +429,6 @@ struct dc_config {
bool force_bios_enable_lttpr; bool force_bios_enable_lttpr;
uint8_t force_bios_fixed_vs; uint8_t force_bios_fixed_vs;
int sdpif_request_limit_words_per_umc; int sdpif_request_limit_words_per_umc;
bool use_old_fixed_vs_sequence;
bool dc_mode_clk_limit_support; bool dc_mode_clk_limit_support;
bool EnableMinDispClkODM; bool EnableMinDispClkODM;
bool enable_auto_dpm_test_logs; bool enable_auto_dpm_test_logs;
......
...@@ -1505,10 +1505,7 @@ enum link_training_result dp_perform_link_training( ...@@ -1505,10 +1505,7 @@ enum link_training_result dp_perform_link_training(
* Non-LT AUX transactions inside training mode. * Non-LT AUX transactions inside training mode.
*/ */
if ((link->chip_caps & EXT_DISPLAY_PATH_CAPS__DP_FIXED_VS_EN) && encoding == DP_8b_10b_ENCODING) if ((link->chip_caps & EXT_DISPLAY_PATH_CAPS__DP_FIXED_VS_EN) && encoding == DP_8b_10b_ENCODING)
if (link->dc->config.use_old_fixed_vs_sequence) status = dp_perform_fixed_vs_pe_training_sequence(link, link_res, &lt_settings);
status = dp_perform_fixed_vs_pe_training_sequence_legacy(link, link_res, &lt_settings);
else
status = dp_perform_fixed_vs_pe_training_sequence(link, link_res, &lt_settings);
else if (encoding == DP_8b_10b_ENCODING) else if (encoding == DP_8b_10b_ENCODING)
status = dp_perform_8b_10b_link_training(link, link_res, &lt_settings); status = dp_perform_8b_10b_link_training(link, link_res, &lt_settings);
else if (encoding == DP_128b_132b_ENCODING) else if (encoding == DP_128b_132b_ENCODING)
......
...@@ -28,11 +28,6 @@ ...@@ -28,11 +28,6 @@
#define __DC_LINK_DP_FIXED_VS_PE_RETIMER_H__ #define __DC_LINK_DP_FIXED_VS_PE_RETIMER_H__
#include "link_dp_training.h" #include "link_dp_training.h"
enum link_training_result dp_perform_fixed_vs_pe_training_sequence_legacy(
struct dc_link *link,
const struct link_resource *link_res,
struct link_training_settings *lt_settings);
enum link_training_result dp_perform_fixed_vs_pe_training_sequence( enum link_training_result dp_perform_fixed_vs_pe_training_sequence(
struct dc_link *link, struct dc_link *link,
const struct link_resource *link_res, const struct link_resource *link_res,
......
...@@ -1941,8 +1941,6 @@ static bool dcn31_resource_construct( ...@@ -1941,8 +1941,6 @@ static bool dcn31_resource_construct(
dc->caps.color.mpc.ogam_rom_caps.hlg = 0; dc->caps.color.mpc.ogam_rom_caps.hlg = 0;
dc->caps.color.mpc.ocsc = 1; dc->caps.color.mpc.ocsc = 1;
dc->config.use_old_fixed_vs_sequence = true;
/* Use pipe context based otg sync logic */ /* Use pipe context based otg sync logic */
dc->config.use_pipe_ctx_sync_logic = true; dc->config.use_pipe_ctx_sync_logic = true;
......
Markdown is supported
0%
or
You are about to add 0 people to the discussion. Proceed with caution.
Finish editing this message first!
Please register or to comment