From 4a74f4b7a827bde2467e7910c4c4baf51c5a6d98 Mon Sep 17 00:00:00 2001 From: Wenjing Liu Date: Tue, 26 Mar 2019 14:36:52 -0400 Subject: [PATCH 2485/2940] drm/amd/display: remove target_dpp hack for dsc Remove dc_dsc hack for MST policy Signed-off-by: Wenjing Liu Reviewed-by: Nikola Cornij Acked-by: Bhawanpreet Lakha Signed-off-by: Alex Deucher --- drivers/gpu/drm/amd/display/dc/dsc/dc_dsc.c | 27 +++++---------------- 1 file changed, 6 insertions(+), 21 deletions(-) diff --git a/drivers/gpu/drm/amd/display/dc/dsc/dc_dsc.c b/drivers/gpu/drm/amd/display/dc/dsc/dc_dsc.c index fe63af836f1e..e71197d35d79 100644 --- a/drivers/gpu/drm/amd/display/dc/dsc/dc_dsc.c +++ b/drivers/gpu/drm/amd/display/dc/dsc/dc_dsc.c @@ -490,7 +490,7 @@ static int fit_num_slices_up(union dsc_enc_slice_caps slice_caps, int num_slices * dsc_enc_caps - DSC encoder capabilities * * target_bandwidth - Target bandwidth to fit the stream into. - * If 0, use maximum compression as per DSC policy. + * If 0, do not calculate target bpp. * * timing - The stream timing to fit into 'target_bandwidth' or apply * maximum compression to, if 'target_badwidth == 0' @@ -517,7 +517,6 @@ static bool setup_dsc_config( int slice_width; int target_bpp; int sink_per_slice_throughput; - // TODO DSC: See if it makes sense to use 2.4% for SST bool is_dsc_possible = false; int num_slices_v; int pic_height; @@ -534,20 +533,11 @@ static bool setup_dsc_config( if (target_bandwidth > 0) { is_dsc_possible = decide_dsc_target_bpp_x16(&dsc_policy, &dsc_common_caps, target_bandwidth, timing, &target_bpp); - } else if (timing->pix_clk_100hz * 12 <= dc_bandwidth_in_kbps_from_timing(timing) * 10) { - /* use 12 target bpp for MST display - * TODO: implement new MST DSC target bpp policy */ - target_bpp = 16*12; - is_dsc_possible = true; - } else { - is_dsc_possible = false; + dsc_cfg->bits_per_pixel = target_bpp; } - if (!is_dsc_possible) goto done; - dsc_cfg->bits_per_pixel = target_bpp; - sink_per_slice_throughput = 0; // Validate available DSC settings against the mode timing @@ -773,18 +763,13 @@ bool dc_dsc_compute_config( struct dc_dsc_config *dsc_cfg) { bool is_dsc_possible = false; - struct dsc_enc_caps dsc_enc_caps; - struct dsc_enc_caps dsc_common_caps; get_dsc_enc_caps(dc, &dsc_enc_caps, timing->pix_clk_100hz); - is_dsc_possible = dc_intersect_dsc_caps(dsc_sink_caps, &dsc_enc_caps, - timing->pixel_encoding, &dsc_common_caps); - if (is_dsc_possible) - is_dsc_possible = setup_dsc_config(dsc_sink_caps, - &dsc_enc_caps, - target_bandwidth, - timing, dsc_cfg); + is_dsc_possible = setup_dsc_config(dsc_sink_caps, + &dsc_enc_caps, + target_bandwidth, + timing, dsc_cfg); return is_dsc_possible; } #endif /* CONFIG_DRM_AMD_DC_DSC_SUPPORT */ -- 2.17.1