drm/amd/display: Cleanup unused SetPlaneConfig
authorAnthony Koo <Anthony.Koo@amd.com>
Fri, 13 Apr 2018 02:40:02 +0000 (22:40 -0400)
committerAlex Deucher <alexander.deucher@amd.com>
Fri, 18 May 2018 21:08:21 +0000 (16:08 -0500)
Signed-off-by: Anthony Koo <Anthony.Koo@amd.com>
Reviewed-by: Aric Cyr <Aric.Cyr@amd.com>
Acked-by: Harry Wentland <harry.wentland@amd.com>
Signed-off-by: Alex Deucher <alexander.deucher@amd.com>
drivers/gpu/drm/amd/display/dc/dce110/dce110_hw_sequencer.c
drivers/gpu/drm/amd/display/dc/dcn10/dcn10_hw_sequencer.c
drivers/gpu/drm/amd/display/dc/inc/hw_sequencer.h

index 2288d0aa773b70778b3164bdf61154e89e316bd1..ae500421edb6e49df39590690246117f341c1856 100644 (file)
@@ -2269,74 +2269,6 @@ static void program_gamut_remap(struct pipe_ctx *pipe_ctx)
 
        pipe_ctx->plane_res.xfm->funcs->transform_set_gamut_remap(pipe_ctx->plane_res.xfm, &adjust);
 }
-
-/**
- * TODO REMOVE, USE UPDATE INSTEAD
- */
-static void set_plane_config(
-       const struct dc *dc,
-       struct pipe_ctx *pipe_ctx,
-       struct resource_context *res_ctx)
-{
-       struct mem_input *mi = pipe_ctx->plane_res.mi;
-       struct dc_plane_state *plane_state = pipe_ctx->plane_state;
-       struct xfm_grph_csc_adjustment adjust;
-       struct out_csc_color_matrix tbl_entry;
-       unsigned int i;
-
-       memset(&adjust, 0, sizeof(adjust));
-       memset(&tbl_entry, 0, sizeof(tbl_entry));
-       adjust.gamut_adjust_type = GRAPHICS_GAMUT_ADJUST_TYPE_BYPASS;
-
-       dce_enable_fe_clock(dc->hwseq, mi->inst, true);
-
-       set_default_colors(pipe_ctx);
-       if (pipe_ctx->stream->csc_color_matrix.enable_adjustment == true) {
-               tbl_entry.color_space =
-                       pipe_ctx->stream->output_color_space;
-
-               for (i = 0; i < 12; i++)
-                       tbl_entry.regval[i] =
-                       pipe_ctx->stream->csc_color_matrix.matrix[i];
-
-               pipe_ctx->plane_res.xfm->funcs->opp_set_csc_adjustment
-                               (pipe_ctx->plane_res.xfm, &tbl_entry);
-       }
-
-       if (pipe_ctx->stream->gamut_remap_matrix.enable_remap == true) {
-               adjust.gamut_adjust_type = GRAPHICS_GAMUT_ADJUST_TYPE_SW;
-
-               for (i = 0; i < CSC_TEMPERATURE_MATRIX_SIZE; i++)
-                       adjust.temperature_matrix[i] =
-                               pipe_ctx->stream->gamut_remap_matrix.matrix[i];
-       }
-
-       pipe_ctx->plane_res.xfm->funcs->transform_set_gamut_remap(pipe_ctx->plane_res.xfm, &adjust);
-
-       pipe_ctx->plane_res.scl_data.lb_params.alpha_en = pipe_ctx->bottom_pipe != 0;
-       program_scaler(dc, pipe_ctx);
-
-       program_surface_visibility(dc, pipe_ctx);
-
-       mi->funcs->mem_input_program_surface_config(
-                       mi,
-                       plane_state->format,
-                       &plane_state->tiling_info,
-                       &plane_state->plane_size,
-                       plane_state->rotation,
-                       NULL,
-                       false);
-       if (mi->funcs->set_blank)
-               mi->funcs->set_blank(mi, pipe_ctx->plane_state->visible);
-
-       if (dc->config.gpu_vm_support)
-               mi->funcs->mem_input_program_pte_vm(
-                               pipe_ctx->plane_res.mi,
-                               plane_state->format,
-                               &plane_state->tiling_info,
-                               plane_state->rotation);
-}
-
 static void update_plane_addr(const struct dc *dc,
                struct pipe_ctx *pipe_ctx)
 {
@@ -3023,7 +2955,6 @@ static const struct hw_sequencer_funcs dce110_funcs = {
        .init_hw = init_hw,
        .apply_ctx_to_hw = dce110_apply_ctx_to_hw,
        .apply_ctx_for_surface = dce110_apply_ctx_for_surface,
-       .set_plane_config = set_plane_config,
        .update_plane_addr = update_plane_addr,
        .update_pending_status = dce110_update_pending_status,
        .set_input_transfer_func = dce110_set_input_transfer_func,
index 572fa601a0ebd13129a20bd2ec0922d8ebfb0802..8adb8dc44af54856f0cff71d76747e113c26307e 100644 (file)
@@ -2487,15 +2487,6 @@ static void set_static_screen_control(struct pipe_ctx **pipe_ctx,
                        set_static_screen_control(pipe_ctx[i]->stream_res.tg, value);
 }
 
-static void set_plane_config(
-       const struct dc *dc,
-       struct pipe_ctx *pipe_ctx,
-       struct resource_context *res_ctx)
-{
-       /* TODO */
-       program_gamut_remap(pipe_ctx);
-}
-
 static void dcn10_config_stereo_parameters(
                struct dc_stream_state *stream, struct crtc_stereo_flags *flags)
 {
@@ -2673,7 +2664,6 @@ static const struct hw_sequencer_funcs dcn10_funcs = {
        .init_hw = dcn10_init_hw,
        .apply_ctx_to_hw = dce110_apply_ctx_to_hw,
        .apply_ctx_for_surface = dcn10_apply_ctx_for_surface,
-       .set_plane_config = set_plane_config,
        .update_plane_addr = dcn10_update_plane_addr,
        .update_dchub = dcn10_update_dchub,
        .update_pending_status = dcn10_update_pending_status,
index 29abf3ecb39c288ecebe27de9224cbc4004f29eb..63fc6c4997892d73b1bb6c25b3261d5ac22b8ee8 100644 (file)
@@ -83,11 +83,6 @@ struct hw_sequencer_funcs {
                        int num_planes,
                        struct dc_state *context);
 
-       void (*set_plane_config)(
-                       const struct dc *dc,
-                       struct pipe_ctx *pipe_ctx,
-                       struct resource_context *res_ctx);
-
        void (*program_gamut_remap)(
                        struct pipe_ctx *pipe_ctx);