On Fri, Jul 23, 2021 at 05:11:01PM -0700, Lucas De Marchi wrote:
> With the removal of CNL, let's consider GLK as the first platform using
> those constants since GLK has DISPLAY_VER == 10.
> 
> Signed-off-by: Lucas De Marchi <lucas.demar...@intel.com>

Reviewed-by: Rodrigo Vivi <rodrigo.v...@intel.com>

> ---
>  drivers/gpu/drm/i915/display/skl_scaler.c | 10 +++++-----
>  drivers/gpu/drm/i915/i915_reg.h           |  4 ++--
>  2 files changed, 7 insertions(+), 7 deletions(-)
> 
> diff --git a/drivers/gpu/drm/i915/display/skl_scaler.c 
> b/drivers/gpu/drm/i915/display/skl_scaler.c
> index 911a113ee006..ebdd3115de16 100644
> --- a/drivers/gpu/drm/i915/display/skl_scaler.c
> +++ b/drivers/gpu/drm/i915/display/skl_scaler.c
> @@ -341,12 +341,12 @@ static u16 cnl_nearest_filter_coef(int t)
>   *
>   */
>  
> -static void cnl_program_nearest_filter_coefs(struct drm_i915_private 
> *dev_priv,
> +static void glk_program_nearest_filter_coefs(struct drm_i915_private 
> *dev_priv,
>                                            enum pipe pipe, int id, int set)
>  {
>       int i;
>  
> -     intel_de_write_fw(dev_priv, CNL_PS_COEF_INDEX_SET(pipe, id, set),
> +     intel_de_write_fw(dev_priv, GLK_PS_COEF_INDEX_SET(pipe, id, set),
>                         PS_COEE_INDEX_AUTO_INC);
>  
>       for (i = 0; i < 17 * 7; i += 2) {
> @@ -359,11 +359,11 @@ static void cnl_program_nearest_filter_coefs(struct 
> drm_i915_private *dev_priv,
>               t = cnl_coef_tap(i + 1);
>               tmp |= cnl_nearest_filter_coef(t) << 16;
>  
> -             intel_de_write_fw(dev_priv, CNL_PS_COEF_DATA_SET(pipe, id, set),
> +             intel_de_write_fw(dev_priv, GLK_PS_COEF_DATA_SET(pipe, id, set),
>                                 tmp);
>       }
>  
> -     intel_de_write_fw(dev_priv, CNL_PS_COEF_INDEX_SET(pipe, id, set), 0);
> +     intel_de_write_fw(dev_priv, GLK_PS_COEF_INDEX_SET(pipe, id, set), 0);
>  }
>  
>  static u32 skl_scaler_get_filter_select(enum drm_scaling_filter filter, int 
> set)
> @@ -386,7 +386,7 @@ static void skl_scaler_setup_filter(struct 
> drm_i915_private *dev_priv, enum pipe
>       case DRM_SCALING_FILTER_DEFAULT:
>               break;
>       case DRM_SCALING_FILTER_NEAREST_NEIGHBOR:
> -             cnl_program_nearest_filter_coefs(dev_priv, pipe, id, set);
> +             glk_program_nearest_filter_coefs(dev_priv, pipe, id, set);
>               break;
>       default:
>               MISSING_CASE(filter);
> diff --git a/drivers/gpu/drm/i915/i915_reg.h b/drivers/gpu/drm/i915/i915_reg.h
> index 91e93f3e9649..d198b1a2d4b5 100644
> --- a/drivers/gpu/drm/i915/i915_reg.h
> +++ b/drivers/gpu/drm/i915/i915_reg.h
> @@ -7726,11 +7726,11 @@ enum {
>  #define SKL_PS_ECC_STAT(pipe, id)  _MMIO_PIPE(pipe,     \
>                       _ID(id, _PS_ECC_STAT_1A, _PS_ECC_STAT_2A),   \
>                       _ID(id, _PS_ECC_STAT_1B, _PS_ECC_STAT_2B))
> -#define CNL_PS_COEF_INDEX_SET(pipe, id, set)  _MMIO_PIPE(pipe,    \
> +#define GLK_PS_COEF_INDEX_SET(pipe, id, set)  _MMIO_PIPE(pipe,    \
>                       _ID(id, _PS_COEF_SET0_INDEX_1A, _PS_COEF_SET0_INDEX_2A) 
> + (set) * 8, \
>                       _ID(id, _PS_COEF_SET0_INDEX_1B, _PS_COEF_SET0_INDEX_2B) 
> + (set) * 8)
>  
> -#define CNL_PS_COEF_DATA_SET(pipe, id, set)  _MMIO_PIPE(pipe,     \
> +#define GLK_PS_COEF_DATA_SET(pipe, id, set)  _MMIO_PIPE(pipe,     \
>                       _ID(id, _PS_COEF_SET0_DATA_1A, _PS_COEF_SET0_DATA_2A) + 
> (set) * 8, \
>                       _ID(id, _PS_COEF_SET0_DATA_1B, _PS_COEF_SET0_DATA_2B) + 
> (set) * 8)
>  /* legacy palette */
> -- 
> 2.31.1
> 
_______________________________________________
Intel-gfx mailing list
Intel-gfx@lists.freedesktop.org
https://lists.freedesktop.org/mailman/listinfo/intel-gfx

Reply via email to