On Wed, Mar 23, 2016 at 06:19:54PM +0000, Dave Gordon wrote:
> @@ -2023,8 +2025,11 @@ static inline struct drm_i915_private 
> *guc_to_i915(struct intel_guc *guc)
>  
>  /* Iterator over subset of engines selected by mask */
>  #define for_each_engine_masked(engine__, dev_priv__, mask__) \
> -     for ((engine__) = &dev_priv->engine[0]; (engine__) < 
> &dev_priv->engine[I915_NUM_ENGINES]; (engine__)++) \
> -             for_each_if (intel_engine_flag((engine__)) & (mask__) && 
> intel_engine_initialized((engine__)))
> +     for ((engine__) = &dev_priv->engine[0]; \
> +          (engine__) < &dev_priv->engine[I915_NUM_ENGINES]; \
> +          (engine__)++) \
> +             for_each_if (((mask__) & intel_engine_flag(engine__)) && \
> +                          intel_engine_initialized(engine__))

You've touched it, might as well fix it!
        s/dev_priv/(dev_priv__)/

With the macro fixed, both patches
Reviewed-by: Chris Wilson <[email protected]>
-Chris

-- 
Chris Wilson, Intel Open Source Technology Centre
_______________________________________________
Intel-gfx mailing list
[email protected]
https://lists.freedesktop.org/mailman/listinfo/intel-gfx

Reply via email to