[Intel-gfx] [PATCH] drm/i915: Use dev_priv as first argument of for_each_pipe()

Chris Wilson chris at chris-wilson.co.uk
Fri Aug 15 20:15:12 CEST 2014


On Fri, Aug 15, 2014 at 06:34:06PM +0100, Damien Lespiau wrote:
> -#define for_each_pipe(p) for ((p) = 0; (p) < INTEL_INFO(dev)->num_pipes; (p)++)
> +#define for_each_pipe(dev, p) for ((p) = 0; (p) < (dev)->info.num_pipes; (p)++)
>  #define for_each_sprite(p, s) for ((s) = 0; (s) < INTEL_INFO(dev)->num_sprites[(p)]; (s)++)

I think we still want to keep the INTEL_INFO() as that seems to be the
favourite for eliminating deadcode when compiling for a subset of
generation. It doesn't hurt to keep it around for a little longer.
-Chris

-- 
Chris Wilson, Intel Open Source Technology Centre



More information about the Intel-gfx mailing list