drm/i915: Fix application of WaInPlaceDecompressionHang
authorMatt Roper <matthew.d.roper@intel.com>
Sat, 17 Jul 2021 05:14:20 +0000 (22:14 -0700)
committerMatt Roper <matthew.d.roper@intel.com>
Wed, 21 Jul 2021 17:44:10 +0000 (10:44 -0700)
commitf4fa096ad4c288d80a8ac89b5cc4861e68e8c435
treed5214241743148de9779fc7536e2740cec1d5ecb
parent7961c5b60f23dff5d82a523f9aeb8ebf34cf9926
drm/i915: Fix application of WaInPlaceDecompressionHang

On SKL we've been applying this workaround on H0+ steppings, which is
actually backwards; H0 is supposed to be the first stepping where the
workaround is no longer needed.  Flip the bounds so that the workaround
applies to all steppings _before_ H0.

On BXT we've been applying this workaround to all steppings, but the
bspec tells us it's only needed until C0.  Pre-C0 GT steppings only
appeared in pre-production hardware, which we no longer support in the
driver, so we can drop the workaround completely for this platform.

On ICL we've been applying this workaround to all steppings, but there
doesn't seem to be any indication that this workaround was ever needed
for this platform (even now-deprecated page 20196 of the bspec doesn't
mention it).  We can go ahead and drop it.

I also don't see any mention of this workaround being needed for KBL,
although this may be an oversight since the workaround is needed for all
steppings of CFL.  I'll leave the workaround in place for KBL to be
safe.

Bspec: 14091, 33450
Signed-off-by: Matt Roper <matthew.d.roper@intel.com>
Reviewed-by: José Roberto de Souza <jose.souza@intel.com>
Link: https://patchwork.freedesktop.org/patch/msgid/20210717051426.4120328-2-matthew.d.roper@intel.com
drivers/gpu/drm/i915/gt/intel_workarounds.c