[Intel-gfx] [PATCH 08/10] drm/i915: Create generic function to setup ring frequency table

[Chris Wilson]

Quoting Sagar Arun Kamble (2017-10-04 15:07:23)
> Prepared intel_update_ring_freq function to setup ring frequency
> for applicable platforms determined by macro - NEEDS_RING_FREQ_UPDATE
> 
> Signed-off-by: Sagar Arun Kamble <sagar.a.kamble at intel.com>
> Cc: Imre Deak <imre.deak at intel.com>
> Cc: Chris Wilson <chris at chris-wilson.co.uk>
> Cc: Joonas Lahtinen <joonas.lahtinen at linux.intel.com>
> Cc: Radoslaw Szwichtenberg <radoslaw.szwichtenberg at intel.com>
> ---
>  drivers/gpu/drm/i915/intel_pm.c | 19 +++++++++++++++----
>  1 file changed, 15 insertions(+), 4 deletions(-)
> 
> diff --git a/drivers/gpu/drm/i915/intel_pm.c b/drivers/gpu/drm/i915/intel_pm.c
> index 1ccc3fb..62aed72 100644
> --- a/drivers/gpu/drm/i915/intel_pm.c
> +++ b/drivers/gpu/drm/i915/intel_pm.c
> @@ -7868,6 +7868,19 @@ static void intel_init_emon(struct drm_i915_private *dev_priv)
>         dev_priv->ips.corr = (lcfuse & LCFUSE_HIV_MASK);
>  }
>  
> +#define NEEDS_RING_FREQ_UPDATE(i915) \
> +          (((INTEL_GEN(i915) >= 9) && \
> +            (IS_GEN9_BC(i915) || IS_CANNONLAKE(i915))) || \
> +           (IS_BROADWELL(i915)) || \
> +           ((INTEL_GEN(i915) >= 6) && \
> +            (!IS_CHERRYVIEW(i915) && !IS_VALLEYVIEW(i915))))

Let's not contort ourselves into using a macro where gcc will compile a
function just as tightly.

Compare modifying a more verbose function than this macro for the next
BC/LP.
-Chris