Commit 1c256f40 authored by Zhan Liu's avatar Zhan Liu Committed by Alex Deucher

drm/amd/display: Remove aconnector condition check for dpcd read

[Why]
Aconnector is not necessary to be NULL in order to read dpcd
successfully.

Actually if we rely on checking aconnector here, we won't be able
to turn off all displays before doing display detection. That will
cause some MST hubs not able to light up.

[How]
Remove aconnector check when turning off all displays at
hardware initialization stage.
Signed-off-by: default avatarZhan Liu <zhan.liu@amd.com>
Reviewed-by: default avatarJoseph Gravenor <joseph.gravenor@amd.com>
Signed-off-by: default avatarAlex Deucher <alexander.deucher@amd.com>
parent 8e0c819d
...@@ -1333,13 +1333,12 @@ void dcn10_init_hw(struct dc *dc) ...@@ -1333,13 +1333,12 @@ void dcn10_init_hw(struct dc *dc)
continue; continue;
/* /*
* core_link_read_dpcd() will invoke dm_helpers_dp_read_dpcd(), * If any of the displays are lit up turn them off.
* which needs to read dpcd info with the help of aconnector. * The reason is that some MST hubs cannot be turned off
* If aconnector (dc->links[i]->prev) is NULL, then dpcd status * completely until we tell them to do so.
* cannot be read. * If not turned off, then displays connected to MST hub
* won't light up.
*/ */
if (dc->links[i]->priv) {
/* if any of the displays are lit up turn them off */
status = core_link_read_dpcd(dc->links[i], DP_SET_POWER, status = core_link_read_dpcd(dc->links[i], DP_SET_POWER,
&dpcd_power_state, sizeof(dpcd_power_state)); &dpcd_power_state, sizeof(dpcd_power_state));
if (status == DC_OK && dpcd_power_state == DP_POWER_STATE_D0) { if (status == DC_OK && dpcd_power_state == DP_POWER_STATE_D0) {
...@@ -1359,7 +1358,6 @@ void dcn10_init_hw(struct dc *dc) ...@@ -1359,7 +1358,6 @@ void dcn10_init_hw(struct dc *dc)
} }
} }
} }
}
/* If taking control over from VBIOS, we may want to optimize our first /* If taking control over from VBIOS, we may want to optimize our first
* mode set, so we need to skip powering down pipes until we know which * mode set, so we need to skip powering down pipes until we know which
......
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