From: Nicholas Kazlauskas <nicholas.kazlaus...@amd.com>

This reverts commit b91fff88686e("drm/amd/display: turn off eDP lcdvdd and 
backlight if not required")

Reason for revert: Causes S4 lightup regressions.

Reviewed-by: Gabe Teeger <gabe.tee...@amd.com>
Signed-off-by: Nicholas Kazlauskas <nicholas.kazlaus...@amd.com>
Signed-off-by: Ray Wu <ray...@amd.com>
---
 drivers/gpu/drm/amd/display/dc/core/dc.c                      | 4 ++--
 .../amd/display/dc/link/protocols/link_edp_panel_control.c    | 2 --
 2 files changed, 2 insertions(+), 4 deletions(-)

diff --git a/drivers/gpu/drm/amd/display/dc/core/dc.c 
b/drivers/gpu/drm/amd/display/dc/core/dc.c
index 66e23507eb82..636999fcaebb 100644
--- a/drivers/gpu/drm/amd/display/dc/core/dc.c
+++ b/drivers/gpu/drm/amd/display/dc/core/dc.c
@@ -6298,10 +6298,10 @@ void dc_query_current_properties(struct dc *dc, struct 
dc_current_properties *pr
 void dc_set_edp_power(const struct dc *dc, struct dc_link *edp_link,
                                 bool powerOn)
 {
-       if (!edp_link || !edp_link->dc || !edp_link->dc->link_srv)
+       if (edp_link->connector_signal != SIGNAL_TYPE_EDP)
                return;
 
-       if (edp_link->connector_signal != SIGNAL_TYPE_EDP)
+       if (edp_link->skip_implict_edp_power_control == false)
                return;
 
        edp_link->dc->link_srv->edp_set_panel_power(edp_link, powerOn);
diff --git 
a/drivers/gpu/drm/amd/display/dc/link/protocols/link_edp_panel_control.c 
b/drivers/gpu/drm/amd/display/dc/link/protocols/link_edp_panel_control.c
index f50c4dbc7553..da74c2b5854f 100644
--- a/drivers/gpu/drm/amd/display/dc/link/protocols/link_edp_panel_control.c
+++ b/drivers/gpu/drm/amd/display/dc/link/protocols/link_edp_panel_control.c
@@ -393,7 +393,6 @@ void edp_set_panel_power(struct dc_link *link, bool powerOn)
 
                // 3. Rx power on
                dpcd_write_rx_power_ctrl(link, true);
-               DC_LOG_BACKLIGHT("eDP power and backlight: Power on");
        } else {
                // 3. Rx power off
                dpcd_write_rx_power_ctrl(link, false);
@@ -405,7 +404,6 @@ void edp_set_panel_power(struct dc_link *link, bool powerOn)
                // 1. panel VDD off
                if (!link->dc->config.edp_no_power_sequencing)
                        link->dc->hwss.edp_power_control(link, false);
-               DC_LOG_BACKLIGHT("eDP power and backlight: Power off");
        }
 }
 
-- 
2.43.0

Reply via email to