On Tue, May 09, 2017 at 12:00:58PM +0300, Mika Kuoppala wrote:
> Michal Wajdeczko <michal.wajdeczko@xxxxxxxxx> writes:
>
> > In order to allow use of e.g. forcewake_domains in a other feature headers
> > included from the top of i915_drv.h, move all uncore related definitions
> > into their own header.
> >
> > Signed-off-by: Michal Wajdeczko <michal.wajdeczko@xxxxxxxxx>
> > Suggested-by: Joonas Lahtinen <joonas.lahtinen@xxxxxxxxxxxxxxx>
> > Cc: Joonas Lahtinen <joonas.lahtinen@xxxxxxxxxxxxxxx>
> > ---
<snip>
> > -
> > -#define __mask_next_bit(mask) ({ \
> > - int __idx = ffs(mask) - 1; \
> > - mask &= ~BIT(__idx); \
> > - __idx; \
> > -})
> > -
>
> for_each_engine_masked needs this macro too, so we should leave it to
> top level or i915_utils.h?
Sure, I'll move it to the i915_utils.h, but at the same time I'm
wondering how did I missed that.
Thanks,
Michal
_______________________________________________
Intel-gfx mailing list
Intel-gfx@xxxxxxxxxxxxxxxxxxxxx
https://lists.freedesktop.org/mailman/listinfo/intel-gfx
