Commit bf5cda33 authored by Hersen Wu's avatar Hersen Wu Committed by Alex Deucher

drm/amd/display: Fix DP PHY test pre-emphasis not set properly

Signed-off-by: default avatarHersen Wu <hersenxs.wu@amd.com>
Reviewed-by: default avatarHersen Wu <hersenxs.wu@amd.com>
Acked-by: default avatarHarry Wentland <Harry.Wentland@amd.com>
Signed-off-by: default avatarAlex Deucher <alexander.deucher@amd.com>
parent 18f39f2d
...@@ -332,14 +332,21 @@ static bool setup_psr(struct dc *dc, const struct dc_stream *stream) ...@@ -332,14 +332,21 @@ static bool setup_psr(struct dc *dc, const struct dc_stream *stream)
} }
static void set_drive_settings(struct dc *dc, static void set_drive_settings(struct dc *dc,
struct link_training_settings *lt_settings) struct link_training_settings *lt_settings,
const struct dc_link *link)
{ {
struct core_dc *core_dc = DC_TO_CORE(dc); struct core_dc *core_dc = DC_TO_CORE(dc);
int i; int i;
for (i = 0; i < core_dc->link_count; i++) for (i = 0; i < core_dc->link_count; i++) {
dc_link_dp_set_drive_settings(&core_dc->links[i]->public, if (&core_dc->links[i]->public == link)
lt_settings); break;
}
if (i >= core_dc->link_count)
ASSERT_CRITICAL(false);
dc_link_dp_set_drive_settings(&core_dc->links[i]->public, lt_settings);
} }
static void perform_link_training(struct dc *dc, static void perform_link_training(struct dc *dc,
......
...@@ -229,10 +229,9 @@ void dp_retrain_link_dp_test(struct core_link *link, ...@@ -229,10 +229,9 @@ void dp_retrain_link_dp_test(struct core_link *link,
link->link_enc, link->link_enc,
SIGNAL_TYPE_DISPLAY_PORT); SIGNAL_TYPE_DISPLAY_PORT);
/* Clear current link setting. /* Clear current link setting. */
* memset(&link->public.cur_link_settings, 0, memset(&link->public.cur_link_settings, 0,
* sizeof(link->public.cur_link_settings)); sizeof(link->public.cur_link_settings));
*/
link->link_enc->funcs->enable_dp_output( link->link_enc->funcs->enable_dp_output(
link->link_enc, link->link_enc,
...@@ -246,6 +245,8 @@ void dp_retrain_link_dp_test(struct core_link *link, ...@@ -246,6 +245,8 @@ void dp_retrain_link_dp_test(struct core_link *link,
link_setting, link_setting,
skip_video_pattern); skip_video_pattern);
link->public.cur_link_settings = *link_setting;
link->dc->hwss.unblank_stream(&pipes[i], link->dc->hwss.unblank_stream(&pipes[i],
link_setting); link_setting);
} }
......
...@@ -112,7 +112,8 @@ struct link_training_settings; ...@@ -112,7 +112,8 @@ struct link_training_settings;
struct dc_link_funcs { struct dc_link_funcs {
void (*set_drive_settings)(struct dc *dc, void (*set_drive_settings)(struct dc *dc,
struct link_training_settings *lt_settings); struct link_training_settings *lt_settings,
const struct dc_link *link);
void (*perform_link_training)(struct dc *dc, void (*perform_link_training)(struct dc *dc,
struct dc_link_settings *link_setting, struct dc_link_settings *link_setting,
bool skip_video_pattern); bool skip_video_pattern);
......
...@@ -1363,7 +1363,7 @@ void dce110_link_encoder_dp_set_lane_settings( ...@@ -1363,7 +1363,7 @@ void dce110_link_encoder_dp_set_lane_settings(
cntl.pixel_clock = link_settings->link_settings.link_rate * cntl.pixel_clock = link_settings->link_settings.link_rate *
LINK_RATE_REF_FREQ_IN_KHZ; LINK_RATE_REF_FREQ_IN_KHZ;
for (lane = 0; lane < link_settings->link_settings.lane_count; ++lane) { for (lane = 0; lane < link_settings->link_settings.lane_count; lane++) {
/* translate lane settings */ /* translate lane settings */
training_lane_set.bits.VOLTAGE_SWING_SET = training_lane_set.bits.VOLTAGE_SWING_SET =
......
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