First of all: Reviewed-by: Rodrigo Vivi <rodrigo.vivi@xxxxxxxxx> 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@xxxxxxxxx> > Signed-off-by: Mat Martineau <mathew.j.martineau@xxxxxxxxxxxxxxx> > --- > 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@xxxxxxxxxxxxxxxxxxxxx http://lists.freedesktop.org/mailman/listinfo/intel-gfx