On Tue, Jul 31, 2018 at 02:07:05PM -0700, Manasi Navare wrote:
> From: "Srivatsa, Anusha" <anusha.sriva...@intel.com>
> 
> DSC has some Rate Control values that remain constant
> across all configurations. These are as per the DSC
> standard.
> 
> v3:
> * Define them in drm_dsc.h as they are
> DSC constants (Manasi)
> v2:
> * Add DP_DSC_ prefix (Jani Nikula)
> 
> Cc: dri-de...@lists.freedesktop.org
> Cc: Manasi Navare <manasi.d.nav...@intel.com>
> Cc: Jani Nikula <jani.nik...@linux.intel.com>
> Cc: Ville Syrjala <ville.syrj...@linux.intel.com>
> Cc: Gaurav K Singh <gaurav.k.si...@intel.com>
> Cc: Harry Wentland <harry.wentl...@amd.com>
> Signed-off-by: Anusha Srivatsa <anusha.sriva...@intel.com>

Tested and double checked the values with DSC spec so
Reviewed-by: Manasi Navare <manasi.d.nav...@intel.com>

> ---
>  include/drm/drm_dsc.h | 6 ++++++
>  1 file changed, 6 insertions(+)
> 
> diff --git a/include/drm/drm_dsc.h b/include/drm/drm_dsc.h
> index eda323d..ebd99d7 100644
> --- a/include/drm/drm_dsc.h
> +++ b/include/drm/drm_dsc.h
> @@ -33,6 +33,12 @@
>  #define DSC_MUX_WORD_SIZE_8_10_BPC   48
>  #define DSC_MUX_WORD_SIZE_12_BPC     64
>  
> +/* DSC Rate Control Constants */
> +#define DSC_RC_MODEL_SIZE_CONST                  8192
> +#define DSC_RC_EDGE_FACTOR_CONST         6
> +#define DSC_RC_TGT_OFFSET_HI_CONST       3
> +#define DSC_RC_TGT_OFFSET_LO_CONST       3
> +
>  /* Configuration for a single Rate Control model range */
>  struct dsc_rc_range_parameters {
>       /* Min Quantization Parameters allowed for this range */
> -- 
> 2.7.4
> 
_______________________________________________
Intel-gfx mailing list
Intel-gfx@lists.freedesktop.org
https://lists.freedesktop.org/mailman/listinfo/intel-gfx

Reply via email to