drm/amd/display: refactor the codes to centralize the stream/pipe checking logic
authorRoy Chan <roy.chan@amd.com>
Wed, 21 Jul 2021 23:35:28 +0000 (19:35 -0400)
committerAlex Deucher <alexander.deucher@amd.com>
Mon, 9 Aug 2021 19:42:56 +0000 (15:42 -0400)
Acked-by: Anson Jacob <Anson.Jacob@amd.com>
Signed-off-by: Roy Chan <roy.chan@amd.com>
Tested-by: Daniel Wheeler <daniel.wheeler@amd.com>
Signed-off-by: Alex Deucher <alexander.deucher@amd.com>
drivers/gpu/drm/amd/display/dc/core/dc.c

index 605e297b7a591933268c7771a9fd9b7a2f7bf019..7a442fcfa6aca46914275c96ab7b19cee5c2255b 100644 (file)
@@ -1481,6 +1481,22 @@ bool dc_validate_seamless_boot_timing(const struct dc *dc,
        return true;
 }
 
+static inline bool should_update_pipe_for_stream(
+               struct dc_state *context,
+               struct pipe_ctx *pipe_ctx,
+               struct dc_stream_state *stream)
+{
+       return (pipe_ctx->stream && pipe_ctx->stream == stream);
+}
+
+static inline bool should_update_pipe_for_plane(
+               struct dc_state *context,
+               struct pipe_ctx *pipe_ctx,
+               struct dc_plane_state *plane_state)
+{
+       return (pipe_ctx->plane_state == plane_state);
+}
+
 void dc_enable_stereo(
        struct dc *dc,
        struct dc_state *context,
@@ -1491,12 +1507,15 @@ void dc_enable_stereo(
        struct pipe_ctx *pipe;
 
        for (i = 0; i < MAX_PIPES; i++) {
-               if (context != NULL)
+               if (context != NULL) {
                        pipe = &context->res_ctx.pipe_ctx[i];
-               else
+               } else {
+                       context = dc->current_state;
                        pipe = &dc->current_state->res_ctx.pipe_ctx[i];
-               for (j = 0 ; pipe && j < stream_count; j++)  {
-                       if (streams[j] && streams[j] == pipe->stream &&
+               }
+
+               for (j = 0; pipe && j < stream_count; j++)  {
+                       if (should_update_pipe_for_stream(context, pipe, streams[j]) &&
                                dc->hwss.setup_stereo)
                                dc->hwss.setup_stereo(pipe, dc);
                }
@@ -2623,6 +2642,7 @@ static void commit_planes_for_stream(struct dc *dc,
 {
        int i, j;
        struct pipe_ctx *top_pipe_to_program = NULL;
+       bool should_lock_all_pipes = (update_type != UPDATE_TYPE_FAST);
 
 #if defined(CONFIG_DRM_AMD_DC_DCN)
        dc_z10_restore(dc);
@@ -2694,7 +2714,7 @@ static void commit_planes_for_stream(struct dc *dc,
                                                top_pipe_to_program->stream_res.tg);
                }
 
-       if ((update_type != UPDATE_TYPE_FAST) && dc->hwss.interdependent_update_lock)
+       if (should_lock_all_pipes && dc->hwss.interdependent_update_lock)
                dc->hwss.interdependent_update_lock(dc, context, true);
        else
                /* Lock the top pipe while updating plane addrs, since freesync requires
@@ -2717,7 +2737,7 @@ static void commit_planes_for_stream(struct dc *dc,
                if (dc->hwss.program_front_end_for_ctx)
                        dc->hwss.program_front_end_for_ctx(dc, context);
 
-               if ((update_type != UPDATE_TYPE_FAST) && dc->hwss.interdependent_update_lock)
+               if (should_lock_all_pipes && dc->hwss.interdependent_update_lock)
                        dc->hwss.interdependent_update_lock(dc, context, false);
                else
                        dc->hwss.pipe_control_lock(dc, top_pipe_to_program, false);
@@ -2733,14 +2753,14 @@ static void commit_planes_for_stream(struct dc *dc,
                                struct pipe_ctx *pipe_ctx = &context->res_ctx.pipe_ctx[j];
                                if (!pipe_ctx->plane_state)
                                        continue;
-                               if (pipe_ctx->plane_state != plane_state)
+                               if (should_update_pipe_for_plane(context, pipe_ctx, plane_state))
                                        continue;
-                               plane_state->triplebuffer_flips = false;
+                               pipe_ctx->plane_state->triplebuffer_flips = false;
                                if (update_type == UPDATE_TYPE_FAST &&
                                        dc->hwss.program_triplebuffer != NULL &&
-                                       !plane_state->flip_immediate && dc->debug.enable_tri_buf) {
+                                       !pipe_ctx->plane_state->flip_immediate && dc->debug.enable_tri_buf) {
                                                /*triple buffer for VUpdate  only*/
-                                               plane_state->triplebuffer_flips = true;
+                                               pipe_ctx->plane_state->triplebuffer_flips = true;
                                }
                        }
                        if (update_type == UPDATE_TYPE_FULL) {
@@ -2756,8 +2776,7 @@ static void commit_planes_for_stream(struct dc *dc,
 
                if (!pipe_ctx->top_pipe &&
                        !pipe_ctx->prev_odm_pipe &&
-                       pipe_ctx->stream &&
-                       pipe_ctx->stream == stream) {
+                       should_update_pipe_for_stream(context, pipe_ctx, stream)) {
                        struct dc_stream_status *stream_status = NULL;
 
                        if (!pipe_ctx->plane_state)
@@ -2810,15 +2829,15 @@ static void commit_planes_for_stream(struct dc *dc,
                                for (j = 0; j < dc->res_pool->pipe_count; j++) {
                                        struct pipe_ctx *pipe_ctx = &context->res_ctx.pipe_ctx[j];
 
-                                       if (pipe_ctx->stream != stream)
+                                       if (!should_update_pipe_for_stream(context, pipe_ctx, stream))
                                                continue;
 
-                                       if (pipe_ctx->plane_state != plane_state)
+                                       if (!should_update_pipe_for_plane(context, pipe_ctx, plane_state))
                                                continue;
 
                                        // GSL has to be used for flip immediate
                                        dc->hwss.set_flip_control_gsl(pipe_ctx,
-                                                       plane_state->flip_immediate);
+                                                       pipe_ctx->plane_state->flip_immediate);
                                }
                        }
 
@@ -2829,25 +2848,26 @@ static void commit_planes_for_stream(struct dc *dc,
                        for (j = 0; j < dc->res_pool->pipe_count; j++) {
                                struct pipe_ctx *pipe_ctx = &context->res_ctx.pipe_ctx[j];
 
-                               if (pipe_ctx->stream != stream)
+                               if (!should_update_pipe_for_stream(context, pipe_ctx, stream))
                                        continue;
 
-                               if (pipe_ctx->plane_state != plane_state)
+                               if (!should_update_pipe_for_plane(context, pipe_ctx, plane_state))
                                        continue;
+
                                /*program triple buffer after lock based on flip type*/
                                if (dc->hwss.program_triplebuffer != NULL && dc->debug.enable_tri_buf) {
                                        /*only enable triplebuffer for  fast_update*/
                                        dc->hwss.program_triplebuffer(
-                                               dc, pipe_ctx, plane_state->triplebuffer_flips);
+                                               dc, pipe_ctx, pipe_ctx->plane_state->triplebuffer_flips);
                                }
-                               if (srf_updates[i].flip_addr)
+                               if (pipe_ctx->plane_state->update_flags.bits.addr_update)
                                        dc->hwss.update_plane_addr(dc, pipe_ctx);
                        }
                }
 
        }
 
-       if ((update_type != UPDATE_TYPE_FAST) && dc->hwss.interdependent_update_lock)
+       if (should_lock_all_pipes && dc->hwss.interdependent_update_lock)
                dc->hwss.interdependent_update_lock(dc, context, false);
        else
                dc->hwss.pipe_control_lock(dc, top_pipe_to_program, false);
@@ -2891,7 +2911,7 @@ static void commit_planes_for_stream(struct dc *dc,
                        continue;
 
                if (pipe_ctx->bottom_pipe || pipe_ctx->next_odm_pipe ||
-                               !pipe_ctx->stream || pipe_ctx->stream != stream ||
+                               !pipe_ctx->stream || !should_update_pipe_for_stream(context, pipe_ctx, stream) ||
                                !pipe_ctx->plane_state->update_flags.bits.addr_update ||
                                pipe_ctx->plane_state->skip_manual_trigger)
                        continue;