[Intel-gfx] [PATCH 1/2] drm/i915/i915_drv: Use proper parameter naming in for_each_gt()
Andrzej Hajda
andrzej.hajda at intel.com
Fri Apr 21 15:26:12 UTC 2023
On 21.04.2023 15:46, Andi Shyti wrote:
> for_each_gt() loops through engines in the GT, not in dev_priv.
> Because it's misleading, call it "gt__" instead of "dev_priv__".
>
> Signed-off-by: Andi Shyti <andi.shyti at linux.intel.com>
With fixes mentioned by Rodrigo.
Reviewed-by: Andrzej Hajda <andrzej.hajda at intel.com>
Regards
Andrzej
> ---
> drivers/gpu/drm/i915/i915_drv.h | 4 ++--
> 1 file changed, 2 insertions(+), 2 deletions(-)
>
> diff --git a/drivers/gpu/drm/i915/i915_drv.h b/drivers/gpu/drm/i915/i915_drv.h
> index fe7eeafe9cff6..c16f8a3cd914f 100644
> --- a/drivers/gpu/drm/i915/i915_drv.h
> +++ b/drivers/gpu/drm/i915/i915_drv.h
> @@ -381,11 +381,11 @@ static inline struct intel_gt *to_gt(struct drm_i915_private *i915)
> }
>
> /* Simple iterator over all initialised engines */
> -#define for_each_engine(engine__, dev_priv__, id__) \
> +#define for_each_engine(engine__, gt__, id__) \
> for ((id__) = 0; \
> (id__) < I915_NUM_ENGINES; \
> (id__)++) \
> - for_each_if ((engine__) = (dev_priv__)->engine[(id__)])
> + for_each_if ((engine__) = (gt__)->engine[(id__)])
>
> /* Iterator over subset of engines selected by mask */
> #define for_each_engine_masked(engine__, gt__, mask__, tmp__) \
More information about the dri-devel
mailing list