On Tue, May 09, 2017 at 12:00:58PM +0300, Mika Kuoppala wrote:
> Michal Wajdeczko <[email protected]> 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 <[email protected]>
> > Suggested-by: Joonas Lahtinen <[email protected]>
> > Cc: Joonas Lahtinen <[email protected]>
> > ---
<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
[email protected]
https://lists.freedesktop.org/mailman/listinfo/intel-gfx