Skip to content

Commit f5860c8

Browse files
Yiling Chenalexdeucher
authored andcommitted
drm/amd/display: Remove unnecessary eDP power down
[why] When first time of link training is fail, eDP would be powered down and would not be powered up for next retry link training. It causes that all of retry link linking would be fail. [how] We has extracted both power up and down sequence from enable/disable link output function before DCN32. We remov eDP power down in dcn32_disable_link_output(). Reviewed-by: Charlene Liu <[email protected]> Signed-off-by: Yiling Chen <[email protected]> Signed-off-by: Tom Chung <[email protected]> Tested-by: Daniel Wheeler <[email protected]> Signed-off-by: Alex Deucher <[email protected]>
1 parent c7ccfc0 commit f5860c8

File tree

1 file changed

+6
-6
lines changed

1 file changed

+6
-6
lines changed

drivers/gpu/drm/amd/display/dc/hwss/dcn32/dcn32_hwseq.c

Lines changed: 6 additions & 6 deletions
Original file line numberDiff line numberDiff line change
@@ -1399,12 +1399,12 @@ void dcn32_disable_link_output(struct dc_link *link,
13991399

14001400
link_hwss->disable_link_output(link, link_res, signal);
14011401
link->phy_state.symclk_state = SYMCLK_OFF_TX_OFF;
1402-
1403-
if (signal == SIGNAL_TYPE_EDP &&
1404-
link->dc->hwss.edp_power_control &&
1405-
!link->skip_implict_edp_power_control)
1406-
link->dc->hwss.edp_power_control(link, false);
1407-
else if (dmcu != NULL && dmcu->funcs->unlock_phy)
1402+
/*
1403+
* Add the logic to extract BOTH power up and power down sequences
1404+
* from enable/disable link output and only call edp panel control
1405+
* in enable_link_dp and disable_link_dp once.
1406+
*/
1407+
if (dmcu != NULL && dmcu->funcs->unlock_phy)
14081408
dmcu->funcs->unlock_phy(dmcu);
14091409

14101410
dc->link_srv->dp_trace_source_sequence(link, DPCD_SOURCE_SEQ_AFTER_DISABLE_LINK_PHY);

0 commit comments

Comments
 (0)