[Intel-gfx] [PATCH] [intel-gfx] drm/i915/intel_csr.c Added ICL Stepping info.

[Rodrigo Vivi]

On Mon, Sep 03, 2018 at 03:45:13AM -0400, Jyoti Yadav wrote:
> As DMC Package contain DMC FW for multiple steppings including default
> stepping. This patch will help to load FW for that particular stepping,
> if FW for that stepping is available, instead of loading default FW.
> 

Cc: Imre Deak <imre.deak at intel.com>
Cc: Anusha Srivatsa <anusha.srivatsa at intel.com>
> Signed-off-by: Jyoti Yadav <jyoti.r.yadav at intel.com>

I'm not sure if I can properly review this, but based on previous
platforms patch lgtm.

Acked-by: Rodrigo Vivi <rodrigo.vivi at intel.com>

I'm just not pushing because I'd like an ack from Anusha and/or Imre.

> ---
>  drivers/gpu/drm/i915/intel_csr.c | 8 ++++++++
>  1 file changed, 8 insertions(+)
> 
> diff --git a/drivers/gpu/drm/i915/intel_csr.c b/drivers/gpu/drm/i915/intel_csr.c
> index 1ec4f09..f6352ab 100644
> --- a/drivers/gpu/drm/i915/intel_csr.c
> +++ b/drivers/gpu/drm/i915/intel_csr.c
> @@ -183,6 +183,11 @@ struct stepping_info {
>  	{'A', '0'}, {'A', '1'}, {'A', '2'},
>  	{'B', '0'}, {'B', '1'}, {'B', '2'}
>  };

nip by checkpatch: missing blank line here.

> +static const struct stepping_info icl_stepping_info[] = {
> +	{'A', '0'}, {'A', '1'}, {'A', '2'},
> +	{'B', '0'}, {'B', '2'},
> +	{'C', '0'}
> +};
>  
>  static const struct stepping_info no_stepping_info = { '*', '*' };
>  
> @@ -198,6 +203,9 @@ struct stepping_info {
>  	} else if (IS_BROXTON(dev_priv)) {
>  		size = ARRAY_SIZE(bxt_stepping_info);
>  		si = bxt_stepping_info;
> +	} else if (IS_ICELAKE(dev_priv)) {
> +		size = ARRAY_SIZE(icl_stepping_info);
> +		si = icl_stepping_info;
>  	} else {
>  		size = 0;
>  		si = NULL;
> -- 
> 1.9.1
> 
> _______________________________________________
> Intel-gfx mailing list
> Intel-gfx at lists.freedesktop.org
> https://lists.freedesktop.org/mailman/listinfo/intel-gfx