On Wed, Feb 08, 2017 at 02:42:03PM +0200, Joonas Lahtinen wrote: > On ti, 2017-02-07 at 18:00 -0800, Michel Thierry wrote: > > As that is what they are meant to be. It will prevent any confusion if we > > have to add other flags in the future. > > > > Signed-off-by: Michel Thierry <michel.thierry@xxxxxxxxx> > > <SNIP> > > > -#define HANG_ALLOW_BAN 1 > > -#define HANG_ALLOW_CAPTURE 2 > > +#define HANG_ALLOW_BAN (1<<0) > > +#define HANG_ALLOW_CAPTURE (1<<1) > > Make them BIT(0) and BIT(1), and this is; This is igt and BIT() is already a hodge-podge of different macros. -Chris -- Chris Wilson, Intel Open Source Technology Centre _______________________________________________ Intel-gfx mailing list Intel-gfx@xxxxxxxxxxxxxxxxxxxxx https://lists.freedesktop.org/mailman/listinfo/intel-gfx