Commit abf34ca4 authored by Wenjing Liu's avatar Wenjing Liu Committed by Alex Deucher

drm/amd/display: use preferred link settings for dp signal only

[why]
We set preferred link settings for virtual signal. However we don't support
virtual signal for UHBR link rate. If preferred is set to UHBR link rate, we
will allow virtual signal with UHBR link rate which causes system crashes.
Reviewed-by: default avatarDillon Varone <dillon.varone@amd.com>
Acked-by: default avatarZaeem Mohamed <zaeem.mohamed@amd.com>
Signed-off-by: default avatarWenjing Liu <wenjing.liu@amd.com>
Tested-by: default avatarDaniel Wheeler <daniel.wheeler@amd.com>
Signed-off-by: default avatarAlex Deucher <alexander.deucher@amd.com>
parent c006d1a1
...@@ -930,21 +930,17 @@ bool link_decide_link_settings(struct dc_stream_state *stream, ...@@ -930,21 +930,17 @@ bool link_decide_link_settings(struct dc_stream_state *stream,
memset(link_setting, 0, sizeof(*link_setting)); memset(link_setting, 0, sizeof(*link_setting));
/* if preferred is specified through AMDDP, use it, if it's enough if (dc_is_dp_signal(stream->signal) &&
* to drive the mode link->preferred_link_setting.lane_count != LANE_COUNT_UNKNOWN &&
*/ link->preferred_link_setting.link_rate != LINK_RATE_UNKNOWN) {
if (link->preferred_link_setting.lane_count != /* if preferred is specified through AMDDP, use it, if it's enough
LANE_COUNT_UNKNOWN && * to drive the mode
link->preferred_link_setting.link_rate != */
LINK_RATE_UNKNOWN) {
*link_setting = link->preferred_link_setting; *link_setting = link->preferred_link_setting;
return true; } else if (stream->signal == SIGNAL_TYPE_DISPLAY_PORT_MST) {
} /* MST doesn't perform link training for now
* TODO: add MST specific link training routine
/* MST doesn't perform link training for now */
* TODO: add MST specific link training routine
*/
if (stream->signal == SIGNAL_TYPE_DISPLAY_PORT_MST) {
decide_mst_link_settings(link, link_setting); decide_mst_link_settings(link, link_setting);
} else if (link->connector_signal == SIGNAL_TYPE_EDP) { } else if (link->connector_signal == SIGNAL_TYPE_EDP) {
/* enable edp link optimization for DSC eDP case */ /* enable edp link optimization for DSC eDP case */
......
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