Re: [Intel-gfx] [PATCH] drm/i915/mtl: Drop Wa_14017240301

[Rodrigo Vivi <rodrigo.vivi@intel.com>]

On Wed, Sep 06, 2023 at 05:03:55PM -0700, Matt Roper wrote:
> Drop Wa_14017240301, which is only relevant to pre-production MTL
> hardware.  Although we usually wait a little bit longer to start
> dropping pre-production workarounds for a platform, it was suggested to
> eliminate this one slightly earlier because it's a bit unusual/ugly:
> this workaround is a display-specific workaround that requires matching
> on the graphics/GT IP version instead of the display IP version.
> 
> Suggested-by: Jani Nikula <jani.nikula@linux.intel.com>
> Signed-off-by: Matt Roper <matthew.d.roper@intel.com>

Reviewed-by: Rodrigo Vivi <rodrigo.vivi@intel.com>

> ---
>  drivers/gpu/drm/i915/display/skl_universal_plane.c | 5 -----
>  1 file changed, 5 deletions(-)
> 
> diff --git a/drivers/gpu/drm/i915/display/skl_universal_plane.c b/drivers/gpu/drm/i915/display/skl_universal_plane.c
> index 517df2aa7a91..007a0bcb3f93 100644
> --- a/drivers/gpu/drm/i915/display/skl_universal_plane.c
> +++ b/drivers/gpu/drm/i915/display/skl_universal_plane.c
> @@ -2170,11 +2170,6 @@ skl_plane_disable_flip_done(struct intel_plane *plane)
>  static bool skl_plane_has_rc_ccs(struct drm_i915_private *i915,
>  				 enum pipe pipe, enum plane_id plane_id)
>  {
> -	/* Wa_14017240301 */
> -	if (IS_GFX_GT_IP_STEP(to_gt(i915), IP_VER(12, 70), STEP_A0, STEP_B0) ||
> -	    IS_GFX_GT_IP_STEP(to_gt(i915), IP_VER(12, 71), STEP_A0, STEP_B0))
> -		return false;
> -
>  	/* Wa_22011186057 */
>  	if (IS_ALDERLAKE_P(i915) && IS_DISPLAY_STEP(i915, STEP_A0, STEP_B0))
>  		return false;
> -- 
> 2.41.0
>