Re: [PATCH] drm/i915/skl: Fix DMC load on Skylake J0 and K0

["Vivi, Rodrigo" <rodrigo.vivi@intel.com>]

First of all:

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

I've checked with DMC folks that DMC 1.23 that we are currently using
and the following releases support J0 and K0.

With this patch we are covering all SKL steppings we know so far, but
besides being concern with new steppings appearing at any moment I'm
mostly concerned about KBL where I just saw a KBL revid=8 and we were
considering revid=0 as KBL H0. So I'm asking myself if we do have
better ways of handling this and preferably killing this table.

In case someone have a good idea please let me know.

But for now we need to move fast with this patch.

Thanks,
Rodrigo.

On Thu, 2016-01-28 at 15:19 -0800, Mat Martineau wrote:
> The driver does not load firmware for unknown steppings, so these new
> steppings must be added to the list.
> 
> Cc: Rodrigo Vivi <rodrigo.vivi@intel.com>
> Signed-off-by: Mat Martineau <mathew.j.martineau@linux.intel.com>
> ---
>  drivers/gpu/drm/i915/intel_csr.c | 3 ++-
>  1 file changed, 2 insertions(+), 1 deletion(-)
> 
> diff --git a/drivers/gpu/drm/i915/intel_csr.c 
> b/drivers/gpu/drm/i915/intel_csr.c
> index 9bb63a8..450bab7 100644
> --- a/drivers/gpu/drm/i915/intel_csr.c
> +++ b/drivers/gpu/drm/i915/intel_csr.c
> @@ -177,7 +177,8 @@ static const struct stepping_info 
> kbl_stepping_info[] = {
>  static const struct stepping_info skl_stepping_info[] = {
>  	{'A', '0'}, {'B', '0'}, {'C', '0'},
>  	{'D', '0'}, {'E', '0'}, {'F', '0'},
> -	{'G', '0'}, {'H', '0'}, {'I', '0'}
> +	{'G', '0'}, {'H', '0'}, {'I', '0'},
> +	{'J', '0'}, {'K', '0'}
>  };
>  
>  static const struct stepping_info bxt_stepping_info[] = {
_______________________________________________
Intel-gfx mailing list
Intel-gfx@lists.freedesktop.org
http://lists.freedesktop.org/mailman/listinfo/intel-gfx