From: Anthony Koo <anthony....@amd.com>

[Why]
During S4/S3 stress test it is possible to resume from S4 without
calling mode set on eDP, meaning high level optimization flag is not
reset. If this is followed by an S3 resume call, driver will see
optimization flag is set and consume it and think backend is powered
on when in fact it is not.

This results in PHY being off in sequence where
S4->Resume->S3->Resume->ApplyOpt->black screen.

[How]
Move optimization flag to stream instead of a DC flag.

Change-Id: I79bbd25f9364bc62ba4ae1886b27d15d8e6a3638
Signed-off-by: Anthony Koo <anthony....@amd.com>
Reviewed-by: Tony Cheng <tony.ch...@amd.com>
Acked-by: Bhawanpreet Lakha <bhawanpreet.la...@amd.com>
---
 drivers/gpu/drm/amd/display/dc/core/dc_link.c      |  4 +--
 drivers/gpu/drm/amd/display/dc/dc.h                |  2 --
 drivers/gpu/drm/amd/display/dc/dc_stream.h         |  1 +
 .../amd/display/dc/dce110/dce110_hw_sequencer.c    | 31 +++++++++++++---------
 4 files changed, 22 insertions(+), 16 deletions(-)

diff --git a/drivers/gpu/drm/amd/display/dc/core/dc_link.c 
b/drivers/gpu/drm/amd/display/dc/core/dc_link.c
index 6638251162b0..53ce2a9b7eed 100644
--- a/drivers/gpu/drm/amd/display/dc/core/dc_link.c
+++ b/drivers/gpu/drm/amd/display/dc/core/dc_link.c
@@ -2497,8 +2497,8 @@ void core_link_enable_stream(
 
        /* eDP lit up by bios already, no need to enable again. */
        if (pipe_ctx->stream->signal == SIGNAL_TYPE_EDP &&
-               core_dc->apply_edp_fast_boot_optimization) {
-               core_dc->apply_edp_fast_boot_optimization = false;
+                       pipe_ctx->stream->apply_edp_fast_boot_optimization) {
+               pipe_ctx->stream->apply_edp_fast_boot_optimization = false;
                pipe_ctx->stream->dpms_off = false;
                return;
        }
diff --git a/drivers/gpu/drm/amd/display/dc/dc.h 
b/drivers/gpu/drm/amd/display/dc/dc.h
index fa1efb339bb2..2bb77198a063 100644
--- a/drivers/gpu/drm/amd/display/dc/dc.h
+++ b/drivers/gpu/drm/amd/display/dc/dc.h
@@ -311,8 +311,6 @@ struct dc {
 
        bool optimized_required;
 
-       bool apply_edp_fast_boot_optimization;
-
        /* FBC compressor */
        struct compressor *fbc_compressor;
 
diff --git a/drivers/gpu/drm/amd/display/dc/dc_stream.h 
b/drivers/gpu/drm/amd/display/dc/dc_stream.h
index 4a9f7e5daccf..c5bd1fbb6982 100644
--- a/drivers/gpu/drm/amd/display/dc/dc_stream.h
+++ b/drivers/gpu/drm/amd/display/dc/dc_stream.h
@@ -102,6 +102,7 @@ struct dc_stream_state {
        int phy_pix_clk;
        enum signal_type signal;
        bool dpms_off;
+       bool apply_edp_fast_boot_optimization;
 
        struct dc_stream_status status;
 
diff --git a/drivers/gpu/drm/amd/display/dc/dce110/dce110_hw_sequencer.c 
b/drivers/gpu/drm/amd/display/dc/dce110/dce110_hw_sequencer.c
index 350ee3e3e34d..0599d468c13f 100644
--- a/drivers/gpu/drm/amd/display/dc/dce110/dce110_hw_sequencer.c
+++ b/drivers/gpu/drm/amd/display/dc/dce110/dce110_hw_sequencer.c
@@ -1557,26 +1557,33 @@ static struct dc_link *get_link_for_edp_not_in_use(
  */
 void dce110_enable_accelerated_mode(struct dc *dc, struct dc_state *context)
 {
+       int i;
        struct dc_link *edp_link_to_turnoff = NULL;
        struct dc_link *edp_link = get_link_for_edp(dc);
-       bool can_eDP_fast_boot_optimize = false;
+       bool can_edp_fast_boot_optimize = false;
+       bool apply_edp_fast_boot_optimization = false;
 
-       if (edp_link) {
-               can_eDP_fast_boot_optimize =
-                               
edp_link->link_enc->funcs->is_dig_enabled(edp_link->link_enc);
-       }
+       if (edp_link)
+               can_edp_fast_boot_optimize =
+                       
edp_link->link_enc->funcs->is_dig_enabled(edp_link->link_enc);
 
-       if (can_eDP_fast_boot_optimize) {
+       if (can_edp_fast_boot_optimize)
                edp_link_to_turnoff = get_link_for_edp_not_in_use(dc, context);
 
-               /* if OS doesn't light up eDP and eDP link is available, we 
want to disable
-                * If resume from S4/S5, should optimization.
-                */
-               if (!edp_link_to_turnoff)
-                       dc->apply_edp_fast_boot_optimization = true;
+       /* if OS doesn't light up eDP and eDP link is available, we want to 
disable
+        * If resume from S4/S5, should optimization.
+        */
+       if (can_edp_fast_boot_optimize && !edp_link_to_turnoff) {
+               /* Find eDP stream and set optimization flag */
+               for (i = 0; i < context->stream_count; i++) {
+                       if (context->streams[i]->signal == SIGNAL_TYPE_EDP) {
+                               
context->streams[i]->apply_edp_fast_boot_optimization = true;
+                               apply_edp_fast_boot_optimization = true;
+                       }
+               }
        }
 
-       if (!dc->apply_edp_fast_boot_optimization) {
+       if (!apply_edp_fast_boot_optimization) {
                if (edp_link_to_turnoff) {
                        /*turn off backlight before DP_blank and encoder 
powered down*/
                        dc->hwss.edp_backlight_control(edp_link_to_turnoff, 
false);
-- 
2.14.1

_______________________________________________
amd-gfx mailing list
amd-gfx@lists.freedesktop.org
https://lists.freedesktop.org/mailman/listinfo/amd-gfx

Reply via email to