[PATCH 20/24] drm/amd/display: Avoid calling blank_stream() twice

[Wayne Lin]

From: Zhongwei Zhang <zhongwei.zh...@amd.com>

[Why]
We've made fix for garbage in dcn31_reset_back_end_for_pipe(), adding
blank_stream() before disable_crtc(). And set_dpms_off() will call
blank_stream() again.

[How]
Add flag to avoid calling blank_stream() twice.

Reviewed-by: Nicholas Kazlauskas <nicholas.kazlaus...@amd.com>
Signed-off-by: Zhongwei Zhang <zhongwei.zh...@amd.com>
Signed-off-by: Wayne Lin <wayne....@amd.com>
---
 .../gpu/drm/amd/display/dc/hwss/dce110/dce110_hwseq.c |  3 +++
 .../gpu/drm/amd/display/dc/hwss/dcn31/dcn31_hwseq.c   | 11 +++++++++--
 .../drm/amd/display/dc/hwss/hw_sequencer_private.h    |  1 +
 3 files changed, 13 insertions(+), 2 deletions(-)

diff --git a/drivers/gpu/drm/amd/display/dc/hwss/dce110/dce110_hwseq.c 
b/drivers/gpu/drm/amd/display/dc/hwss/dce110/dce110_hwseq.c
index a8174669bc49..b96b189e4a41 100644
--- a/drivers/gpu/drm/amd/display/dc/hwss/dce110/dce110_hwseq.c
+++ b/drivers/gpu/drm/amd/display/dc/hwss/dce110/dce110_hwseq.c
@@ -1222,6 +1222,9 @@ void dce110_blank_stream(struct pipe_ctx *pipe_ctx)
        struct dc_link *link = stream->link;
        struct dce_hwseq *hws = link->dc->hwseq;
 
+       if (hws && hws->wa_state.skip_blank_stream)
+               return;
+
        if (link->local_sink && link->local_sink->sink_signal == 
SIGNAL_TYPE_EDP) {
                if (!link->skip_implict_edp_power_control)
                        hws->funcs.edp_backlight_control(link, false);
diff --git a/drivers/gpu/drm/amd/display/dc/hwss/dcn31/dcn31_hwseq.c 
b/drivers/gpu/drm/amd/display/dc/hwss/dcn31/dcn31_hwseq.c
index f38340aa3f15..5ba3999991b0 100644
--- a/drivers/gpu/drm/amd/display/dc/hwss/dcn31/dcn31_hwseq.c
+++ b/drivers/gpu/drm/amd/display/dc/hwss/dcn31/dcn31_hwseq.c
@@ -526,9 +526,15 @@ static void dcn31_reset_back_end_for_pipe(
 
        link = pipe_ctx->stream->link;
 
+       if (dc->hwseq)
+               dc->hwseq->wa_state.skip_blank_stream = false;
+
        if ((!pipe_ctx->stream->dpms_off || link->link_status.link_active) &&
-               (link->connector_signal == SIGNAL_TYPE_EDP))
+               (link->connector_signal == SIGNAL_TYPE_EDP)) {
                dc->hwss.blank_stream(pipe_ctx);
+               if (dc->hwseq)
+                       dc->hwseq->wa_state.skip_blank_stream = true;
+       }
 
        pipe_ctx->stream_res.tg->funcs->set_dsc_config(
                        pipe_ctx->stream_res.tg,
@@ -570,7 +576,8 @@ static void dcn31_reset_back_end_for_pipe(
                        pipe_ctx->stream_res.audio = NULL;
                }
        }
-
+       if (dc->hwseq)
+               dc->hwseq->wa_state.skip_blank_stream = false;
        pipe_ctx->stream = NULL;
        DC_LOG_DEBUG("Reset back end for pipe %d, tg:%d\n",
                                        pipe_ctx->pipe_idx, 
pipe_ctx->stream_res.tg->inst);
diff --git a/drivers/gpu/drm/amd/display/dc/hwss/hw_sequencer_private.h 
b/drivers/gpu/drm/amd/display/dc/hwss/hw_sequencer_private.h
index 09bc65c2fa23..1e2d247fbbac 100644
--- a/drivers/gpu/drm/amd/display/dc/hwss/hw_sequencer_private.h
+++ b/drivers/gpu/drm/amd/display/dc/hwss/hw_sequencer_private.h
@@ -49,6 +49,7 @@ struct hwseq_wa_state {
        bool DEGVIDCN10_253_applied;
        bool disallow_self_refresh_during_multi_plane_transition_applied;
        unsigned int 
disallow_self_refresh_during_multi_plane_transition_applied_on_frame;
+       bool skip_blank_stream;
 };
 
 struct pipe_ctx;
-- 
2.43.0