[Intel-gfx] [PATCH] drm/i915: Remove pre-production pooled-EU w/a for Broxton

David Weinehall david.weinehall at linux.intel.com
Wed Nov 15 18:01:41 UTC 2017


On Tue, Nov 14, 2017 at 01:51:16PM +0000, Chris Wilson wrote:
> WaEnablePooledEuFor2x6 only applies to preproduction models, unsupported
> since commit 0102ba1fd8af ("drm/i915: Add early BXT sdv to the list of
> preproduction machines").
> 
> Signed-off-by: Chris Wilson <chris at chris-wilson.co.uk>
> Cc: Jani Nikula <jani.nikula at intel.com>

Reviewed-by: David Weinehall <david.weinehall at linux.intel.com>

> ---
>  drivers/gpu/drm/i915/intel_device_info.c | 10 ----------
>  1 file changed, 10 deletions(-)
> 
> diff --git a/drivers/gpu/drm/i915/intel_device_info.c b/drivers/gpu/drm/i915/intel_device_info.c
> index 78bf7374fbdd..f609cfb7f1b2 100644
> --- a/drivers/gpu/drm/i915/intel_device_info.c
> +++ b/drivers/gpu/drm/i915/intel_device_info.c
> @@ -235,16 +235,6 @@ static void gen9_sseu_info_init(struct drm_i915_private *dev_priv)
>  #define IS_SS_DISABLED(ss)	(!(sseu->subslice_mask & BIT(ss)))
>  		info->has_pooled_eu = hweight8(sseu->subslice_mask) == 3;
>  
> -		/*
> -		 * There is a HW issue in 2x6 fused down parts that requires
> -		 * Pooled EU to be enabled as a WA. The pool configuration
> -		 * changes depending upon which subslice is fused down. This
> -		 * doesn't affect if the device has all 3 subslices enabled.
> -		 */
> -		/* WaEnablePooledEuFor2x6:bxt */
> -		info->has_pooled_eu |= (hweight8(sseu->subslice_mask) == 2 &&
> -					IS_BXT_REVID(dev_priv, 0, BXT_REVID_B_LAST));
> -
>  		sseu->min_eu_in_pool = 0;
>  		if (info->has_pooled_eu) {
>  			if (IS_SS_DISABLED(2) || IS_SS_DISABLED(0))
> -- 
> 2.15.0
> 
> _______________________________________________
> Intel-gfx mailing list
> Intel-gfx at lists.freedesktop.org
> https://lists.freedesktop.org/mailman/listinfo/intel-gfx


More information about the Intel-gfx mailing list