[PATCH 012/120] drm/amd/display: mpcc disconnect and pipe pg in multi-display

[Harry Wentland]

From: Tony Cheng <tony.ch...@amd.com>

still quite hacky.  but this address not properly shutdown pipe video underlay
+ enable another display case, as well as mode changes with video overlay.

Change-Id: I28dde6776bbf05e32324dcad6fbf1e91478c5452
Signed-off-by: Eric Yang <eric.ya...@amd.com>
Reviewed-by: Tony Cheng <tony.ch...@amd.com>
Reviewed-by: Eric Yang <eric.ya...@amd.com>
Acked-by: Harry Wentland <harry.wentl...@amd.com>
---
 .../drm/amd/display/dc/dcn10/dcn10_hw_sequencer.c  | 23 +++++++++++++++++-----
 1 file changed, 18 insertions(+), 5 deletions(-)

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 b5a925362c8c..d607ca94bd13 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
@@ -1669,30 +1669,46 @@ static void dcn10_apply_ctx_for_surface(
                struct core_surface *surface,
                struct validate_context *context)
 {
-       int i;
+       int i, be_idx;
 
        if (dc->public.debug.sanity_checks)
                verify_allow_pstate_change_high(dc->hwseq);
 
+       if (!surface)
+               return;
+
+       for (be_idx = 0; be_idx < dc->res_pool->pipe_count; be_idx++)
+               if (surface == context->res_ctx.pipe_ctx[be_idx].surface)
+                       break;
+
        /* reset unused mpcc */
        for (i = 0; i < dc->res_pool->pipe_count; i++) {
                struct pipe_ctx *pipe_ctx = &context->res_ctx.pipe_ctx[i];
                struct pipe_ctx *old_pipe_ctx =
                                &dc->current_context->res_ctx.pipe_ctx[i];
+
+               if (!pipe_ctx->surface && !old_pipe_ctx->surface)
+                       continue;
+
                /*
                 * Powergate reused pipes that are not powergated
                 * fairly hacky right now, using opp_id as indicator
                 */
+
                if (pipe_ctx->surface && !old_pipe_ctx->surface) {
-                       if (pipe_ctx->mpcc->opp_id != 0xf)
+                       if (pipe_ctx->mpcc->opp_id != 0xf && pipe_ctx->tg->inst 
== be_idx)
                                dcn10_power_down_fe(dc, pipe_ctx->pipe_idx);
                }
 
+
                if ((!pipe_ctx->surface && old_pipe_ctx->surface)
                                || (!pipe_ctx->stream && old_pipe_ctx->stream)) 
{
                        struct mpcc_cfg mpcc_cfg;
                        int opp_id_cached = old_pipe_ctx->mpcc->opp_id;
 
+                       if (old_pipe_ctx->tg->inst != be_idx)
+                               continue;
+
                        if (!old_pipe_ctx->top_pipe) {
                                ASSERT(0);
                                continue;
@@ -1725,9 +1741,6 @@ static void dcn10_apply_ctx_for_surface(
                }
        }
 
-       if (!surface)
-               return;
-
        for (i = 0; i < dc->res_pool->pipe_count; i++) {
                struct pipe_ctx *pipe_ctx = &context->res_ctx.pipe_ctx[i];
 
-- 
2.11.0

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