drm/amd/display: Remove duplicate configuration

The function that commits planes calls the same set of functions twice,
and in the case of the FAMs utilization, it is not desired to call the
dmub, hwss_build and hwss_execute. This commit just removes the
unnecessary calls to those functions.

Acked-by: Roman Li <roman.li@amd.com>
Signed-off-by: Rodrigo Siqueira <Rodrigo.Siqueira@amd.com>
Tested-by: Daniel Wheeler <daniel.wheeler@amd.com>
Signed-off-by: Alex Deucher <alexander.deucher@amd.com>

diff --git a/drivers/gpu/drm/amd/display/dc/core/dc.c b/drivers/gpu/drm/amd/display/dc/core/dc.c
index 6f534c2e76b7dded2a97766a65977ae2dc96afbe..ae04937e60b0078be1afff073d6128d36c989a99 100644 (file)
--- a/drivers/gpu/drm/amd/display/dc/core/dc.c
+++ b/drivers/gpu/drm/amd/display/dc/core/dc.c
@@ -3741,24 +3741,6 @@ static void commit_planes_for_stream_fast(struct dc *dc,
                                context->block_sequence_steps);
        }
 
-       build_dmub_cmd_list(dc,
-                       srf_updates,
-                       surface_count,
-                       stream,
-                       context,
-                       context->dc_dmub_cmd,
-                       &(context->dmub_cmd_count));
-       hwss_build_fast_sequence(dc,
-                       context->dc_dmub_cmd,
-                       context->dmub_cmd_count,
-                       context->block_sequence,
-                       &(context->block_sequence_steps),
-                       top_pipe_to_program,
-                       stream_status,
-                       context);
-       hwss_execute_sequence(dc,
-                       context->block_sequence,
-                       context->block_sequence_steps);
        /* Clear update flags so next flip doesn't have redundant programming
         * (if there's no stream update, the update flags are not cleared).
         * Surface updates are cleared unconditionally at the beginning of each flip,