On Fri, 15 Aug 2014, Damien Lespiau <damien.lespiau@xxxxxxxxx> wrote: > Chris has decided that enough is enough. It's time to fixup dev Vs > dev_priv and the, oh so awful, INTEL_INFO(). This is a modest > contribution to the crusade. > -#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)++) Naming the first argument "dev" disagrees with the subject. BR, Jani. -- Jani Nikula, Intel Open Source Technology Center _______________________________________________ Intel-gfx mailing list Intel-gfx@xxxxxxxxxxxxxxxxxxxxx http://lists.freedesktop.org/mailman/listinfo/intel-gfx