> From: Zanoni, Paulo R
> Em Sex, 2016-09-16 às 16:59 +0300, Jani Nikula escreveu:
> > Only production steppings are supported, drop workarounds for
> > anything
> > else. The series is split by revision so we can bikeshed if there are
> > steppings some people still need to use for some reason.
> 
> Bikeshed: in patches 2 and 3 you could have added platform tags to the
> workaround tags, while also adding the missing space to a
> /* comment*/.
> 
> Jani S., Yann: perhaps we could try to check if our CI/QA systems still
> have these machines? Just "lspci -nn | grep VGA" on the SKL systems and
> check whether rev <= 5.

The CI system doesn't have any pre-production SKL machines. They were dropped 
as soon as we got production machines.

Tomi

> If we conclude our CI system doesn't include these machines:
> Reviewed-by: Paulo Zanoni <paulo.r.zan...@intel.com>

> > Jani Nikula (5):
> >   drm/i915/skl: drop workarounds for A0 and B0 revisions
> >   drm/i915/skl: drop workarounds for C0 revision
> >   drm/i915/skl: drop workarounds for D0 revision
> >   drm/i915/skl: drop workarounds for E0 revision
> >   drm/i915/skl: drop workarounds for F0 revision
> >
> >  drivers/gpu/drm/i915/intel_dp.c               |  4 --
> >  drivers/gpu/drm/i915/intel_dp_link_training.c |  3 --
> >  drivers/gpu/drm/i915/intel_guc_loader.c       |  8 ++--
> >  drivers/gpu/drm/i915/intel_lrc.c              | 23 +++++------
> >  drivers/gpu/drm/i915/intel_pm.c               |  3 +-
> >  drivers/gpu/drm/i915/intel_ringbuffer.c       | 58 +++++----------
> > ------------
> >  6 files changed, 23 insertions(+), 76 deletions(-)
_______________________________________________
Intel-gfx mailing list
Intel-gfx@lists.freedesktop.org
https://lists.freedesktop.org/mailman/listinfo/intel-gfx

Reply via email to