drm/i915/vdsc: Remove FIXME in intel_dsc_compute_config
authorSuraj Kandpal <suraj.kandpal@intel.com>
Mon, 26 Jun 2023 13:05:55 +0000 (18:35 +0530)
committerAnkit Nautiyal <ankit.k.nautiyal@intel.com>
Wed, 12 Jul 2023 09:20:54 +0000 (14:50 +0530)
Remove the FIXME and the code related to it as after verification
it does seem the previous values were typos and no hardware spec
mentions using these particular rc_params.

Signed-off-by: Suraj Kandpal <suraj.kandpal@intel.com>
Reviewed-by: Luca Coelho <luciano.coelho@intel.com>
Signed-off-by: Ankit Nautiyal <ankit.k.nautiyal@intel.com>
Link: https://patchwork.freedesktop.org/patch/msgid/20230626130555.2391750-1-suraj.kandpal@intel.com
drivers/gpu/drm/i915/display/intel_vdsc.c

index 79d794786dfb678f764844450c029d0c3fdfda3a..9d76c27567845b4fc54a9dcf5c354fea530f9aed 100644 (file)
@@ -306,18 +306,6 @@ int intel_dsc_compute_params(struct intel_crtc_state *pipe_config)
 
                if (ret)
                        return ret;
-
-               /*
-                * FIXME: verify that the hardware actually needs these
-                * modifications rather than them being simple typos.
-                */
-               if (compressed_bpp == 6 &&
-                   vdsc_cfg->bits_per_component == 8)
-                       vdsc_cfg->rc_quant_incr_limit1 = 23;
-
-               if (compressed_bpp == 8 &&
-                   vdsc_cfg->bits_per_component == 14)
-                       vdsc_cfg->rc_range_params[0].range_bpg_offset = 0;
        }
 
        /*