drm/i915/dsc: configure hardware using specified rc_model_size
authorJani Nikula <jani.nikula@intel.com>
Tue, 8 Dec 2020 12:33:51 +0000 (14:33 +0200)
committerJani Nikula <jani.nikula@intel.com>
Thu, 10 Dec 2020 08:48:53 +0000 (10:48 +0200)
The rc_model_size is specified in the DSC config, and the hardware
programming should respect that instead of hard coding a value of 8192.

Regardless, the rc_model_size in DSC config is currently hard coded to
the same value, so this should have no impact, other than allowing the
use of other sizes as needed.

Cc: Manasi Navare <manasi.d.navare@intel.com>
Cc: Vandita Kulkarni <vandita.kulkarni@intel.com>
Reviewed-by: Vandita Kulkarni <vandita.kulkarni@intel.com>
Signed-off-by: Jani Nikula <jani.nikula@intel.com>
Link: https://patchwork.freedesktop.org/patch/msgid/27d86ad25832bbb985f6e996f3d02dca01a66895.1607429866.git.jani.nikula@intel.com
drivers/gpu/drm/i915/display/intel_vdsc.c

index e2716a6..22d0867 100644 (file)
@@ -741,7 +741,7 @@ static void intel_dsc_pps_configure(const struct intel_crtc_state *crtc_state)
 
        /* Populate PICTURE_PARAMETER_SET_9 registers */
        pps_val = 0;
-       pps_val |= DSC_RC_MODEL_SIZE(DSC_RC_MODEL_SIZE_CONST) |
+       pps_val |= DSC_RC_MODEL_SIZE(vdsc_cfg->rc_model_size) |
                DSC_RC_EDGE_FACTOR(DSC_RC_EDGE_FACTOR_CONST);
        drm_info(&dev_priv->drm, "PPS9 = 0x%08x\n", pps_val);
        if (!is_pipe_dsc(crtc_state)) {