From: Yi-Ling Chen Date: Mon, 1 Jun 2020 02:46:07 +0000 (+0800) Subject: drm/amd/display: Fixed using wrong eDP power sequence function pointer X-Git-Tag: v5.15~3115^2~25^2~39 X-Git-Url: http://review.tizen.org/git/?a=commitdiff_plain;h=75cb89112e948bfcbe93a49f25d3b83519e0d0f2;p=platform%2Fkernel%2Flinux-starfive.git drm/amd/display: Fixed using wrong eDP power sequence function pointer [why] dc->hwss->edp_backlight_control is null, it would casue it only be off main-link of eDP. It is not worng behavior for eDP power sequence off. [how] Must use hwseq->funcs.edp_backlight_control finction pointer for edp backlight. Signed-off-by: Yi-Ling Chen Reviewed-by: Sung Lee Acked-by: Rodrigo Siqueira Signed-off-by: Alex Deucher --- diff --git a/drivers/gpu/drm/amd/display/dc/dcn10/dcn10_hw_sequencer.c b/drivers/gpu/drm/amd/display/dc/dcn10/dcn10_hw_sequencer.c index 845e7f8..abb160b 100644 --- a/drivers/gpu/drm/amd/display/dc/dcn10/dcn10_hw_sequencer.c +++ b/drivers/gpu/drm/amd/display/dc/dcn10/dcn10_hw_sequencer.c @@ -1394,10 +1394,10 @@ void dcn10_init_hw(struct dc *dc) if (edp_link && edp_link->link_enc->funcs->is_dig_enabled && edp_link->link_enc->funcs->is_dig_enabled(edp_link->link_enc) && - dc->hwss.edp_backlight_control && + dc->hwseq->funcs.edp_backlight_control && dc->hwss.power_down && dc->hwss.edp_power_control) { - dc->hwss.edp_backlight_control(edp_link, false); + dc->hwseq->funcs.edp_backlight_control(edp_link, false); dc->hwss.power_down(dc); dc->hwss.edp_power_control(edp_link, false); } else {