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

Reply via email to