Re: [PATCH 0/5] drm/i915/skl: drop pre-production stepping workarounds

[Paulo Zanoni <paulo.r.zanoni@intel.com>]

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.

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

> 
> BR,
> Jani.
> 
> 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