On Thu, 22 Sep 2016, Paulo Zanoni <paulo.r.zanoni@xxxxxxxxx> wrote: > 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@xxxxxxxxx> Pushed patches 1-5, thanks for the review, and sent a separate patch to address the bikesheds [1], along with a resend of your pre-production hardware warning patch [2]. BR, Jani. [1] http://patchwork.freedesktop.org/patch/msgid/1474891672-23414-1-git-send-email-jani.nikula@xxxxxxxxx [2] http://patchwork.freedesktop.org/patch/msgid/1474891672-23414-2-git-send-email-jani.nikula@xxxxxxxxx > >> >> 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(-) >> -- Jani Nikula, Intel Open Source Technology Center _______________________________________________ Intel-gfx mailing list Intel-gfx@xxxxxxxxxxxxxxxxxxxxx https://lists.freedesktop.org/mailman/listinfo/intel-gfx
