[Intel-gfx] [PATCH 1/5] drm/i915: Remove CNL from WA 827

[Clint Taylor]

On 10/29/2018 04:00 PM, Rodrigo Vivi wrote:
> CNL A stepping was the only affected there.
> But also it is time to clean old pre-production
> CNL Workarounds, so let's just remove and clean
> this W/A.
>
> Cc: Maarten Lankhorst <maarten.lankhorst at linux.intel.com>
> Cc: Ville Syrjälä <ville.syrjala at linux.intel.com>
> Signed-off-by: Rodrigo Vivi <rodrigo.vivi at intel.com>
> ---
>   drivers/gpu/drm/i915/intel_display.c | 4 ++--
>   1 file changed, 2 insertions(+), 2 deletions(-)
>
> diff --git a/drivers/gpu/drm/i915/intel_display.c b/drivers/gpu/drm/i915/intel_display.c
> index c3cadc09f859..dd88ffe9e273 100644
> --- a/drivers/gpu/drm/i915/intel_display.c
> +++ b/drivers/gpu/drm/i915/intel_display.c
> @@ -5242,8 +5242,8 @@ static bool needs_nv12_wa(struct drm_i915_private *dev_priv,
>   	if (IS_SKYLAKE(dev_priv) || IS_BROXTON(dev_priv))
>   		return false;
>   
> -	if ((IS_GEN9(dev_priv) && !IS_GEMINILAKE(dev_priv)) ||
> -	    IS_CANNONLAKE(dev_priv))
> +	/* WA Display #0827: Gen9:all */
> +	if (IS_GEN9(dev_priv) && !IS_GEMINILAKE(dev_priv))
>   		return true;
>   
>   	return false;

Looks good.
Reviewed-by: Clint Taylor <Clinton.A.Taylor at intel.com>