[Intel-gfx] [PATCH i-g-t v2 7/7] tests: Use BIT macro instead of (1<<x)

Michel Thierry michel.thierry at intel.com
Thu Feb 9 18:23:18 UTC 2017


Mostly done with coccinelle,
@@
expression x;
@@
(
- (1<<x)
+ BIT(x)
|
- (1 << x)
+ BIT(x)
|
- 1 << x
+ BIT(x)
|
- (1UL<<x)
+ BIT(x)
|
- (1UL << x)
+ BIT(x)
|
- 1UL << x
+ BIT(x)
|
- (1ULL<<x)
+ BIT_ULL(x)
|
- (1ULL << x)
+ BIT_ULL(x)
|
- 1ULL << x
+ BIT_ULL(x)
)

v2: Minus a few cases (Chris)
    Squash lib import in gem_create (Joonas)

Cc: Chris Wilson <chris at chris-wilson.co.uk>
Cc: Joonas Lahtinen <joonas.lahtinen at linux.intel.com>
Signed-off-by: Michel Thierry <michel.thierry at intel.com>
---
 tests/drv_missed_irq.c              |  4 +-
 tests/drv_module_reload.c           |  4 +-
 tests/eviction_common.c             |  8 ++--
 tests/gem_bad_address.c             |  2 +-
 tests/gem_bad_reloc.c               |  4 +-
 tests/gem_busy.c                    | 24 ++++++------
 tests/gem_close_race.c              |  6 +--
 tests/gem_concurrent_all.c          |  2 +-
 tests/gem_create.c                  |  3 +-
 tests/gem_cs_prefetch.c             |  2 +-
 tests/gem_cs_tlb.c                  |  2 +-
 tests/gem_ctx_switch.c              |  4 +-
 tests/gem_ctx_thrash.c              |  2 +-
 tests/gem_double_irq_loop.c         |  4 +-
 tests/gem_exec_alignment.c          |  6 +--
 tests/gem_exec_async.c              |  8 ++--
 tests/gem_exec_blt.c                | 20 +++++-----
 tests/gem_exec_create.c             |  4 +-
 tests/gem_exec_faulting_reloc.c     | 14 +++----
 tests/gem_exec_fence.c              | 14 +++----
 tests/gem_exec_flush.c              | 10 ++---
 tests/gem_exec_gttfill.c            |  4 +-
 tests/gem_exec_latency.c            |  8 ++--
 tests/gem_exec_lut_handle.c         |  4 +-
 tests/gem_exec_nop.c                |  4 +-
 tests/gem_exec_parallel.c           |  6 +--
 tests/gem_exec_params.c             |  8 ++--
 tests/gem_exec_parse.c              |  4 +-
 tests/gem_exec_reloc.c              |  8 ++--
 tests/gem_exec_reuse.c              |  4 +-
 tests/gem_exec_schedule.c           |  6 +--
 tests/gem_exec_store.c              |  4 +-
 tests/gem_exec_suspend.c            |  6 +--
 tests/gem_exec_whisper.c            | 10 ++---
 tests/gem_fence_upload.c            |  4 +-
 tests/gem_fenced_exec_thrash.c      |  4 +-
 tests/gem_gtt_hog.c                 |  4 +-
 tests/gem_lut_handle.c              |  2 +-
 tests/gem_persistent_relocs.c       | 10 ++---
 tests/gem_pin.c                     |  6 +--
 tests/gem_pipe_control_store_loop.c | 18 ++++-----
 tests/gem_pwrite.c                  |  8 ++--
 tests/gem_pwrite_pread.c            | 12 +++---
 tests/gem_reloc_vs_gpu.c            | 12 +++---
 tests/gem_reset_stats.c             |  6 +--
 tests/gem_ringfill.c                |  6 +--
 tests/gem_shrink.c                  |  4 +-
 tests/gem_softpin.c                 | 12 +++---
 tests/gem_storedw_batches_loop.c    |  2 +-
 tests/gem_streaming_writes.c        | 10 ++---
 tests/gem_stress.c                  |  8 ++--
 tests/gem_sync.c                    | 10 ++---
 tests/gem_tiled_pread_basic.c       |  2 +-
 tests/gem_tiled_wb.c                |  2 +-
 tests/gem_tiled_wc.c                |  2 +-
 tests/gem_write_read_ring_switch.c  |  4 +-
 tests/gen3_mixed_blits.c            |  6 +--
 tests/gen3_render_linear_blits.c    |  6 +--
 tests/gen3_render_mixed_blits.c     |  6 +--
 tests/gen3_render_tiledx_blits.c    |  6 +--
 tests/gen3_render_tiledy_blits.c    |  6 +--
 tests/gen7_forcewake_mt.c           |  2 +-
 tests/kms_atomic.c                  |  6 +--
 tests/kms_atomic_transition.c       | 20 +++++-----
 tests/kms_ccs.c                     |  8 ++--
 tests/kms_cursor_crc.c              |  4 +-
 tests/kms_cursor_legacy.c           |  4 +-
 tests/kms_fbc_crc.c                 |  2 +-
 tests/kms_flip.c                    | 74 ++++++++++++++++++-------------------
 tests/kms_frontbuffer_tracking.c    | 26 ++++++-------
 tests/kms_pipe_color.c              |  6 +--
 tests/kms_pipe_crc_basic.c          |  4 +-
 tests/kms_plane.c                   | 10 ++---
 tests/kms_psr_sink_crc.c            |  2 +-
 tests/kms_render.c                  |  2 +-
 tests/kms_setmode.c                 | 11 +++---
 tests/perf.c                        | 52 +++++++++++++-------------
 tests/pm_rpm.c                      |  8 ++--
 tests/pm_rps.c                      |  4 +-
 tests/pm_sseu.c                     |  4 +-
 tests/prime_busy.c                  |  6 +--
 tests/prime_nv_pcopy.c              |  4 +-
 tests/prime_vgem.c                  |  8 ++--
 tests/sw_sync.c                     |  8 ++--
 tests/testdisplay.c                 |  2 +-
 tests/vgem_basic.c                  |  8 ++--
 86 files changed, 343 insertions(+), 343 deletions(-)

diff --git a/tests/drv_missed_irq.c b/tests/drv_missed_irq.c
index 2766c346..05f58567 100644
--- a/tests/drv_missed_irq.c
+++ b/tests/drv_missed_irq.c
@@ -58,11 +58,11 @@ static void trigger_missed_interrupt(int fd, unsigned ring)
 	i = 0;
 	batch[i] = MI_BATCH_BUFFER_START;
 	if (gen >= 8) {
-		batch[i] |= 1 << 8 | 1;
+		batch[i] |= BIT(8) | 1;
 		batch[++i] = 0;
 		batch[++i] = 0;
 	} else if (gen >= 6) {
-		batch[i] |= 1 << 8;
+		batch[i] |= BIT(8);
 		batch[++i] = 0;
 	} else {
 		batch[i] |= 2 << 6;
diff --git a/tests/drv_module_reload.c b/tests/drv_module_reload.c
index 32e1324b..3a7e73eb 100644
--- a/tests/drv_module_reload.c
+++ b/tests/drv_module_reload.c
@@ -88,7 +88,7 @@ static void store_dword(int fd, unsigned ring)
 	obj[1].relocation_count = 1;
 
 	i = 0;
-	batch[i] = MI_STORE_DWORD_IMM | (gen < 6 ? 1 << 22 : 0);
+	batch[i] = MI_STORE_DWORD_IMM | (gen < 6 ? BIT(22) : 0);
 	if (gen >= 8) {
 		batch[++i] = 0;
 		batch[++i] = 0;
@@ -142,7 +142,7 @@ static void store_all(int fd)
 
 	offset = sizeof(uint32_t);
 	i = 0;
-	batch[i] = MI_STORE_DWORD_IMM | (gen < 6 ? 1 << 22 : 0);
+	batch[i] = MI_STORE_DWORD_IMM | (gen < 6 ? BIT(22) : 0);
 	if (gen >= 8) {
 		batch[++i] = 0;
 		batch[++i] = 0;
diff --git a/tests/eviction_common.c b/tests/eviction_common.c
index e39555de..13502f6f 100644
--- a/tests/eviction_common.c
+++ b/tests/eviction_common.c
@@ -40,10 +40,10 @@ struct igt_eviction_test_ops {
 	void	 (*clear)(int fd, uint32_t bo, uint64_t size);
 };
 
-#define FORKING_EVICTIONS_INTERRUPTIBLE	  (1 << 0)
-#define FORKING_EVICTIONS_SWAPPING	  (1 << 1)
-#define FORKING_EVICTIONS_DUP_DRMFD	  (1 << 2)
-#define FORKING_EVICTIONS_MEMORY_PRESSURE (1 << 3)
+#define FORKING_EVICTIONS_INTERRUPTIBLE	  BIT(0)
+#define FORKING_EVICTIONS_SWAPPING	  BIT(1)
+#define FORKING_EVICTIONS_DUP_DRMFD	  BIT(2)
+#define FORKING_EVICTIONS_MEMORY_PRESSURE BIT(3)
 #define ALL_FORKING_EVICTIONS	(FORKING_EVICTIONS_INTERRUPTIBLE | \
 				 FORKING_EVICTIONS_SWAPPING | \
 				 FORKING_EVICTIONS_DUP_DRMFD | \
diff --git a/tests/gem_bad_address.c b/tests/gem_bad_address.c
index a970dfa4..19cbad0c 100644
--- a/tests/gem_bad_address.c
+++ b/tests/gem_bad_address.c
@@ -47,7 +47,7 @@ static void
 bad_store(void)
 {
 	BEGIN_BATCH(4, 0);
-	OUT_BATCH(MI_STORE_DWORD_IMM | MI_MEM_VIRTUAL | 1 << 21);
+	OUT_BATCH(MI_STORE_DWORD_IMM | MI_MEM_VIRTUAL | BIT(21));
 	OUT_BATCH(0);
 	OUT_BATCH(BAD_GTT_DEST);
 	OUT_BATCH(0xdeadbeef);
diff --git a/tests/gem_bad_reloc.c b/tests/gem_bad_reloc.c
index 2351ce91..99d36992 100644
--- a/tests/gem_bad_reloc.c
+++ b/tests/gem_bad_reloc.c
@@ -42,7 +42,7 @@
 IGT_TEST_DESCRIPTION("Simulates SNA behaviour using negative self-relocations"
 		     " for STATE_BASE_ADDRESS command packets.");
 
-#define USE_LUT (1 << 12)
+#define USE_LUT BIT(12)
 #define BIAS (256*1024)
 
 /* Simulates SNA behaviour using negative self-relocations for
@@ -137,7 +137,7 @@ static void negative_reloc_blt(int fd)
 		*b++ = XY_COLOR_BLT_CMD_NOLEN |
 			((gen >= 8) ? 5 : 4) |
 			COLOR_BLT_WRITE_ALPHA | XY_COLOR_BLT_WRITE_RGB;
-		*b++ = 0xf0 << 16 | 1 << 25 | 1 << 24 | 4096;
+		*b++ = 0xf0 << 16 | BIT(25) | BIT(24) | 4096;
 		*b++ = 1 << 16 | 0;
 		*b++ = 2 << 16 | 1024;
 		*b++ = ~0;
diff --git a/tests/gem_busy.c b/tests/gem_busy.c
index 39360109..af79154c 100644
--- a/tests/gem_busy.c
+++ b/tests/gem_busy.c
@@ -24,7 +24,7 @@
 #include "igt.h"
 #include "igt_rand.h"
 
-#define LOCAL_EXEC_NO_RELOC (1<<11)
+#define LOCAL_EXEC_NO_RELOC BIT(11)
 
 /* Exercise the busy-ioctl, ensuring the ABI is never broken */
 IGT_TEST_DESCRIPTION("Basic check of busy-ioctl ABI.");
@@ -80,14 +80,14 @@ static uint32_t busy_blt(int fd)
 		       I915_GEM_DOMAIN_CPU, I915_GEM_DOMAIN_CPU);
 
 #define COPY_BLT_CMD		(2<<29|0x53<<22|0x6)
-#define BLT_WRITE_ALPHA		(1<<21)
-#define BLT_WRITE_RGB		(1<<20)
+#define BLT_WRITE_ALPHA		BIT(21)
+#define BLT_WRITE_RGB		BIT(20)
 	while (factor--) {
 		/* XY_SRC_COPY */
 		map[i++] = COPY_BLT_CMD | BLT_WRITE_ALPHA | BLT_WRITE_RGB;
 		if (has_64bit_reloc)
 			map[i-1] += 2;
-		map[i++] = 0xcc << 16 | 1 << 25 | 1 << 24 | (4*1024);
+		map[i++] = 0xcc << 16 | BIT(25) | BIT(24) | (4*1024);
 		map[i++] = 0;
 		map[i++] = 256 << 16 | 1024;
 
@@ -185,21 +185,21 @@ static void semaphore(int fd, unsigned ring, uint32_t flags)
 	igt_assert(exec_noop(fd, handle, ring | flags, false));
 	igt_assert(still_busy(fd, handle[BUSY]));
 	__gem_busy(fd, handle[TEST], &read, &write);
-	igt_assert_eq(read, 1 << ring);
+	igt_assert_eq(read, BIT(ring));
 	igt_assert_eq(write, 0);
 
 	/* Requeue with a write */
 	igt_assert(exec_noop(fd, handle, ring | flags, true));
 	igt_assert(still_busy(fd, handle[BUSY]));
 	__gem_busy(fd, handle[TEST], &read, &write);
-	igt_assert_eq(read, 1 << ring);
+	igt_assert_eq(read, BIT(ring));
 	igt_assert_eq(write, ring);
 
 	/* Now queue it for a read across all available rings */
 	active = 0;
 	for (i = I915_EXEC_RENDER; i <= I915_EXEC_VEBOX; i++) {
 		if (exec_noop(fd, handle, i | flags, false))
-			active |= 1 << i;
+			active |= BIT(i);
 	}
 	igt_assert(still_busy(fd, handle[BUSY]));
 	__gem_busy(fd, handle[TEST], &read, &write);
@@ -262,7 +262,7 @@ static void one(int fd, unsigned ring, uint32_t flags, unsigned test_flags)
 		store[count].delta = sizeof(uint32_t) * count;
 		store[count].read_domains = I915_GEM_DOMAIN_INSTRUCTION;
 		store[count].write_domain = I915_GEM_DOMAIN_INSTRUCTION;
-		batch[i] = MI_STORE_DWORD_IMM | (gen < 6 ? 1 << 22 : 0);
+		batch[i] = MI_STORE_DWORD_IMM | (gen < 6 ? BIT(22) : 0);
 		if (gen >= 8) {
 			batch[++i] = 0;
 			batch[++i] = 0;
@@ -287,11 +287,11 @@ static void one(int fd, unsigned ring, uint32_t flags, unsigned test_flags)
 	store[count].write_domain = 0;
 	batch[i] = MI_BATCH_BUFFER_START;
 	if (gen >= 8) {
-		batch[i] |= 1 << 8 | 1;
+		batch[i] |= BIT(8) | 1;
 		batch[++i] = 0;
 		batch[++i] = 0;
 	} else if (gen >= 6) {
-		batch[i] |= 1 << 8;
+		batch[i] |= BIT(8);
 		batch[++i] = 0;
 	} else {
 		batch[i] |= 2 << 6;
@@ -335,10 +335,10 @@ static void one(int fd, unsigned ring, uint32_t flags, unsigned test_flags)
 	}
 
 	igt_assert_eq(write[SCRATCH], ring);
-	igt_assert_eq_u32(read[SCRATCH], 1 << ring);
+	igt_assert_eq_u32(read[SCRATCH], BIT(ring));
 
 	igt_assert_eq(write[BATCH], 0);
-	igt_assert_eq_u32(read[BATCH], 1 << ring);
+	igt_assert_eq_u32(read[BATCH], BIT(ring));
 
 	/* Calling busy in a loop should be enough to flush the rendering */
 	memset(&tv, 0, sizeof(tv));
diff --git a/tests/gem_close_race.c b/tests/gem_close_race.c
index 05e95547..6877642c 100644
--- a/tests/gem_close_race.c
+++ b/tests/gem_close_race.c
@@ -45,8 +45,8 @@
 #define OBJECT_SIZE (256 * 1024)
 
 #define COPY_BLT_CMD		(2<<29|0x53<<22|0x6)
-#define BLT_WRITE_ALPHA		(1<<21)
-#define BLT_WRITE_RGB		(1<<20)
+#define BLT_WRITE_ALPHA		BIT(21)
+#define BLT_WRITE_RGB		BIT(20)
 
 static uint32_t devid;
 static bool has_64bit_relocations;
@@ -69,7 +69,7 @@ static void selfcopy(int fd, uint32_t handle, int loops)
 	if (has_64bit_relocations)
 		*b += 2;
 	b++;
-	*b++ = 0xcc << 16 | 1 << 25 | 1 << 24 | (4*1024);
+	*b++ = 0xcc << 16 | BIT(25) | BIT(24) | (4*1024);
 	*b++ = 0;
 	*b++ = 1 << 16 | 1024;
 
diff --git a/tests/gem_concurrent_all.c b/tests/gem_concurrent_all.c
index 66250849..6ef83a93 100644
--- a/tests/gem_concurrent_all.c
+++ b/tests/gem_concurrent_all.c
@@ -651,7 +651,7 @@ gpu_set_bo(struct buffers *buffers, drm_intel_bo *bo, uint32_t val)
 		*b = buffers->width;
 	} else
 		*b = buffers->width << 2;
-	*b++ |= 0xf0 << 16 | 1 << 25 | 1 << 24;
+	*b++ |= 0xf0 << 16 | BIT(25) | BIT(24);
 	*b++ = 0;
 	*b++ = buffers->height << 16 | buffers->width;
 	reloc[0].offset = (b - buf) * sizeof(uint32_t);
diff --git a/tests/gem_create.c b/tests/gem_create.c
index de7b8209..87d121a2 100644
--- a/tests/gem_create.c
+++ b/tests/gem_create.c
@@ -53,6 +53,7 @@
 #include "intel_io.h"
 #include "intel_chipset.h"
 #include "igt_aux.h"
+#include "igt_bitops.h"
 #include "drmtest.h"
 #include "drm.h"
 #include "i915_drm.h"
@@ -68,7 +69,7 @@ struct local_i915_gem_create_v2 {
 	uint64_t size;
 	uint32_t handle;
 	uint32_t pad;
-#define I915_CREATE_PLACEMENT_STOLEN (1<<0)
+#define I915_CREATE_PLACEMENT_STOLEN BIT(0)
 	uint32_t flags;
 } create;
 
diff --git a/tests/gem_cs_prefetch.c b/tests/gem_cs_prefetch.c
index fd9bb793..9f72a366 100644
--- a/tests/gem_cs_prefetch.c
+++ b/tests/gem_cs_prefetch.c
@@ -62,7 +62,7 @@ static void setup(int fd, int gen, struct shadow *shadow)
 	shadow->handle = gem_create(fd, 4096);
 
 	i = 0;
-	buf[i++] = MI_STORE_DWORD_IMM | (gen < 6 ? 1 << 22 : 0);
+	buf[i++] = MI_STORE_DWORD_IMM | (gen < 6 ? BIT(22) : 0);
 	if (gen >= 8) {
 		buf[i++] = BATCH_SIZE - sizeof(uint32_t);
 		buf[i++] = 0;
diff --git a/tests/gem_cs_tlb.c b/tests/gem_cs_tlb.c
index 0338a1e0..670dc1ce 100644
--- a/tests/gem_cs_tlb.c
+++ b/tests/gem_cs_tlb.c
@@ -53,7 +53,7 @@
 IGT_TEST_DESCRIPTION("Check whether we correctly invalidate the cs tlb.");
 
 #define LOCAL_I915_EXEC_VEBOX	(4<<0)
-#define EXEC_OBJECT_PINNED	(1<<4)
+#define EXEC_OBJECT_PINNED	BIT(4)
 #define BATCH_SIZE (1024*1024)
 
 static bool has_softpin(int fd)
diff --git a/tests/gem_ctx_switch.c b/tests/gem_ctx_switch.c
index 00122806..cad0fd77 100644
--- a/tests/gem_ctx_switch.c
+++ b/tests/gem_ctx_switch.c
@@ -40,8 +40,8 @@
 #include <time.h>
 #include "drm.h"
 
-#define LOCAL_I915_EXEC_NO_RELOC (1<<11)
-#define LOCAL_I915_EXEC_HANDLE_LUT (1<<12)
+#define LOCAL_I915_EXEC_NO_RELOC BIT(11)
+#define LOCAL_I915_EXEC_HANDLE_LUT BIT(12)
 
 #define INTERRUPTIBLE 1
 
diff --git a/tests/gem_ctx_thrash.c b/tests/gem_ctx_thrash.c
index cc7ea1a7..2908f391 100644
--- a/tests/gem_ctx_thrash.c
+++ b/tests/gem_ctx_thrash.c
@@ -212,7 +212,7 @@ static void single(const char *name, bool all_engines)
 				buf[++i] = offset >> 32;
 			} else if (gen >= 4) {
 				if (gen < 6)
-					buf[i] |= 1 << 22;
+					buf[i] |= BIT(22);
 				buf[++i] = 0;
 				buf[++i] = offset;
 			} else {
diff --git a/tests/gem_double_irq_loop.c b/tests/gem_double_irq_loop.c
index 71ed3b0c..21ae2563 100644
--- a/tests/gem_double_irq_loop.c
+++ b/tests/gem_double_irq_loop.c
@@ -53,7 +53,7 @@ IGT_TEST_DESCRIPTION("Basic check for missed IRQs on blt ring.");
 
 
 #define MI_COND_BATCH_BUFFER_END	(0x36<<23 | 1)
-#define MI_DO_COMPARE			(1<<21)
+#define MI_DO_COMPARE			BIT(21)
 static void
 dummy_reloc_loop(void)
 {
@@ -78,7 +78,7 @@ dummy_reloc_loop(void)
 		OUT_BATCH(0); /* reserved */
 		OUT_RELOC(target_buffer, I915_GEM_DOMAIN_RENDER,
 				I915_GEM_DOMAIN_RENDER, 0);
-		OUT_BATCH(MI_NOOP | (1<<22) | (0xf));
+		OUT_BATCH(MI_NOOP | BIT(22) | (0xf));
 		ADVANCE_BATCH();
 		intel_batchbuffer_flush(batch);
 
diff --git a/tests/gem_exec_alignment.c b/tests/gem_exec_alignment.c
index 0549af33..2d3bb1d7 100644
--- a/tests/gem_exec_alignment.c
+++ b/tests/gem_exec_alignment.c
@@ -100,11 +100,11 @@ static void many(int fd)
 	for (i = 0; i < count; i++) {
 		execobj[i].handle = gem_create(fd, 4096);
 		if ((gtt_size-1) >> 32)
-			execobj[i].flags = 1<<3; /* EXEC_OBJECT_SUPPORTS_48B_ADDRESS */
+			execobj[i].flags = BIT(3); /* EXEC_OBJECT_SUPPORTS_48B_ADDRESS */
 	}
 	execobj[i].handle = gem_create(fd, 4096);
 	if ((gtt_size-1) >> 32)
-		execobj[i].flags = 1<<3; /* EXEC_OBJECT_SUPPORTS_48B_ADDRESS */
+		execobj[i].flags = BIT(3); /* EXEC_OBJECT_SUPPORTS_48B_ADDRESS */
 	gem_write(fd, execobj[i].handle, 0, &bbe, sizeof(bbe));
 
 	memset(&execbuf, 0, sizeof(execbuf));
@@ -149,7 +149,7 @@ static void single(int fd)
 
 	memset(&execobj, 0, sizeof(execobj));
 	execobj.handle = gem_create(fd, 4096);
-	execobj.flags = 1<<3; /* EXEC_OBJECT_SUPPORTS_48B_ADDRESS */
+	execobj.flags = BIT(3); /* EXEC_OBJECT_SUPPORTS_48B_ADDRESS */
 	gem_write(fd, execobj.handle, 0, &batch, sizeof(batch));
 
 	memset(&execbuf, 0, sizeof(execbuf));
diff --git a/tests/gem_exec_async.c b/tests/gem_exec_async.c
index 5b379cdf..802d767a 100644
--- a/tests/gem_exec_async.c
+++ b/tests/gem_exec_async.c
@@ -23,7 +23,7 @@
 
 #include "igt.h"
 
-#define LOCAL_OBJECT_ASYNC (1 << 6)
+#define LOCAL_OBJECT_ASYNC BIT(6)
 #define LOCAL_PARAM_HAS_EXEC_ASYNC 43
 
 IGT_TEST_DESCRIPTION("Check that we can issue concurrent writes across the engines.");
@@ -61,7 +61,7 @@ static void store_dword(int fd, unsigned ring,
 	obj[1].relocation_count = 1;
 
 	i = 0;
-	batch[i] = MI_STORE_DWORD_IMM | (gen < 6 ? 1 << 22 : 0);
+	batch[i] = MI_STORE_DWORD_IMM | (gen < 6 ? BIT(22) : 0);
 	if (gen >= 8) {
 		batch[++i] = offset;
 		batch[++i] = 0;
@@ -118,11 +118,11 @@ static void one(int fd, unsigned ring, uint32_t flags)
 	i = 0;
 	batch[i] = MI_BATCH_BUFFER_START;
 	if (gen >= 8) {
-		batch[i] |= 1 << 8 | 1;
+		batch[i] |= BIT(8) | 1;
 		batch[++i] = 0;
 		batch[++i] = 0;
 	} else if (gen >= 6) {
-		batch[i] |= 1 << 8;
+		batch[i] |= BIT(8);
 		batch[++i] = 0;
 	} else {
 		batch[i] |= 2 << 6;
diff --git a/tests/gem_exec_blt.c b/tests/gem_exec_blt.c
index 932884b0..15da3798 100644
--- a/tests/gem_exec_blt.c
+++ b/tests/gem_exec_blt.c
@@ -42,13 +42,13 @@
 #define OBJECT_SIZE 16384
 
 #define COPY_BLT_CMD		(2<<29|0x53<<22|0x6)
-#define BLT_WRITE_ALPHA		(1<<21)
-#define BLT_WRITE_RGB		(1<<20)
-#define BLT_SRC_TILED		(1<<15)
-#define BLT_DST_TILED		(1<<11)
+#define BLT_WRITE_ALPHA		BIT(21)
+#define BLT_WRITE_RGB		BIT(20)
+#define BLT_SRC_TILED		BIT(15)
+#define BLT_DST_TILED		BIT(11)
 
-#define LOCAL_I915_EXEC_NO_RELOC (1<<11)
-#define LOCAL_I915_EXEC_HANDLE_LUT (1<<12)
+#define LOCAL_I915_EXEC_NO_RELOC BIT(11)
+#define LOCAL_I915_EXEC_HANDLE_LUT BIT(12)
 
 static int gem_linear_blt(int fd,
 			  uint32_t *batch,
@@ -60,14 +60,14 @@ static int gem_linear_blt(int fd,
 	uint32_t *b = batch;
 	int height = length / (16 * 1024);
 
-	igt_assert_lte(height, 1 << 16);
+	igt_assert_lte(height, BIT(16));
 
 	if (height) {
 		int i = 0;
 		b[i++] = COPY_BLT_CMD | BLT_WRITE_ALPHA | BLT_WRITE_RGB;
 		if (intel_gen(intel_get_drm_devid(fd)) >= 8)
 			b[i-1]+=2;
-		b[i++] = 0xcc << 16 | 1 << 25 | 1 << 24 | (16*1024);
+		b[i++] = 0xcc << 16 | BIT(25) | BIT(24) | (16*1024);
 		b[i++] = 0;
 		b[i++] = height << 16 | (4*1024);
 		b[i++] = 0;
@@ -105,7 +105,7 @@ static int gem_linear_blt(int fd,
 		b[i++] = COPY_BLT_CMD | BLT_WRITE_ALPHA | BLT_WRITE_RGB;
 		if (intel_gen(intel_get_drm_devid(fd)) >= 8)
 			b[i-1]+=2;
-		b[i++] = 0xcc << 16 | 1 << 25 | 1 << 24 | (16*1024);
+		b[i++] = 0xcc << 16 | BIT(25) | BIT(24) | (16*1024);
 		b[i++] = height << 16;
 		b[i++] = (1+height) << 16 | (length / 4);
 		b[i++] = 0;
@@ -235,7 +235,7 @@ static void run(int object_size, bool dumb)
 	}
 	gem_sync(fd, handle);
 
-	for (count = 1; count <= 1<<12; count <<= 1) {
+	for (count = 1; count <= BIT(12); count <<= 1) {
 		struct timeval start, end;
 		const int reps = 9;
 		double t[reps], sum;
diff --git a/tests/gem_exec_create.c b/tests/gem_exec_create.c
index 25926150..19e2abcf 100644
--- a/tests/gem_exec_create.c
+++ b/tests/gem_exec_create.c
@@ -40,8 +40,8 @@
 #include <time.h>
 #include "drm.h"
 
-#define LOCAL_I915_EXEC_NO_RELOC (1<<11)
-#define LOCAL_I915_EXEC_HANDLE_LUT (1<<12)
+#define LOCAL_I915_EXEC_NO_RELOC BIT(11)
+#define LOCAL_I915_EXEC_HANDLE_LUT BIT(12)
 
 #define LOCAL_I915_EXEC_BSD_SHIFT      (13)
 #define LOCAL_I915_EXEC_BSD_MASK       (3 << LOCAL_I915_EXEC_BSD_SHIFT)
diff --git a/tests/gem_exec_faulting_reloc.c b/tests/gem_exec_faulting_reloc.c
index c17ce2da..3a631f45 100644
--- a/tests/gem_exec_faulting_reloc.c
+++ b/tests/gem_exec_faulting_reloc.c
@@ -50,10 +50,10 @@ IGT_TEST_DESCRIPTION("Submit patches with relocations in memory that will"
 #define OBJECT_SIZE 16384
 
 #define COPY_BLT_CMD_NOLEN	(2<<29|0x53<<22)
-#define BLT_WRITE_ALPHA		(1<<21)
-#define BLT_WRITE_RGB		(1<<20)
-#define BLT_SRC_TILED		(1<<15)
-#define BLT_DST_TILED		(1<<11)
+#define BLT_WRITE_ALPHA		BIT(21)
+#define BLT_WRITE_RGB		BIT(20)
+#define BLT_SRC_TILED		BIT(15)
+#define BLT_DST_TILED		BIT(11)
 
 uint32_t devid;
 
@@ -66,7 +66,7 @@ static int gem_linear_blt(uint32_t *batch,
 	uint32_t *b = batch;
 	int height = length / (16 * 1024);
 
-	igt_assert_lte(height, 1 << 16);
+	igt_assert_lte(height, BIT(16));
 
 	if (height) {
 		int i = 0;
@@ -75,7 +75,7 @@ static int gem_linear_blt(uint32_t *batch,
 			b[i-1] |= 8;
 		else
 			b[i-1] |= 6;
-		b[i++] = 0xcc << 16 | 1 << 25 | 1 << 24 | (16*1024);
+		b[i++] = 0xcc << 16 | BIT(25) | BIT(24) | (16*1024);
 		b[i++] = 0;
 		b[i++] = height << 16 | (4*1024);
 		b[i++] = 0;
@@ -119,7 +119,7 @@ static int gem_linear_blt(uint32_t *batch,
 			b[i-1] |= 8;
 		else
 			b[i-1] |= 6;
-		b[i++] = 0xcc << 16 | 1 << 25 | 1 << 24 | (16*1024);
+		b[i++] = 0xcc << 16 | BIT(25) | BIT(24) | (16*1024);
 		b[i++] = height << 16;
 		b[i++] = (1+height) << 16 | (length / 4);
 		b[i++] = 0;
diff --git a/tests/gem_exec_fence.c b/tests/gem_exec_fence.c
index ddc5e7f2..974c6849 100644
--- a/tests/gem_exec_fence.c
+++ b/tests/gem_exec_fence.c
@@ -30,8 +30,8 @@
 IGT_TEST_DESCRIPTION("Check that execbuf waits for explicit fences");
 
 #define LOCAL_PARAM_HAS_EXEC_FENCE 44
-#define LOCAL_EXEC_FENCE_IN (1 << 16)
-#define LOCAL_EXEC_FENCE_OUT (1 << 17)
+#define LOCAL_EXEC_FENCE_IN BIT(16)
+#define LOCAL_EXEC_FENCE_OUT BIT(17)
 #define LOCAL_IOCTL_I915_GEM_EXECBUFFER2_WR       DRM_IOWR(DRM_COMMAND_BASE + DRM_I915_GEM_EXECBUFFER2, struct drm_i915_gem_execbuffer2)
 
 static bool can_mi_store_dword(int gen, unsigned engine)
@@ -73,7 +73,7 @@ static void store(int fd, unsigned ring, int fence, uint32_t target, unsigned of
 	reloc.delta = sizeof(uint32_t) * offset_value;
 	reloc.read_domains = I915_GEM_DOMAIN_INSTRUCTION;
 	reloc.write_domain = I915_GEM_DOMAIN_INSTRUCTION;
-	batch[i] = MI_STORE_DWORD_IMM | (gen < 6 ? 1 << 22 : 0);
+	batch[i] = MI_STORE_DWORD_IMM | (gen < 6 ? BIT(22) : 0);
 	if (gen >= 8) {
 		batch[++i] = reloc.delta;
 		batch[++i] = 0;
@@ -153,11 +153,11 @@ static void test_fence_busy(int fd, unsigned ring, unsigned flags)
 	i = 0;
 	batch[i] = MI_BATCH_BUFFER_START;
 	if (gen >= 8) {
-		batch[i] |= 1 << 8 | 1;
+		batch[i] |= BIT(8) | 1;
 		batch[++i] = 0;
 		batch[++i] = 0;
 	} else if (gen >= 6) {
-		batch[i] |= 1 << 8;
+		batch[i] |= BIT(8);
 		batch[++i] = 0;
 	} else {
 		batch[i] |= 2 << 6;
@@ -245,11 +245,11 @@ static void test_fence_await(int fd, unsigned ring, unsigned flags)
 	i = 0;
 	batch[i] = MI_BATCH_BUFFER_START;
 	if (gen >= 8) {
-		batch[i] |= 1 << 8 | 1;
+		batch[i] |= BIT(8) | 1;
 		batch[++i] = 0;
 		batch[++i] = 0;
 	} else if (gen >= 6) {
-		batch[i] |= 1 << 8;
+		batch[i] |= BIT(8);
 		batch[++i] = 0;
 	} else {
 		batch[i] |= 2 << 6;
diff --git a/tests/gem_exec_flush.c b/tests/gem_exec_flush.c
index 4b51fbb0..4c417070 100644
--- a/tests/gem_exec_flush.c
+++ b/tests/gem_exec_flush.c
@@ -162,7 +162,7 @@ static void run(int fd, unsigned ring, int nchild, int timeout,
 		memset(&execbuf, 0, sizeof(execbuf));
 		execbuf.buffers_ptr = to_user_pointer(obj);
 		execbuf.buffer_count = 3;
-		execbuf.flags = ring | (1 << 11) | (1<<12);
+		execbuf.flags = ring | BIT(11) | BIT(12);
 		if (gen < 6)
 			execbuf.flags |= I915_EXEC_SECURE;
 
@@ -191,7 +191,7 @@ static void run(int fd, unsigned ring, int nchild, int timeout,
 			reloc0[i].write_domain = I915_GEM_DOMAIN_INSTRUCTION;
 
 			offset = obj[0].offset + reloc0[i].delta;
-			*b++ = MI_STORE_DWORD_IMM | (gen < 6 ? 1 << 22 : 0);
+			*b++ = MI_STORE_DWORD_IMM | (gen < 6 ? BIT(22) : 0);
 			if (gen >= 8) {
 				*b++ = offset;
 				*b++ = offset >> 32;
@@ -225,7 +225,7 @@ static void run(int fd, unsigned ring, int nchild, int timeout,
 			reloc1[i].write_domain = I915_GEM_DOMAIN_INSTRUCTION;
 
 			offset = obj[0].offset + reloc1[i].delta;
-			*b++ = MI_STORE_DWORD_IMM | (gen < 6 ? 1 << 22 : 0);
+			*b++ = MI_STORE_DWORD_IMM | (gen < 6 ? BIT(22) : 0);
 			if (gen >= 8) {
 				*b++ = offset;
 				*b++ = offset >> 32;
@@ -394,7 +394,7 @@ static void batch(int fd, unsigned ring, int nchild, int timeout,
 		memset(&execbuf, 0, sizeof(execbuf));
 		execbuf.buffers_ptr = to_user_pointer(obj);
 		execbuf.buffer_count = 2;
-		execbuf.flags = ring | (1 << 11) | (1<<12);
+		execbuf.flags = ring | BIT(11) | BIT(12);
 		if (gen < 6)
 			execbuf.flags |= I915_EXEC_SECURE;
 
@@ -464,7 +464,7 @@ static void batch(int fd, unsigned ring, int nchild, int timeout,
 				reloc.delta = i * sizeof(uint32_t);
 
 				offset = reloc.presumed_offset + reloc.delta;
-				*b++ = MI_STORE_DWORD_IMM | (gen < 6 ? 1 << 22 : 0);
+				*b++ = MI_STORE_DWORD_IMM | (gen < 6 ? BIT(22) : 0);
 				if (gen >= 8) {
 					*b++ = offset;
 					*b++ = offset >> 32;
diff --git a/tests/gem_exec_gttfill.c b/tests/gem_exec_gttfill.c
index 6ab88b15..cf4fb2a9 100644
--- a/tests/gem_exec_gttfill.c
+++ b/tests/gem_exec_gttfill.c
@@ -78,9 +78,9 @@ static void submit(int fd, int gen,
 	reloc[1].read_domains = I915_GEM_DOMAIN_INSTRUCTION;
 
 	n = 0;
-	batch[n] = MI_STORE_DWORD_IMM | (gen < 6 ? 1 << 22 : 0);
+	batch[n] = MI_STORE_DWORD_IMM | (gen < 6 ? BIT(22) : 0);
 	if (gen >= 8) {
-		batch[n] |= 1 << 21;
+		batch[n] |= BIT(21);
 		batch[n]++;
 		batch[++n] = reloc[0].delta;/* lower_32_bits(address) */
 		batch[++n] = 0; /* upper_32_bits(address) */
diff --git a/tests/gem_exec_latency.c b/tests/gem_exec_latency.c
index 6e6cc57b..d62466f4 100644
--- a/tests/gem_exec_latency.c
+++ b/tests/gem_exec_latency.c
@@ -41,8 +41,8 @@
 #include "igt_sysfs.h"
 #include "igt_vgem.h"
 
-#define LOCAL_I915_EXEC_NO_RELOC (1<<11)
-#define LOCAL_I915_EXEC_HANDLE_LUT (1<<12)
+#define LOCAL_I915_EXEC_NO_RELOC BIT(11)
+#define LOCAL_I915_EXEC_HANDLE_LUT BIT(12)
 
 #define LOCAL_I915_EXEC_BSD_SHIFT      (13)
 #define LOCAL_I915_EXEC_BSD_MASK       (3 << LOCAL_I915_EXEC_BSD_SHIFT)
@@ -191,11 +191,11 @@ static void latency_on_ring(int fd,
 		i = 16 * j + (has_64bit_reloc ? 4 : 3);
 		map[i] = MI_BATCH_BUFFER_START;
 		if (gen >= 8) {
-			map[i] |= 1 << 8 | 1;
+			map[i] |= BIT(8) | 1;
 			map[i + 1] = offset;
 			map[i + 2] = offset >> 32;
 		} else if (gen >= 6) {
-			map[i] |= 1 << 8;
+			map[i] |= BIT(8);
 			map[i + 1] = offset;
 		} else {
 			map[i] |= 2 << 6;
diff --git a/tests/gem_exec_lut_handle.c b/tests/gem_exec_lut_handle.c
index 09b55b9e..6ab25ed4 100644
--- a/tests/gem_exec_lut_handle.c
+++ b/tests/gem_exec_lut_handle.c
@@ -43,8 +43,8 @@ IGT_TEST_DESCRIPTION("Exercises the basic execbuffer using the handle LUT"
 
 #define BATCH_SIZE		(1024*1024)
 
-#define LOCAL_I915_EXEC_NO_RELOC (1<<11)
-#define LOCAL_I915_EXEC_HANDLE_LUT (1<<12)
+#define LOCAL_I915_EXEC_NO_RELOC BIT(11)
+#define LOCAL_I915_EXEC_HANDLE_LUT BIT(12)
 
 #define MAX_NUM_EXEC 2048
 #define MAX_NUM_RELOC 4096
diff --git a/tests/gem_exec_nop.c b/tests/gem_exec_nop.c
index 0d7a4d06..9f05edb8 100644
--- a/tests/gem_exec_nop.c
+++ b/tests/gem_exec_nop.c
@@ -43,8 +43,8 @@
 #include <time.h>
 #include "drm.h"
 
-#define LOCAL_I915_EXEC_NO_RELOC (1<<11)
-#define LOCAL_I915_EXEC_HANDLE_LUT (1<<12)
+#define LOCAL_I915_EXEC_NO_RELOC BIT(11)
+#define LOCAL_I915_EXEC_HANDLE_LUT BIT(12)
 
 #define LOCAL_I915_EXEC_BSD_SHIFT      (13)
 #define LOCAL_I915_EXEC_BSD_MASK       (3 << LOCAL_I915_EXEC_BSD_SHIFT)
diff --git a/tests/gem_exec_parallel.c b/tests/gem_exec_parallel.c
index 3be5017a..d7000cd8 100644
--- a/tests/gem_exec_parallel.c
+++ b/tests/gem_exec_parallel.c
@@ -32,8 +32,8 @@
 #include "igt.h"
 #include "igt_gt.h"
 
-#define LOCAL_I915_EXEC_NO_RELOC (1<<11)
-#define LOCAL_I915_EXEC_HANDLE_LUT (1<<12)
+#define LOCAL_I915_EXEC_NO_RELOC BIT(11)
+#define LOCAL_I915_EXEC_HANDLE_LUT BIT(12)
 
 #define LOCAL_I915_EXEC_BSD_SHIFT      (13)
 #define LOCAL_I915_EXEC_BSD_MASK       (3 << LOCAL_I915_EXEC_BSD_SHIFT)
@@ -121,7 +121,7 @@ static void *thread(void *data)
 		fd = t->fd;
 
 	i = 0;
-	batch[i] = MI_STORE_DWORD_IMM | (t->gen < 6 ? 1 << 22 : 0);
+	batch[i] = MI_STORE_DWORD_IMM | (t->gen < 6 ? BIT(22) : 0);
 	if (t->gen >= 8) {
 		batch[++i] = 4*t->id;
 		batch[++i] = 0;
diff --git a/tests/gem_exec_params.c b/tests/gem_exec_params.c
index fbc0ab30..082bf5f8 100644
--- a/tests/gem_exec_params.c
+++ b/tests/gem_exec_params.c
@@ -42,11 +42,11 @@
 
 #define LOCAL_I915_EXEC_VEBOX (4<<0)
 #define LOCAL_I915_EXEC_BSD_MASK (3<<13)
-#define LOCAL_I915_EXEC_BSD_RING1 (1<<13)
+#define LOCAL_I915_EXEC_BSD_RING1 BIT(13)
 #define LOCAL_I915_EXEC_BSD_RING2 (2<<13)
-#define LOCAL_I915_EXEC_RESOURCE_STREAMER (1<<15)
-#define LOCAL_I915_EXEC_FENCE_IN (1 << 16)
-#define LOCAL_I915_EXEC_FENCE_OUT (1 << 17)
+#define LOCAL_I915_EXEC_RESOURCE_STREAMER BIT(15)
+#define LOCAL_I915_EXEC_FENCE_IN BIT(16)
+#define LOCAL_I915_EXEC_FENCE_OUT BIT(17)
 
 static bool has_ring(int fd, unsigned ring_exec_flags)
 {
diff --git a/tests/gem_exec_parse.c b/tests/gem_exec_parse.c
index ea39c481..3d51ba03 100644
--- a/tests/gem_exec_parse.c
+++ b/tests/gem_exec_parse.c
@@ -53,8 +53,8 @@
 #define MI_DISPLAY_FLIP ((0x14 << 23) | 1)
 
 #define GFX_OP_PIPE_CONTROL	((0x3<<29)|(0x3<<27)|(0x2<<24)|2)
-#define   PIPE_CONTROL_QW_WRITE	(1<<14)
-#define   PIPE_CONTROL_LRI_POST_OP (1<<23)
+#define   PIPE_CONTROL_QW_WRITE	BIT(14)
+#define   PIPE_CONTROL_LRI_POST_OP BIT(23)
 
 static int parser_version;
 
diff --git a/tests/gem_exec_reloc.c b/tests/gem_exec_reloc.c
index 8356d852..bf407f2e 100644
--- a/tests/gem_exec_reloc.c
+++ b/tests/gem_exec_reloc.c
@@ -29,8 +29,8 @@ IGT_TEST_DESCRIPTION("Basic sanity check of execbuf-ioctl relocations.");
 #define LOCAL_I915_EXEC_BSD_SHIFT      (13)
 #define LOCAL_I915_EXEC_BSD_MASK       (3 << LOCAL_I915_EXEC_BSD_SHIFT)
 
-#define LOCAL_I915_EXEC_NO_RELOC (1<<11)
-#define LOCAL_I915_EXEC_HANDLE_LUT (1<<12)
+#define LOCAL_I915_EXEC_NO_RELOC BIT(11)
+#define LOCAL_I915_EXEC_HANDLE_LUT BIT(12)
 
 #define ENGINE_MASK  (I915_EXEC_RING_MASK | LOCAL_I915_EXEC_BSD_MASK)
 
@@ -61,7 +61,7 @@ static void write_dword(int fd,
 	obj[1].handle = gem_create(fd, 4096);
 
 	i = 0;
-	buf[i++] = MI_STORE_DWORD_IMM | (gen < 6 ? 1<<22 : 0);
+	buf[i++] = MI_STORE_DWORD_IMM | (gen < 6 ? BIT(22) : 0);
 	if (gen >= 8) {
 		buf[i++] = target_offset;
 		buf[i++] = target_offset >> 32;
@@ -292,7 +292,7 @@ static void active(int fd, unsigned engine)
 	for (pass = 0; pass < 1024; pass++) {
 		uint32_t batch[16];
 		int i = 0;
-		batch[i] = MI_STORE_DWORD_IMM | (gen < 6 ? 1 << 22 : 0);
+		batch[i] = MI_STORE_DWORD_IMM | (gen < 6 ? BIT(22) : 0);
 		if (gen >= 8) {
 			batch[++i] = 0;
 			batch[++i] = 0;
diff --git a/tests/gem_exec_reuse.c b/tests/gem_exec_reuse.c
index f0323c8b..bf4ae062 100644
--- a/tests/gem_exec_reuse.c
+++ b/tests/gem_exec_reuse.c
@@ -51,7 +51,7 @@ static void noop(struct noop *n,
 	memset(&execbuf, 0, sizeof(execbuf));
 	execbuf.buffers_ptr = to_user_pointer(n->obj);
 	execbuf.buffer_count = count + 1;
-	execbuf.flags = ring | 1 << 12;
+	execbuf.flags = ring | BIT(12);
 	execbuf.rsvd1 = ctx;
 	gem_execbuf(n->fd, &execbuf);
 }
@@ -162,7 +162,7 @@ igt_main
 		if (max < gtt_size)
 			gtt_size = max;
 
-		no.nhandles = 1 << (fls(gtt_size) - 1);
+		no.nhandles = BIT(fls(gtt_size) - 1);
 		intel_require_memory(no.nhandles, 4096, CHECK_RAM);
 
 		no.max_age = no.nhandles / 2;
diff --git a/tests/gem_exec_schedule.c b/tests/gem_exec_schedule.c
index d8454d58..c92c1317 100644
--- a/tests/gem_exec_schedule.c
+++ b/tests/gem_exec_schedule.c
@@ -87,7 +87,7 @@ static void store_dword(int fd, uint32_t ctx, unsigned ring,
 	obj[2].relocation_count = 1;
 
 	i = 0;
-	batch[i] = MI_STORE_DWORD_IMM | (gen < 6 ? 1 << 22 : 0);
+	batch[i] = MI_STORE_DWORD_IMM | (gen < 6 ? BIT(22) : 0);
 	if (gen >= 8) {
 		batch[++i] = offset;
 		batch[++i] = 0;
@@ -147,11 +147,11 @@ static uint32_t *make_busy(int fd, uint32_t target, unsigned ring)
 	i = 0;
 	batch[i] = MI_BATCH_BUFFER_START;
 	if (gen >= 8) {
-		batch[i] |= 1 << 8 | 1;
+		batch[i] |= BIT(8) | 1;
 		batch[++i] = 0;
 		batch[++i] = 0;
 	} else if (gen >= 6) {
-		batch[i] |= 1 << 8;
+		batch[i] |= BIT(8);
 		batch[++i] = 0;
 	} else {
 		batch[i] |= 2 << 6;
diff --git a/tests/gem_exec_store.c b/tests/gem_exec_store.c
index 08248966..eed32f47 100644
--- a/tests/gem_exec_store.c
+++ b/tests/gem_exec_store.c
@@ -71,7 +71,7 @@ static void store_dword(int fd, unsigned ring)
 	obj[1].relocation_count = 1;
 
 	i = 0;
-	batch[i] = MI_STORE_DWORD_IMM | (gen < 6 ? 1 << 22 : 0);
+	batch[i] = MI_STORE_DWORD_IMM | (gen < 6 ? BIT(22) : 0);
 	if (gen >= 8) {
 		batch[++i] = 0;
 		batch[++i] = 0;
@@ -122,7 +122,7 @@ static void store_all(int fd)
 
 	offset = sizeof(uint32_t);
 	i = 0;
-	batch[i] = MI_STORE_DWORD_IMM | (gen < 6 ? 1 << 22 : 0);
+	batch[i] = MI_STORE_DWORD_IMM | (gen < 6 ? BIT(22) : 0);
 	if (gen >= 8) {
 		batch[++i] = 0;
 		batch[++i] = 0;
diff --git a/tests/gem_exec_suspend.c b/tests/gem_exec_suspend.c
index c250761e..e66248a5 100644
--- a/tests/gem_exec_suspend.c
+++ b/tests/gem_exec_suspend.c
@@ -43,7 +43,7 @@
 #define ENGINE_MASK  (I915_EXEC_RING_MASK | LOCAL_I915_EXEC_BSD_MASK)
 
 #define UNCACHED (0<<8)
-#define CACHED (1<<8)
+#define CACHED BIT(8)
 
 static void run_test(int fd, unsigned ring, unsigned flags);
 
@@ -146,7 +146,7 @@ static void run_test(int fd, unsigned engine, unsigned flags)
 	memset(&execbuf, 0, sizeof(execbuf));
 	execbuf.buffers_ptr = to_user_pointer(obj);
 	execbuf.buffer_count = 2;
-	execbuf.flags = 1 << 11;
+	execbuf.flags = BIT(11);
 	if (gen < 6)
 		execbuf.flags |= I915_EXEC_SECURE;
 
@@ -182,7 +182,7 @@ static void run_test(int fd, unsigned engine, unsigned flags)
 		offset = reloc.presumed_offset + reloc.delta;
 
 		b = 0;
-		buf[b] = MI_STORE_DWORD_IMM | (gen < 6 ? 1 << 22 : 0);
+		buf[b] = MI_STORE_DWORD_IMM | (gen < 6 ? BIT(22) : 0);
 		if (gen >= 8) {
 			buf[++b] = offset;
 			buf[++b] = offset >> 32;
diff --git a/tests/gem_exec_whisper.c b/tests/gem_exec_whisper.c
index 7f8f568f..b37a6a46 100644
--- a/tests/gem_exec_whisper.c
+++ b/tests/gem_exec_whisper.c
@@ -31,8 +31,8 @@
 #include "igt_gt.h"
 #include "igt_sysfs.h"
 
-#define LOCAL_I915_EXEC_NO_RELOC (1<<11)
-#define LOCAL_I915_EXEC_HANDLE_LUT (1<<12)
+#define LOCAL_I915_EXEC_NO_RELOC BIT(11)
+#define LOCAL_I915_EXEC_HANDLE_LUT BIT(12)
 
 #define LOCAL_I915_EXEC_BSD_SHIFT      (13)
 #define LOCAL_I915_EXEC_BSD_MASK       (3 << LOCAL_I915_EXEC_BSD_SHIFT)
@@ -164,11 +164,11 @@ static void init_hang(struct hang *h)
 	batch[i++] = 0x00c00fee;
 	batch[i] = MI_BATCH_BUFFER_START;
 	if (gen >= 8) {
-		batch[i] |= 1 << 8 | 1;
+		batch[i] |= BIT(8) | 1;
 		batch[++i] = 0;
 		batch[++i] = 0;
 	} else if (gen >= 6) {
-		batch[i] |= 1 << 8;
+		batch[i] |= BIT(8);
 		batch[++i] = 0;
 	} else {
 		batch[i] |= 2 << 6;
@@ -273,7 +273,7 @@ static void whisper(int fd, unsigned engine, unsigned flags)
 		}
 
 		i = 0;
-		batch[i] = MI_STORE_DWORD_IMM | (gen < 6 ? 1 << 22 : 0);
+		batch[i] = MI_STORE_DWORD_IMM | (gen < 6 ? BIT(22) : 0);
 		if (gen >= 8) {
 			batch[++i] = store.offset + loc;
 			batch[++i] = (store.offset + loc) >> 32;
diff --git a/tests/gem_fence_upload.c b/tests/gem_fence_upload.c
index 7d9acdc0..d2f140cd 100644
--- a/tests/gem_fence_upload.c
+++ b/tests/gem_fence_upload.c
@@ -142,8 +142,8 @@ static void *write_thread_performance(void *closure)
 	return NULL;
 }
 
-#define READ (1<<0)
-#define WRITE (1<<1)
+#define READ BIT(0)
+#define WRITE BIT(1)
 static const char *direction_string(unsigned mask)
 {
 	switch (mask) {
diff --git a/tests/gem_fenced_exec_thrash.c b/tests/gem_fenced_exec_thrash.c
index 2966bc92..476bb47d 100644
--- a/tests/gem_fenced_exec_thrash.c
+++ b/tests/gem_fenced_exec_thrash.c
@@ -141,8 +141,8 @@ static void fill_reloc(struct drm_i915_gem_relocation_entry *reloc, uint32_t han
 	reloc->write_domain = 0;
 }
 
-#define BUSY_LOAD (1 << 0)
-#define INTERRUPTIBLE (1 << 1)
+#define BUSY_LOAD BIT(0)
+#define INTERRUPTIBLE BIT(1)
 
 static void run_test(int fd, int num_fences, int expected_errno,
 		     unsigned flags)
diff --git a/tests/gem_gtt_hog.c b/tests/gem_gtt_hog.c
index 2fb487d7..a48d1e33 100644
--- a/tests/gem_gtt_hog.c
+++ b/tests/gem_gtt_hog.c
@@ -74,7 +74,7 @@ static void busy(data_t *data, uint32_t handle, int size, int loops)
 		*b++ = XY_COLOR_BLT_CMD_NOLEN |
 			((data->intel_gen >= 8) ? 5 : 4) |
 			COLOR_BLT_WRITE_ALPHA | XY_COLOR_BLT_WRITE_RGB;
-		*b++ = 0xf0 << 16 | 1 << 25 | 1 << 24 | 4096;
+		*b++ = 0xf0 << 16 | BIT(25) | BIT(24) | 4096;
 		*b++ = 0;
 		*b++ = size >> 12 << 16 | 1024;
 		reloc[i].offset = (b - buf) * sizeof(uint32_t);
@@ -103,7 +103,7 @@ static void busy(data_t *data, uint32_t handle, int size, int loops)
 	execbuf.buffers_ptr = to_user_pointer(gem_exec);
 	execbuf.buffer_count = 2;
 	execbuf.batch_len = (b - buf) * sizeof(buf[0]);
-	execbuf.flags = 1 << 11;
+	execbuf.flags = BIT(11);
 	if (HAS_BLT_RING(data->devid))
 		execbuf.flags |= I915_EXEC_BLT;
 
diff --git a/tests/gem_lut_handle.c b/tests/gem_lut_handle.c
index 10516b4f..41c28161 100644
--- a/tests/gem_lut_handle.c
+++ b/tests/gem_lut_handle.c
@@ -43,7 +43,7 @@ IGT_TEST_DESCRIPTION("Exercises the basic execbuffer using the handle LUT"
 
 #define BATCH_SIZE		(1024*1024)
 
-#define LOCAL_I915_EXEC_HANDLE_LUT (1<<12)
+#define LOCAL_I915_EXEC_HANDLE_LUT BIT(12)
 
 #define NORMAL 0
 #define USE_LUT 0x1
diff --git a/tests/gem_persistent_relocs.c b/tests/gem_persistent_relocs.c
index 9c885307..8eb432f2 100644
--- a/tests/gem_persistent_relocs.c
+++ b/tests/gem_persistent_relocs.c
@@ -89,7 +89,7 @@ static drm_intel_bo *create_special_bo(void)
 	BATCH((3 << 24) | (0xf0 << 16) | small_pitch);
 	special_line_ofs = 4*len;
 	BATCH(0);
-	BATCH(1 << 16 | 1);
+	BATCH(BIT(16) | 1);
 	special_reloc_ofs = 4*len;
 	BATCH(0);
 	if (intel_gen(devid) >= 8)
@@ -271,10 +271,10 @@ static void do_test(int fd, bool faulting_reloc)
 		igt_enable_prefault();
 }
 
-#define INTERRUPT	(1 << 0)
-#define FAULTING	(1 << 1)
-#define THRASH		(1 << 2)
-#define THRASH_INACTIVE	(1 << 3)
+#define INTERRUPT	BIT(0)
+#define FAULTING	BIT(1)
+#define THRASH		BIT(2)
+#define THRASH_INACTIVE	BIT(3)
 #define ALL_FLAGS	(INTERRUPT | FAULTING | THRASH | THRASH_INACTIVE)
 static void do_forked_test(int fd, unsigned flags)
 {
diff --git a/tests/gem_pin.c b/tests/gem_pin.c
index 9ea3b1e7..643138f2 100644
--- a/tests/gem_pin.c
+++ b/tests/gem_pin.c
@@ -43,8 +43,8 @@
 IGT_TEST_DESCRIPTION("Exercises pinning of small buffer objects.");
 
 #define COPY_BLT_CMD            (2<<29|0x53<<22|0x6)
-#define BLT_WRITE_ALPHA         (1<<21)
-#define BLT_WRITE_RGB           (1<<20)
+#define BLT_WRITE_ALPHA         BIT(21)
+#define BLT_WRITE_RGB           BIT(20)
 
 static void exec(int fd, uint32_t handle, uint32_t offset)
 {
@@ -94,7 +94,7 @@ static int gem_linear_blt(int fd,
 	uint32_t *b = batch;
 
 	*b++ = COPY_BLT_CMD | BLT_WRITE_ALPHA | BLT_WRITE_RGB;
-	*b++ = 0x66 << 16 | 1 << 25 | 1 << 24 | (4*1024);
+	*b++ = 0x66 << 16 | BIT(25) | BIT(24) | (4*1024);
 	*b++ = 0;
 	*b++ = (length / (4*1024)) << 16 | 1024;
 	*b++ = 0;
diff --git a/tests/gem_pipe_control_store_loop.c b/tests/gem_pipe_control_store_loop.c
index a155ad14..c064a22c 100644
--- a/tests/gem_pipe_control_store_loop.c
+++ b/tests/gem_pipe_control_store_loop.c
@@ -50,15 +50,15 @@ struct intel_batchbuffer *batch;
 uint32_t devid;
 
 #define GFX_OP_PIPE_CONTROL	((0x3<<29)|(0x3<<27)|(0x2<<24)|2)
-#define   PIPE_CONTROL_WRITE_IMMEDIATE	(1<<14)
+#define   PIPE_CONTROL_WRITE_IMMEDIATE	BIT(14)
 #define   PIPE_CONTROL_WRITE_TIMESTAMP	(3<<14)
-#define   PIPE_CONTROL_DEPTH_STALL (1<<13)
-#define   PIPE_CONTROL_WC_FLUSH	(1<<12)
-#define   PIPE_CONTROL_IS_FLUSH	(1<<11) /* MBZ on Ironlake */
-#define   PIPE_CONTROL_TC_FLUSH (1<<10) /* GM45+ only */
-#define   PIPE_CONTROL_STALL_AT_SCOREBOARD (1<<1)
-#define   PIPE_CONTROL_CS_STALL	(1<<20)
-#define   PIPE_CONTROL_GLOBAL_GTT (1<<2) /* in addr dword */
+#define   PIPE_CONTROL_DEPTH_STALL BIT(13)
+#define   PIPE_CONTROL_WC_FLUSH	BIT(12)
+#define   PIPE_CONTROL_IS_FLUSH	BIT(11) /* MBZ on Ironlake */
+#define   PIPE_CONTROL_TC_FLUSH BIT(10) /* GM45+ only */
+#define   PIPE_CONTROL_STALL_AT_SCOREBOARD BIT(1)
+#define   PIPE_CONTROL_CS_STALL	BIT(20)
+#define   PIPE_CONTROL_GLOBAL_GTT BIT(2) /* in addr dword */
 
 /* Like the store dword test, but we create new command buffers each time */
 static void
@@ -78,7 +78,7 @@ store_pipe_control_loop(bool preuse_buffer)
 			COLOR_BLIT_COPY_BATCH_START(0);
 			OUT_BATCH((3 << 24) | (0xf0 << 16) | 64);
 			OUT_BATCH(0);
-			OUT_BATCH(1 << 16 | 1);
+			OUT_BATCH(BIT(16) | 1);
 
 			/*
 			 * IMPORTANT: We need to preuse the buffer in a
diff --git a/tests/gem_pwrite.c b/tests/gem_pwrite.c
index 8db5454c..3b0e4067 100644
--- a/tests/gem_pwrite.c
+++ b/tests/gem_pwrite.c
@@ -42,10 +42,10 @@
 #define OBJECT_SIZE 16384
 
 #define COPY_BLT_CMD		(2<<29|0x53<<22|0x6)
-#define BLT_WRITE_ALPHA		(1<<21)
-#define BLT_WRITE_RGB		(1<<20)
-#define BLT_SRC_TILED		(1<<15)
-#define BLT_DST_TILED		(1<<11)
+#define BLT_WRITE_ALPHA		BIT(21)
+#define BLT_WRITE_RGB		BIT(20)
+#define BLT_SRC_TILED		BIT(15)
+#define BLT_DST_TILED		BIT(11)
 
 static void do_gem_write(int fd, uint32_t handle, void *buf, int len, int loops)
 {
diff --git a/tests/gem_pwrite_pread.c b/tests/gem_pwrite_pread.c
index e1543f22..d2249b62 100644
--- a/tests/gem_pwrite_pread.c
+++ b/tests/gem_pwrite_pread.c
@@ -43,10 +43,10 @@
 #define OBJECT_SIZE 16384
 
 #define COPY_BLT_CMD		(2<<29|0x53<<22)
-#define BLT_WRITE_ALPHA		(1<<21)
-#define BLT_WRITE_RGB		(1<<20)
-#define BLT_SRC_TILED		(1<<15)
-#define BLT_DST_TILED		(1<<11)
+#define BLT_WRITE_ALPHA		BIT(21)
+#define BLT_WRITE_RGB		BIT(20)
+#define BLT_SRC_TILED		BIT(15)
+#define BLT_DST_TILED		BIT(11)
 
 uint32_t is_64bit;
 uint32_t exec_flags;
@@ -56,9 +56,9 @@ static inline void build_batch(uint32_t *batch, int len, uint32_t *batch_len)
 	unsigned int i = 0;
 
 	batch[i++] = COPY_BLT_CMD | BLT_WRITE_ALPHA | BLT_WRITE_RGB | (is_64bit ? 8 : 6);
-	batch[i++] = 0xcc << 16 | 1 << 25 | 1 << 24 | len;
+	batch[i++] = 0xcc << 16 | BIT(25) | BIT(24) | len;
 	batch[i++] = 0;
-	batch[i++] = 1 << 16 | (len / 4);
+	batch[i++] = BIT(16) | (len / 4);
 	batch[i++] = 0; /* dst */
 	if (is_64bit)
 		batch[i++] = 0;
diff --git a/tests/gem_reloc_vs_gpu.c b/tests/gem_reloc_vs_gpu.c
index ef898231..7120db7d 100644
--- a/tests/gem_reloc_vs_gpu.c
+++ b/tests/gem_reloc_vs_gpu.c
@@ -81,7 +81,7 @@ static void create_special_bo(void)
 
 	BATCH((3 << 24) | (0xf0 << 16) | small_pitch);
 	BATCH(0);
-	BATCH(1 << 16 | 1);
+	BATCH(BIT(16) | 1);
 	special_reloc_ofs = 4*len;
 	BATCH(0);
 	if (intel_gen(devid) >= 8)
@@ -244,11 +244,11 @@ static void do_test(int fd, bool faulting_reloc,
 		igt_enable_prefault();
 }
 
-#define INTERRUPT	(1 << 0)
-#define FAULTING	(1 << 1)
-#define THRASH		(1 << 2)
-#define THRASH_INACTIVE	(1 << 3)
-#define HANG		(1 << 4)
+#define INTERRUPT	BIT(0)
+#define FAULTING	BIT(1)
+#define THRASH		BIT(2)
+#define THRASH_INACTIVE	BIT(3)
+#define HANG		BIT(4)
 #define ALL_FLAGS	(HANG | INTERRUPT | FAULTING | THRASH | THRASH_INACTIVE)
 static void do_forked_test(int fd, unsigned flags)
 {
diff --git a/tests/gem_reset_stats.c b/tests/gem_reset_stats.c
index c4ce4ac2..2bceb6cb 100644
--- a/tests/gem_reset_stats.c
+++ b/tests/gem_reset_stats.c
@@ -43,9 +43,9 @@
 
 
 #define RS_NO_ERROR      0
-#define RS_BATCH_ACTIVE  (1 << 0)
-#define RS_BATCH_PENDING (1 << 1)
-#define RS_UNKNOWN       (1 << 2)
+#define RS_BATCH_ACTIVE  BIT(0)
+#define RS_BATCH_PENDING BIT(1)
+#define RS_UNKNOWN       BIT(2)
 
 
 static uint32_t devid;
diff --git a/tests/gem_ringfill.c b/tests/gem_ringfill.c
index 8e221cf8..d765d4da 100644
--- a/tests/gem_ringfill.c
+++ b/tests/gem_ringfill.c
@@ -99,7 +99,7 @@ static int setup_execbuf(int fd,
 	memset(reloc, 0, 1024*sizeof(*reloc));
 
 	execbuf->buffers_ptr = to_user_pointer(obj);
-	execbuf->flags = ring | (1 << 11) | (1 << 12);
+	execbuf->flags = ring | BIT(11) | BIT(12);
 
 	if (gen > 3 && gen < 6)
 		execbuf->flags |= I915_EXEC_SECURE;
@@ -139,12 +139,12 @@ static int setup_execbuf(int fd,
 			*b++ = offset >> 32;
 		} else if (gen >= 4) {
 			if (gen < 6)
-				b[-1] |= 1 << 22;
+				b[-1] |= BIT(22);
 			*b++ = 0;
 			*b++ = offset;
 			reloc[i].offset += sizeof(*batch);
 		} else {
-			b[-1] |= 1 << 22;
+			b[-1] |= BIT(22);
 			b[-1] -= 1;
 			*b++ = offset;
 		}
diff --git a/tests/gem_shrink.c b/tests/gem_shrink.c
index bd0f17da..1877bb00 100644
--- a/tests/gem_shrink.c
+++ b/tests/gem_shrink.c
@@ -137,7 +137,7 @@ static void execbufN(int fd, uint64_t alloc)
 	for (int i = 1; i <= count; i++) {
 		int j = count - i;
 
-		obj[j].handle = gem_create(fd, 1 << 20);
+		obj[j].handle = gem_create(fd, BIT(20));
 		execbuf.buffers_ptr = to_user_pointer(&obj[j]);
 		execbuf.buffer_count = i + 1;
 		gem_execbuf(fd, &execbuf);
@@ -165,7 +165,7 @@ static void hang(int fd, uint64_t alloc)
 	for (int i = 1; i <= count; i++) {
 		int j = count - i;
 
-		obj[j].handle = gem_create(fd, 1 << 20);
+		obj[j].handle = gem_create(fd, BIT(20));
 		execbuf.buffers_ptr = to_user_pointer(&obj[j]);
 		execbuf.buffer_count = i + 1;
 		gem_execbuf(fd, &execbuf);
diff --git a/tests/gem_softpin.c b/tests/gem_softpin.c
index 7678acfe..55dee1ae 100644
--- a/tests/gem_softpin.c
+++ b/tests/gem_softpin.c
@@ -28,8 +28,8 @@
 
 #include "igt.h"
 
-#define EXEC_OBJECT_PINNED	(1<<4)
-#define EXEC_OBJECT_SUPPORTS_48B_ADDRESS (1<<3)
+#define EXEC_OBJECT_PINNED	BIT(4)
+#define EXEC_OBJECT_SUPPORTS_48B_ADDRESS BIT(3)
 
 /* gen8_canonical_addr
  * Used to convert any address into canonical form, i.e. [63:48] == [47].
@@ -227,8 +227,8 @@ static uint64_t busy_batch(int fd)
 		  object[0].offset, object[1].offset);
 
 #define COPY_BLT_CMD		(2<<29|0x53<<22|0x6)
-#define BLT_WRITE_ALPHA		(1<<21)
-#define BLT_WRITE_RGB		(1<<20)
+#define BLT_WRITE_ALPHA		BIT(21)
+#define BLT_WRITE_RGB		BIT(20)
 	gem_set_domain(fd, object[1].handle,
 		       I915_GEM_DOMAIN_CPU, I915_GEM_DOMAIN_CPU);
 	while (factor--) {
@@ -236,7 +236,7 @@ static uint64_t busy_batch(int fd)
 		map[i++] = COPY_BLT_CMD | BLT_WRITE_ALPHA | BLT_WRITE_RGB;
 		if (has_64bit_reloc)
 			map[i-1] += 2;
-		map[i++] = 0xcc << 16 | 1 << 25 | 1 << 24 | (4*1024);
+		map[i++] = 0xcc << 16 | BIT(25) | BIT(24) | (4*1024);
 		map[i++] = 0;
 		map[i++] = 256 << 16 | 1024;
 		map[i++] = object[0].offset;
@@ -424,7 +424,7 @@ static void test_noreloc(int fd, enum sleep sleep)
 	gem_set_domain(fd, object[i].handle,
 		       I915_GEM_DOMAIN_CPU, I915_GEM_DOMAIN_CPU);
 	for (i = 0; i < ARRAY_SIZE(object) - 1; i++) {
-		*b++ = MI_STORE_DWORD_IMM | (gen < 6 ? 1 << 22 : 0);
+		*b++ = MI_STORE_DWORD_IMM | (gen < 6 ? BIT(22) : 0);
 		if (gen >= 8) {
 			*b++ = object[i].offset;
 			*b++ = object[i].offset >> 32;
diff --git a/tests/gem_storedw_batches_loop.c b/tests/gem_storedw_batches_loop.c
index 4b0249a6..c3d38363 100644
--- a/tests/gem_storedw_batches_loop.c
+++ b/tests/gem_storedw_batches_loop.c
@@ -42,7 +42,7 @@ static drm_intel_bufmgr *bufmgr;
 static drm_intel_bo *target_bo;
 static int has_ppgtt = 0;
 
-#define SECURE_DISPATCH (1<<0)
+#define SECURE_DISPATCH BIT(0)
 
 /* Like the store dword test, but we create new command buffers each time */
 static void
diff --git a/tests/gem_streaming_writes.c b/tests/gem_streaming_writes.c
index 2b35410c..3b726228 100644
--- a/tests/gem_streaming_writes.c
+++ b/tests/gem_streaming_writes.c
@@ -43,11 +43,11 @@
 #define CHUNK_SIZE 32
 
 #define COPY_BLT_CMD		(2<<29|0x53<<22|0x6)
-#define BLT_WRITE_ALPHA		(1<<21)
-#define BLT_WRITE_RGB		(1<<20)
+#define BLT_WRITE_ALPHA		BIT(21)
+#define BLT_WRITE_RGB		BIT(20)
 #define BLT_WRITE_ARGB (BLT_WRITE_ALPHA | BLT_WRITE_RGB)
 
-#define LOCAL_I915_EXEC_HANDLE_LUT (1<<12)
+#define LOCAL_I915_EXEC_HANDLE_LUT BIT(12)
 
 IGT_TEST_DESCRIPTION("Test of streaming writes into active GPU sources");
 
@@ -162,7 +162,7 @@ static void test_streaming(int fd, int mode, int sync)
 			if (has_64bit_reloc)
 				b[k] += 2;
 			k++;
-			b[k++] = 0xcc << 16 | 1 << 25 | 1 << 24 | 4096;
+			b[k++] = 0xcc << 16 | BIT(25) | BIT(24) | 4096;
 			b[k++] = (y << 16) | x;
 			b[k++] = ((y+1) << 16) | (x + (CHUNK_SIZE >> 2));
 			b[k++] = dst_offset;
@@ -337,7 +337,7 @@ static void test_batch(int fd, int mode, int reverse)
 			if (has_64bit_reloc)
 				base[k] += 2;
 			k++;
-			base[k++] = 0xcc << 16 | 1 << 25 | 1 << 24 | 4096;
+			base[k++] = 0xcc << 16 | BIT(25) | BIT(24) | 4096;
 			base[k++] = (y << 16) | x;
 			base[k++] = ((y+1) << 16) | (x + (CHUNK_SIZE >> 2));
 			base[k++] = dst_offset;
diff --git a/tests/gem_stress.c b/tests/gem_stress.c
index 4d0de5c4..74f66d18 100644
--- a/tests/gem_stress.c
+++ b/tests/gem_stress.c
@@ -187,7 +187,7 @@ static void keep_gpu_busy(void)
 {
 	int tmp;
 
-	tmp = 1 << gpu_busy_load;
+	tmp = BIT(gpu_busy_load);
 	igt_assert_lte(tmp, 1024);
 
 	emit_blt(busy_bo, 0, 4096, 0, 0, tmp, 128,
@@ -540,15 +540,15 @@ static void init_set(unsigned set)
 		if (buffers[set][i].tiling == I915_TILING_NONE) {
 			/* min 64 byte stride */
 			r %= 8;
-			buffers[set][i].stride = 64 * (1 << r);
+			buffers[set][i].stride = 64 * BIT(r);
 		} else if (IS_GEN2(devid)) {
 			/* min 128 byte stride */
 			r %= 7;
-			buffers[set][i].stride = 128 * (1 << r);
+			buffers[set][i].stride = 128 * BIT(r);
 		} else {
 			/* min 512 byte stride */
 			r %= 5;
-			buffers[set][i].stride = 512 * (1 << r);
+			buffers[set][i].stride = 512 * BIT(r);
 		}
 
 		sanitize_stride(&buffers[set][i]);
diff --git a/tests/gem_sync.c b/tests/gem_sync.c
index 33d356ad..011da675 100644
--- a/tests/gem_sync.c
+++ b/tests/gem_sync.c
@@ -27,8 +27,8 @@
 #include "igt.h"
 #include "igt_sysfs.h"
 
-#define LOCAL_I915_EXEC_NO_RELOC (1<<11)
-#define LOCAL_I915_EXEC_HANDLE_LUT (1<<12)
+#define LOCAL_I915_EXEC_NO_RELOC BIT(11)
+#define LOCAL_I915_EXEC_HANDLE_LUT BIT(12)
 
 #define LOCAL_I915_EXEC_BSD_SHIFT      (13)
 #define LOCAL_I915_EXEC_BSD_MASK       (3 << LOCAL_I915_EXEC_BSD_SHIFT)
@@ -241,7 +241,7 @@ store_ring(int fd, unsigned ring, int num_children, int timeout)
 			reloc[i].write_domain = I915_GEM_DOMAIN_INSTRUCTION;
 
 			offset = object[0].offset + reloc[i].delta;
-			*b++ = MI_STORE_DWORD_IMM | (gen < 6 ? 1 << 22 : 0);
+			*b++ = MI_STORE_DWORD_IMM | (gen < 6 ? BIT(22) : 0);
 			if (gen >= 8) {
 				*b++ = offset;
 				*b++ = offset >> 32;
@@ -367,7 +367,7 @@ __store_many(int fd, unsigned ring, int timeout, unsigned long *cycles)
 		reloc[i].write_domain = I915_GEM_DOMAIN_INSTRUCTION;
 
 		offset = object[0].offset + reloc[i].delta;
-		*b++ = MI_STORE_DWORD_IMM | (gen < 6 ? 1 << 22 : 0);
+		*b++ = MI_STORE_DWORD_IMM | (gen < 6 ? BIT(22) : 0);
 		if (gen >= 8) {
 			*b++ = offset;
 			*b++ = offset >> 32;
@@ -646,7 +646,7 @@ store_all(int fd, int num_children, int timeout)
 			reloc[i].write_domain = I915_GEM_DOMAIN_INSTRUCTION;
 
 			offset = object[0].offset + reloc[i].delta;
-			*b++ = MI_STORE_DWORD_IMM | (gen < 6 ? 1 << 22 : 0);
+			*b++ = MI_STORE_DWORD_IMM | (gen < 6 ? BIT(22) : 0);
 			if (gen >= 8) {
 				*b++ = offset;
 				*b++ = offset >> 32;
diff --git a/tests/gem_tiled_pread_basic.c b/tests/gem_tiled_pread_basic.c
index 425bb07e..894dfcb6 100644
--- a/tests/gem_tiled_pread_basic.c
+++ b/tests/gem_tiled_pread_basic.c
@@ -84,7 +84,7 @@ create_bo(int fd)
 static int
 swizzle_bit(int bit, int offset)
 {
-	return (offset & (1 << bit)) >> (bit - 6);
+	return (offset & BIT(bit)) >> (bit - 6);
 }
 
 struct offset {
diff --git a/tests/gem_tiled_wb.c b/tests/gem_tiled_wb.c
index 67d54bd3..5eb3348d 100644
--- a/tests/gem_tiled_wb.c
+++ b/tests/gem_tiled_wb.c
@@ -83,7 +83,7 @@ create_bo(int fd)
 static int
 swizzle_bit(int bit, int offset)
 {
-	return (offset & (1 << bit)) >> (bit - 6);
+	return (offset & BIT(bit)) >> (bit - 6);
 }
 
 /* Translate from a swizzled offset in the tiled buffer to the corresponding
diff --git a/tests/gem_tiled_wc.c b/tests/gem_tiled_wc.c
index 65ac3851..98ba2bf6 100644
--- a/tests/gem_tiled_wc.c
+++ b/tests/gem_tiled_wc.c
@@ -76,7 +76,7 @@ create_bo(int fd)
 static int
 swizzle_bit(int bit, int offset)
 {
-	return (offset & (1 << bit)) >> (bit - 6);
+	return (offset & BIT(bit)) >> (bit - 6);
 }
 
 /* Translate from a swizzled offset in the tiled buffer to the corresponding
diff --git a/tests/gem_write_read_ring_switch.c b/tests/gem_write_read_ring_switch.c
index d00d4f95..8fbfe563 100644
--- a/tests/gem_write_read_ring_switch.c
+++ b/tests/gem_write_read_ring_switch.c
@@ -96,7 +96,7 @@ static void run_test(int ring)
 		  (0xff << 16) |
 		  128);
 	OUT_BATCH(0); /* dst x1,y1 */
-	OUT_BATCH((1 << 16) | 1);
+	OUT_BATCH(BIT(16) | 1);
 	OUT_RELOC_FENCED(target_bo, I915_GEM_DOMAIN_RENDER, I915_GEM_DOMAIN_RENDER, 0);
 	OUT_BATCH(COLOR);
 	ADVANCE_BATCH();
@@ -125,7 +125,7 @@ static void run_test(int ring)
 		OUT_BATCH(MI_FLUSH_DW | 1);
 		OUT_BATCH(0); /* reserved */
 		OUT_RELOC(target_bo, I915_GEM_DOMAIN_RENDER, 0, 0);
-		OUT_BATCH(MI_NOOP | (1<<22) | (0xf));
+		OUT_BATCH(MI_NOOP | BIT(22) | (0xf));
 		ADVANCE_BATCH();
 	}
 	intel_batchbuffer_flush_on_ring(batch, ring);
diff --git a/tests/gen3_mixed_blits.c b/tests/gen3_mixed_blits.c
index 1159b4eb..2833e28e 100644
--- a/tests/gen3_mixed_blits.c
+++ b/tests/gen3_mixed_blits.c
@@ -132,7 +132,7 @@ render_copy(int fd,
 		ENABLE_STENCIL_TEST_MASK | STENCIL_TEST_MASK(0xff));
 	*b++ = (_3DSTATE_LOAD_STATE_IMMEDIATE_1 | I1_LOAD_S(3) | I1_LOAD_S(4) | I1_LOAD_S(5) | 2);
 	*b++ = (0x00000000);	/* Disable texture coordinate wrap-shortest */
-	*b++ = ((1 << S4_POINT_WIDTH_SHIFT) |
+	*b++ = (BIT(S4_POINT_WIDTH_SHIFT) |
 		S4_LINE_WIDTH_ONE |
 		S4_CULLMODE_NONE |
 		S4_VFMT_XY);
@@ -161,7 +161,7 @@ render_copy(int fd,
 
 #define TEX_COUNT 1
 	*b++ = (_3DSTATE_MAP_STATE | (3 * TEX_COUNT));
-	*b++ = ((1 << TEX_COUNT) - 1);
+	*b++ = (BIT(TEX_COUNT) - 1);
 	*b = fill_reloc(r++, b-batch, src, I915_GEM_DOMAIN_SAMPLER, 0); b++;
 	*b++ = (MAPSURF_32BIT | MT_32BIT_ARGB8888 | tiling_bits |
 		(HEIGHT - 1) << MS3_HEIGHT_SHIFT |
@@ -169,7 +169,7 @@ render_copy(int fd,
 	*b++ = ((WIDTH-1) << MS4_PITCH_SHIFT);
 
 	*b++ = (_3DSTATE_SAMPLER_STATE | (3 * TEX_COUNT));
-	*b++ = ((1 << TEX_COUNT) - 1);
+	*b++ = (BIT(TEX_COUNT) - 1);
 	*b++ = (MIPFILTER_NONE << SS2_MIP_FILTER_SHIFT |
 		FILTER_NEAREST << SS2_MAG_FILTER_SHIFT |
 		FILTER_NEAREST << SS2_MIN_FILTER_SHIFT);
diff --git a/tests/gen3_render_linear_blits.c b/tests/gen3_render_linear_blits.c
index e56bff93..c0015e3a 100644
--- a/tests/gen3_render_linear_blits.c
+++ b/tests/gen3_render_linear_blits.c
@@ -125,7 +125,7 @@ copy(int fd, uint32_t dst, uint32_t src)
 		ENABLE_STENCIL_TEST_MASK | STENCIL_TEST_MASK(0xff));
 	*b++ = (_3DSTATE_LOAD_STATE_IMMEDIATE_1 | I1_LOAD_S(3) | I1_LOAD_S(4) | I1_LOAD_S(5) | 2);
 	*b++ = (0x00000000);	/* Disable texture coordinate wrap-shortest */
-	*b++ = ((1 << S4_POINT_WIDTH_SHIFT) |
+	*b++ = (BIT(S4_POINT_WIDTH_SHIFT) |
 		S4_LINE_WIDTH_ONE |
 		S4_CULLMODE_NONE |
 		S4_VFMT_XY);
@@ -144,7 +144,7 @@ copy(int fd, uint32_t dst, uint32_t src)
 	/* samler state */
 #define TEX_COUNT 1
 	*b++ = (_3DSTATE_MAP_STATE | (3 * TEX_COUNT));
-	*b++ = ((1 << TEX_COUNT) - 1);
+	*b++ = (BIT(TEX_COUNT) - 1);
 	*b = fill_reloc(r++, b-batch, src, I915_GEM_DOMAIN_SAMPLER, 0); b++;
 	*b++ = (MAPSURF_32BIT | MT_32BIT_ARGB8888 |
 		(HEIGHT - 1) << MS3_HEIGHT_SHIFT |
@@ -152,7 +152,7 @@ copy(int fd, uint32_t dst, uint32_t src)
 	*b++ = ((WIDTH-1) << MS4_PITCH_SHIFT);
 
 	*b++ = (_3DSTATE_SAMPLER_STATE | (3 * TEX_COUNT));
-	*b++ = ((1 << TEX_COUNT) - 1);
+	*b++ = (BIT(TEX_COUNT) - 1);
 	*b++ = (MIPFILTER_NONE << SS2_MIP_FILTER_SHIFT |
 		FILTER_NEAREST << SS2_MAG_FILTER_SHIFT |
 		FILTER_NEAREST << SS2_MIN_FILTER_SHIFT);
diff --git a/tests/gen3_render_mixed_blits.c b/tests/gen3_render_mixed_blits.c
index 6cc8d056..74e99aa6 100644
--- a/tests/gen3_render_mixed_blits.c
+++ b/tests/gen3_render_mixed_blits.c
@@ -127,7 +127,7 @@ copy(int fd,
 		ENABLE_STENCIL_TEST_MASK | STENCIL_TEST_MASK(0xff));
 	*b++ = (_3DSTATE_LOAD_STATE_IMMEDIATE_1 | I1_LOAD_S(3) | I1_LOAD_S(4) | I1_LOAD_S(5) | 2);
 	*b++ = (0x00000000);	/* Disable texture coordinate wrap-shortest */
-	*b++ = ((1 << S4_POINT_WIDTH_SHIFT) |
+	*b++ = (BIT(S4_POINT_WIDTH_SHIFT) |
 		S4_LINE_WIDTH_ONE |
 		S4_CULLMODE_NONE |
 		S4_VFMT_XY);
@@ -152,7 +152,7 @@ copy(int fd,
 
 #define TEX_COUNT 1
 	*b++ = (_3DSTATE_MAP_STATE | (3 * TEX_COUNT));
-	*b++ = ((1 << TEX_COUNT) - 1);
+	*b++ = (BIT(TEX_COUNT) - 1);
 	*b = fill_reloc(r++, b-batch, src, I915_GEM_DOMAIN_SAMPLER, 0); b++;
 	*b++ = (MAPSURF_32BIT | MT_32BIT_ARGB8888 | tiling_bits |
 		(HEIGHT - 1) << MS3_HEIGHT_SHIFT |
@@ -160,7 +160,7 @@ copy(int fd,
 	*b++ = ((WIDTH-1) << MS4_PITCH_SHIFT);
 
 	*b++ = (_3DSTATE_SAMPLER_STATE | (3 * TEX_COUNT));
-	*b++ = ((1 << TEX_COUNT) - 1);
+	*b++ = (BIT(TEX_COUNT) - 1);
 	*b++ = (MIPFILTER_NONE << SS2_MIP_FILTER_SHIFT |
 		FILTER_NEAREST << SS2_MAG_FILTER_SHIFT |
 		FILTER_NEAREST << SS2_MIN_FILTER_SHIFT);
diff --git a/tests/gen3_render_tiledx_blits.c b/tests/gen3_render_tiledx_blits.c
index 6706d3a3..40517e21 100644
--- a/tests/gen3_render_tiledx_blits.c
+++ b/tests/gen3_render_tiledx_blits.c
@@ -124,7 +124,7 @@ copy(int fd, uint32_t dst, uint32_t src)
 		ENABLE_STENCIL_TEST_MASK | STENCIL_TEST_MASK(0xff));
 	*b++ = (_3DSTATE_LOAD_STATE_IMMEDIATE_1 | I1_LOAD_S(3) | I1_LOAD_S(4) | I1_LOAD_S(5) | 2);
 	*b++ = (0x00000000);	/* Disable texture coordinate wrap-shortest */
-	*b++ = ((1 << S4_POINT_WIDTH_SHIFT) |
+	*b++ = (BIT(S4_POINT_WIDTH_SHIFT) |
 		S4_LINE_WIDTH_ONE |
 		S4_CULLMODE_NONE |
 		S4_VFMT_XY);
@@ -143,7 +143,7 @@ copy(int fd, uint32_t dst, uint32_t src)
 	/* samler state */
 #define TEX_COUNT 1
 	*b++ = (_3DSTATE_MAP_STATE | (3 * TEX_COUNT));
-	*b++ = ((1 << TEX_COUNT) - 1);
+	*b++ = (BIT(TEX_COUNT) - 1);
 	*b = fill_reloc(r++, b-batch, src, I915_GEM_DOMAIN_SAMPLER, 0); b++;
 	*b++ = (MAPSURF_32BIT | MT_32BIT_ARGB8888 |
 		MS3_TILED_SURFACE |
@@ -152,7 +152,7 @@ copy(int fd, uint32_t dst, uint32_t src)
 	*b++ = ((WIDTH-1) << MS4_PITCH_SHIFT);
 
 	*b++ = (_3DSTATE_SAMPLER_STATE | (3 * TEX_COUNT));
-	*b++ = ((1 << TEX_COUNT) - 1);
+	*b++ = (BIT(TEX_COUNT) - 1);
 	*b++ = (MIPFILTER_NONE << SS2_MIP_FILTER_SHIFT |
 		FILTER_NEAREST << SS2_MAG_FILTER_SHIFT |
 		FILTER_NEAREST << SS2_MIN_FILTER_SHIFT);
diff --git a/tests/gen3_render_tiledy_blits.c b/tests/gen3_render_tiledy_blits.c
index 44e88d4d..255f03b7 100644
--- a/tests/gen3_render_tiledy_blits.c
+++ b/tests/gen3_render_tiledy_blits.c
@@ -124,7 +124,7 @@ copy(int fd, uint32_t dst, uint32_t src)
 		ENABLE_STENCIL_TEST_MASK | STENCIL_TEST_MASK(0xff));
 	*b++ = (_3DSTATE_LOAD_STATE_IMMEDIATE_1 | I1_LOAD_S(3) | I1_LOAD_S(4) | I1_LOAD_S(5) | 2);
 	*b++ = (0x00000000);	/* Disable texture coordinate wrap-shortest */
-	*b++ = ((1 << S4_POINT_WIDTH_SHIFT) |
+	*b++ = (BIT(S4_POINT_WIDTH_SHIFT) |
 		S4_LINE_WIDTH_ONE |
 		S4_CULLMODE_NONE |
 		S4_VFMT_XY);
@@ -143,7 +143,7 @@ copy(int fd, uint32_t dst, uint32_t src)
 	/* samler state */
 #define TEX_COUNT 1
 	*b++ = (_3DSTATE_MAP_STATE | (3 * TEX_COUNT));
-	*b++ = ((1 << TEX_COUNT) - 1);
+	*b++ = (BIT(TEX_COUNT) - 1);
 	*b = fill_reloc(r++, b-batch, src, I915_GEM_DOMAIN_SAMPLER, 0); b++;
 	*b++ = (MAPSURF_32BIT | MT_32BIT_ARGB8888 |
 		MS3_TILED_SURFACE | MS3_TILE_WALK |
@@ -152,7 +152,7 @@ copy(int fd, uint32_t dst, uint32_t src)
 	*b++ = ((WIDTH-1) << MS4_PITCH_SHIFT);
 
 	*b++ = (_3DSTATE_SAMPLER_STATE | (3 * TEX_COUNT));
-	*b++ = ((1 << TEX_COUNT) - 1);
+	*b++ = (BIT(TEX_COUNT) - 1);
 	*b++ = (MIPFILTER_NONE << SS2_MIP_FILTER_SHIFT |
 		FILTER_NEAREST << SS2_MAG_FILTER_SHIFT |
 		FILTER_NEAREST << SS2_MIN_FILTER_SHIFT);
diff --git a/tests/gen7_forcewake_mt.c b/tests/gen7_forcewake_mt.c
index 07320ef9..aeaec44f 100644
--- a/tests/gen7_forcewake_mt.c
+++ b/tests/gen7_forcewake_mt.c
@@ -107,7 +107,7 @@ static void *thread(void *arg)
 {
 	struct thread *t = arg;
 	uint32_t *forcewake_mt = (uint32_t *)((char *)t->mmio + FORCEWAKE_MT);
-	uint32_t bit = 1 << t->bit;
+	uint32_t bit = BIT(t->bit);
 
 	while (1) {
 		*forcewake_mt = bit << 16 | bit;
diff --git a/tests/kms_atomic.c b/tests/kms_atomic.c
index d6273f4f..f1f35c7e 100644
--- a/tests/kms_atomic.c
+++ b/tests/kms_atomic.c
@@ -79,8 +79,8 @@ IGT_TEST_DESCRIPTION("Test atomic modesetting API");
 
 enum kms_atomic_check_relax {
 	ATOMIC_RELAX_NONE = 0,
-	CRTC_RELAX_MODE = (1 << 0),
-	PLANE_RELAX_FB = (1 << 1)
+	CRTC_RELAX_MODE = BIT(0),
+	PLANE_RELAX_FB = BIT(1)
 };
 
 /**
@@ -420,7 +420,7 @@ find_plane(struct kms_atomic_state *state, enum plane_type type,
 			continue;
 		if (type != NUM_PLANE_TYPE_PROPS && plane->type != type)
 			continue;
-		if (crtc && !(plane->crtc_mask & (1 << crtc->idx)))
+		if (crtc && !(plane->crtc_mask & BIT(crtc->idx)))
 			continue;
 
 		plane_get_current_state(plane);
diff --git a/tests/kms_atomic_transition.c b/tests/kms_atomic_transition.c
index 797134c0..6b262a28 100644
--- a/tests/kms_atomic_transition.c
+++ b/tests/kms_atomic_transition.c
@@ -59,7 +59,7 @@ wm_setup_plane(igt_display_t *display, enum pipe pipe,
 	for_each_plane_on_pipe(display, pipe, plane) {
 		int i = plane->index;
 
-		if (!((1 << plane->index) & mask)) {
+		if (!(BIT(plane->index) & mask)) {
 			igt_plane_set_fb(plane, NULL);
 			continue;
 		}
@@ -197,7 +197,7 @@ retry:
 		set_sprite_wh(display, pipe, parms, sprite_fb,
 			      alpha, sprite_width, sprite_height);
 
-		wm_setup_plane(display, pipe, (1 << n_planes) - 1, parms);
+		wm_setup_plane(display, pipe, BIT(n_planes) - 1, parms);
 		ret = igt_display_try_commit_atomic(display, DRM_MODE_ATOMIC_TEST_ONLY | DRM_MODE_ATOMIC_ALLOW_MODESET, NULL);
 
 		if (ret == -EINVAL) {
@@ -359,7 +359,7 @@ run_transition_test(igt_display_t *display, enum pipe pipe, igt_output_t *output
 	struct igt_fb fb, argb_fb, sprite_fb;
 	drmModeModeInfo *mode, override_mode;
 	igt_plane_t *plane;
-	uint32_t iter_max = 1 << display->pipes[pipe].n_planes, i;
+	uint32_t iter_max = BIT(display->pipes[pipe].n_planes), i;
 	struct plane_parms parms[display->pipes[pipe].n_planes];
 	bool skip_test = false;
 	unsigned flags = DRM_MODE_PAGE_FLIP_EVENT;
@@ -545,16 +545,16 @@ static unsigned set_combinations(igt_display_t *display, unsigned mask, struct i
 			DRM_PLANE_TYPE_PRIMARY);
 		drmModeModeInfo *mode = NULL;
 
-		if (!(mask & (1 << pipe))) {
+		if (!(mask & BIT(pipe))) {
 			if (display->pipes[pipe].mode_blob) {
-				event_mask |= 1 << pipe;
+				event_mask |= BIT(pipe);
 				igt_plane_set_fb(plane, NULL);
 			}
 
 			continue;
 		}
 
-		event_mask |= 1 << pipe;
+		event_mask |= BIT(pipe);
 
 		for_each_valid_output_on_pipe(display, pipe, output) {
 			if (output->pending_crtc_idx_mask)
@@ -592,7 +592,7 @@ static void collect_crcs_mask(igt_pipe_crc_t **pipe_crcs, unsigned mask, igt_crc
 	int i;
 
 	for (i = 0; i < I915_MAX_PIPES; i++) {
-		if (!((1 << i) & mask))
+		if (!(BIT(i) & mask))
 			continue;
 
 		if (!pipe_crcs[i])
@@ -606,7 +606,7 @@ static void run_modeset_tests(igt_display_t *display, int howmany, bool nonblock
 {
 	struct igt_fb fbs[2];
 	int i, j;
-	unsigned iter_max = 1 << display->n_pipes;
+	unsigned iter_max = BIT(display->n_pipes);
 	igt_pipe_crc_t *pipe_crcs[I915_MAX_PIPES];
 	igt_output_t *output;
 	unsigned width = 0, height = 0;
@@ -712,12 +712,12 @@ static void run_modeset_tests(igt_display_t *display, int howmany, bool nonblock
 				continue;
 
 			for (int k = 0; k < I915_MAX_PIPES; k++) {
-				if (i & (1 << k)) {
+				if (i & BIT(k)) {
 					igt_assert_crc_equal(&crcs[0][k], &crcs[3][k]);
 					igt_assert_crc_equal(&crcs[0][k], &crcs[4][k]);
 				}
 
-				if (j & (1 << k))
+				if (j & BIT(k))
 					igt_assert_crc_equal(&crcs[1][k], &crcs[2][k]);
 			}
 		}
diff --git a/tests/kms_ccs.c b/tests/kms_ccs.c
index 11acda89..415a0e45 100644
--- a/tests/kms_ccs.c
+++ b/tests/kms_ccs.c
@@ -29,10 +29,10 @@ IGT_TEST_DESCRIPTION("Test render compression (RC), in which the main surface "
 		     "the display uses to interpret the compressed data.");
 
 enum test_flags {
-	TEST_CRC			= 1 << 1,
-	TEST_ROTATE_180			= 1 << 2,
-	TEST_BAD_PIXEL_FORMAT		= 1 << 3,
-	TEST_BAD_ROTATION_90		= 1 << 4,
+	TEST_CRC			= BIT(1),
+	TEST_ROTATE_180			= BIT(2),
+	TEST_BAD_PIXEL_FORMAT		= BIT(3),
+	TEST_BAD_ROTATION_90		= BIT(4),
 };
 
 typedef struct {
diff --git a/tests/kms_cursor_crc.c b/tests/kms_cursor_crc.c
index 4851e18f..009883da 100644
--- a/tests/kms_cursor_crc.c
+++ b/tests/kms_cursor_crc.c
@@ -63,8 +63,8 @@ typedef struct {
 	unsigned flags;
 } data_t;
 
-#define TEST_DPMS (1<<0)
-#define TEST_SUSPEND (1<<1)
+#define TEST_DPMS BIT(0)
+#define TEST_SUSPEND BIT(1)
 
 static void draw_cursor(cairo_t *cr, int x, int y, int cw, int ch)
 {
diff --git a/tests/kms_cursor_legacy.c b/tests/kms_cursor_legacy.c
index c7083a07..310a57f1 100644
--- a/tests/kms_cursor_legacy.c
+++ b/tests/kms_cursor_legacy.c
@@ -533,11 +533,11 @@ static uint32_t *make_busy(int fd, uint32_t target)
 	i = 0;
 	batch[i] = MI_BATCH_BUFFER_START;
 	if (gen >= 8) {
-		batch[i] |= 1 << 8 | 1;
+		batch[i] |= BIT(8) | 1;
 		batch[++i] = 0;
 		batch[++i] = 0;
 	} else if (gen >= 6) {
-		batch[i] |= 1 << 8;
+		batch[i] |= BIT(8);
 		batch[++i] = 0;
 	} else {
 		batch[i] |= 2 << 6;
diff --git a/tests/kms_fbc_crc.c b/tests/kms_fbc_crc.c
index 96af06a0..221d4658 100644
--- a/tests/kms_fbc_crc.c
+++ b/tests/kms_fbc_crc.c
@@ -108,7 +108,7 @@ static void fill_blt(data_t *data,
 	COLOR_BLIT_COPY_BATCH_START(flags);
 	OUT_BATCH(3 << 24 | 0xf0 << 16 | pitch);
 	OUT_BATCH(0);
-	OUT_BATCH(1 << 16 | 1);
+	OUT_BATCH(BIT(16) | 1);
 	OUT_RELOC_FENCED(dst, I915_GEM_DOMAIN_RENDER, I915_GEM_DOMAIN_RENDER, 0);
 	OUT_BATCH(pixel);
 	ADVANCE_BATCH();
diff --git a/tests/kms_flip.c b/tests/kms_flip.c
index 41406dfe..5439323c 100644
--- a/tests/kms_flip.c
+++ b/tests/kms_flip.c
@@ -46,40 +46,40 @@
 
 #include "igt_stats.h"
 
-#define TEST_DPMS		(1 << 0)
-#define TEST_WITH_DUMMY_BCS	(1 << 1)
-#define TEST_WITH_DUMMY_RCS	(1 << 2)
-#define TEST_PAN		(1 << 3)
-#define TEST_MODESET		(1 << 4)
-#define TEST_CHECK_TS		(1 << 5)
-#define TEST_EBUSY		(1 << 6)
-#define TEST_EINVAL		(1 << 7)
-#define TEST_FLIP		(1 << 8)
-#define TEST_VBLANK		(1 << 9)
-#define TEST_VBLANK_BLOCK	(1 << 10)
-#define TEST_VBLANK_ABSOLUTE	(1 << 11)
-#define TEST_VBLANK_EXPIRED_SEQ	(1 << 12)
-#define TEST_FB_RECREATE	(1 << 13)
-#define TEST_RMFB		(1 << 14)
-#define TEST_HANG		(1 << 15)
-#define TEST_NOEVENT		(1 << 16)
-#define TEST_FB_BAD_TILING	(1 << 17)
-#define TEST_SINGLE_BUFFER	(1 << 18)
-#define TEST_DPMS_OFF		(1 << 19)
-#define TEST_NO_2X_OUTPUT	(1 << 20)
-#define TEST_DPMS_OFF_OTHERS	(1 << 21)
-#define TEST_ENOENT		(1 << 22)
-#define TEST_FENCE_STRESS	(1 << 23)
-#define TEST_VBLANK_RACE	(1 << 24)
-#define TEST_RPM		(1 << 25)
-#define TEST_SUSPEND		(1 << 26)
-#define TEST_TS_CONT		(1 << 27)
-#define TEST_BO_TOOBIG		(1 << 28)
-#define TEST_HANG_ONCE		(1 << 29)
-#define TEST_BASIC		(1 << 30)
-
-#define EVENT_FLIP		(1 << 0)
-#define EVENT_VBLANK		(1 << 1)
+#define TEST_DPMS		BIT(0)
+#define TEST_WITH_DUMMY_BCS	BIT(1)
+#define TEST_WITH_DUMMY_RCS	BIT(2)
+#define TEST_PAN		BIT(3)
+#define TEST_MODESET		BIT(4)
+#define TEST_CHECK_TS		BIT(5)
+#define TEST_EBUSY		BIT(6)
+#define TEST_EINVAL		BIT(7)
+#define TEST_FLIP		BIT(8)
+#define TEST_VBLANK		BIT(9)
+#define TEST_VBLANK_BLOCK	BIT(10)
+#define TEST_VBLANK_ABSOLUTE	BIT(11)
+#define TEST_VBLANK_EXPIRED_SEQ	BIT(12)
+#define TEST_FB_RECREATE	BIT(13)
+#define TEST_RMFB		BIT(14)
+#define TEST_HANG		BIT(15)
+#define TEST_NOEVENT		BIT(16)
+#define TEST_FB_BAD_TILING	BIT(17)
+#define TEST_SINGLE_BUFFER	BIT(18)
+#define TEST_DPMS_OFF		BIT(19)
+#define TEST_NO_2X_OUTPUT	BIT(20)
+#define TEST_DPMS_OFF_OTHERS	BIT(21)
+#define TEST_ENOENT		BIT(22)
+#define TEST_FENCE_STRESS	BIT(23)
+#define TEST_VBLANK_RACE	BIT(24)
+#define TEST_RPM		BIT(25)
+#define TEST_SUSPEND		BIT(26)
+#define TEST_TS_CONT		BIT(27)
+#define TEST_BO_TOOBIG		BIT(28)
+#define TEST_HANG_ONCE		BIT(29)
+#define TEST_BASIC		BIT(30)
+
+#define EVENT_FLIP		BIT(0)
+#define EVENT_VBLANK		BIT(1)
 
 #ifndef DRM_CAP_TIMESTAMP_MONOTONIC
 #define DRM_CAP_TIMESTAMP_MONOTONIC 6
@@ -900,7 +900,7 @@ static void connector_find_preferred_mode(uint32_t connector_id, int crtc_idx,
 {
 	struct kmstest_connector_config config;
 
-	if (!kmstest_get_connector_config(drm_fd, connector_id, 1 << crtc_idx,
+	if (!kmstest_get_connector_config(drm_fd, connector_id, BIT(crtc_idx),
 					  &config)) {
 		o->mode_valid = 0;
 		return;
@@ -945,11 +945,11 @@ static void connector_find_compatible_mode(int crtc_idx0, int crtc_idx1,
 	int n, m;
 
 	if (!kmstest_get_connector_config(drm_fd, o->_connector[0],
-					  1 << crtc_idx0, &config[0]))
+					  BIT(crtc_idx0), &config[0]))
 		return;
 
 	if (!kmstest_get_connector_config(drm_fd, o->_connector[1],
-					  1 << crtc_idx1, &config[1])) {
+					  BIT(crtc_idx1), &config[1])) {
 		kmstest_free_connector_config(&config[0]);
 		return;
 	}
diff --git a/tests/kms_frontbuffer_tracking.c b/tests/kms_frontbuffer_tracking.c
index 4f4848bc..e9a8c7f6 100644
--- a/tests/kms_frontbuffer_tracking.c
+++ b/tests/kms_frontbuffer_tracking.c
@@ -389,7 +389,7 @@ static void init_mode_params(struct modeset_params *params, uint32_t crtc_id,
 	int i;
 
 	for (i = 0; i < drm.plane_res->count_planes; i++)
-		if ((drm.planes[i]->possible_crtcs & (1 << crtc_idx)) &&
+		if ((drm.planes[i]->possible_crtcs & BIT(crtc_idx)) &&
 		    drm.plane_types[i] == DRM_PLANE_TYPE_OVERLAY) {
 			overlay_plane_id = drm.planes[i]->plane_id;
 			break;
@@ -524,7 +524,7 @@ static bool init_modeset_cached_params(void)
 	igt_require(drm.res->count_crtcs >= 2);
 	scnd_crtc_id = kmstest_find_crtc_for_connector(drm.fd, drm.res,
 						      scnd_connector,
-			1 << kmstest_get_crtc_idx(drm.res, prim_crtc_id));
+			BIT(kmstest_get_crtc_idx(drm.res, prim_crtc_id)));
 	init_mode_params(&scnd_mode_params, scnd_crtc_id,
 			 scnd_connector, scnd_mode);
 	print_mode_info("Secondary", &scnd_mode_params);
@@ -1628,18 +1628,18 @@ static void do_flush(const struct test_mode *t)
 	gem_set_domain(drm.fd, target->fb->gem_handle, I915_GEM_DOMAIN_GTT, 0);
 }
 
-#define DONT_ASSERT_CRC			(1 << 0)
-#define DONT_ASSERT_FEATURE_STATUS	(1 << 1)
+#define DONT_ASSERT_CRC			BIT(0)
+#define DONT_ASSERT_FEATURE_STATUS	BIT(1)
 
 #define FBC_ASSERT_FLAGS		(0xF << 2)
-#define ASSERT_FBC_ENABLED		(1 << 2)
-#define ASSERT_FBC_DISABLED		(1 << 3)
-#define ASSERT_LAST_ACTION_CHANGED	(1 << 4)
-#define ASSERT_NO_ACTION_CHANGE		(1 << 5)
+#define ASSERT_FBC_ENABLED		BIT(2)
+#define ASSERT_FBC_DISABLED		BIT(3)
+#define ASSERT_LAST_ACTION_CHANGED	BIT(4)
+#define ASSERT_NO_ACTION_CHANGE		BIT(5)
 
 #define PSR_ASSERT_FLAGS		(3 << 6)
-#define ASSERT_PSR_ENABLED		(1 << 6)
-#define ASSERT_PSR_DISABLED		(1 << 7)
+#define ASSERT_PSR_ENABLED		BIT(6)
+#define ASSERT_PSR_DISABLED		BIT(7)
 
 static int adjust_assertion_flags(const struct test_mode *t, int flags)
 {
@@ -2296,7 +2296,7 @@ static void set_prim_plane_for_params(struct modeset_params *params)
 	uint32_t plane_id = 0;
 
 	for (i = 0; i < drm.plane_res->count_planes; i++)
-		if ((drm.planes[i]->possible_crtcs & (1 << crtc_idx)) &&
+		if ((drm.planes[i]->possible_crtcs & BIT(crtc_idx)) &&
 		    drm.plane_types[i] == DRM_PLANE_TYPE_PRIMARY)
 			plane_id = drm.planes[i]->plane_id;
 	igt_assert(plane_id);
@@ -2591,7 +2591,7 @@ static bool prim_plane_disabled(void)
 	int crtc_idx = kmstest_get_crtc_idx(drm.res, prim_mode_params.crtc_id);
 
 	for (i = 0; i < drm.plane_res->count_planes; i++) {
-		if ((drm.planes[i]->possible_crtcs & (1 << crtc_idx)) &&
+		if ((drm.planes[i]->possible_crtcs & BIT(crtc_idx)) &&
 		    drm.plane_types[i] == DRM_PLANE_TYPE_PRIMARY) {
 			found = true;
 			disabled = (drm.planes[i]->crtc_id == 0);
@@ -2730,7 +2730,7 @@ static void scaledprimary_subtest(const struct test_mode *t)
 			 pick_color(&new_fb, COLOR_MAGENTA));
 
 	for (i = 0; i < drm.plane_res->count_planes; i++)
-		if ((drm.planes[i]->possible_crtcs & (1 << prim_crtc_idx)) &&
+		if ((drm.planes[i]->possible_crtcs & BIT(prim_crtc_idx)) &&
 		    drm.plane_types[i] == DRM_PLANE_TYPE_PRIMARY)
 			plane_id = drm.planes[i]->plane_id;
 
diff --git a/tests/kms_pipe_color.c b/tests/kms_pipe_color.c
index c7a5d2f1..e9c47317 100644
--- a/tests/kms_pipe_color.c
+++ b/tests/kms_pipe_color.c
@@ -148,8 +148,8 @@ static struct _drm_color_lut *coeffs_to_lut(data_t *data,
 {
 	struct _drm_color_lut *lut;
 	uint32_t i;
-	uint32_t max_value = (1 << 16) - 1;
-	uint32_t mask = ((1 << color_depth) - 1) << 8;
+	uint32_t max_value = BIT(16) - 1;
+	uint32_t mask = (BIT(color_depth) - 1) << 8;
 
 	lut = malloc(sizeof(struct _drm_color_lut) * lut_size);
 
@@ -218,7 +218,7 @@ static void set_ctm(igt_pipe_t *pipe, const double *coefficients)
 		if (coefficients[i] < 0) {
 			ctm.matrix[i] =
 				(int64_t) (-coefficients[i] * ((int64_t) 1L << 32));
-			ctm.matrix[i] |= 1ULL << 63;
+			ctm.matrix[i] |= BIT_ULL(63);
 		} else
 			ctm.matrix[i] =
 				(int64_t) (coefficients[i] * ((int64_t) 1L << 32));
diff --git a/tests/kms_pipe_crc_basic.c b/tests/kms_pipe_crc_basic.c
index dd2ad3c3..23fdf2f1 100644
--- a/tests/kms_pipe_crc_basic.c
+++ b/tests/kms_pipe_crc_basic.c
@@ -86,8 +86,8 @@ static void test_bad_source(data_t *data)
 
 #define N_CRCS	3
 
-#define TEST_SEQUENCE (1<<0)
-#define TEST_NONBLOCK (1<<1)
+#define TEST_SEQUENCE BIT(0)
+#define TEST_NONBLOCK BIT(1)
 
 static void
 test_read_crc_for_output(data_t *data, int pipe, igt_output_t *output,
diff --git a/tests/kms_plane.c b/tests/kms_plane.c
index d0de0f52..e7ea5ce5 100644
--- a/tests/kms_plane.c
+++ b/tests/kms_plane.c
@@ -143,8 +143,8 @@ create_fb_for_mode__position(data_t *data, drmModeModeInfo *mode,
 }
 
 enum {
-	TEST_POSITION_FULLY_COVERED = 1 << 0,
-	TEST_DPMS = 1 << 1,
+	TEST_POSITION_FULLY_COVERED = BIT(0),
+	TEST_DPMS = BIT(1),
 };
 
 static void
@@ -286,9 +286,9 @@ create_fb_for_mode__panning(data_t *data, drmModeModeInfo *mode,
 }
 
 enum {
-	TEST_PANNING_TOP_LEFT	  = 1 << 0,
-	TEST_PANNING_BOTTOM_RIGHT = 1 << 1,
-	TEST_SUSPEND_RESUME	  = 1 << 2,
+	TEST_PANNING_TOP_LEFT	  = BIT(0),
+	TEST_PANNING_BOTTOM_RIGHT = BIT(1),
+	TEST_SUSPEND_RESUME	  = BIT(2),
 };
 
 static void
diff --git a/tests/kms_psr_sink_crc.c b/tests/kms_psr_sink_crc.c
index 8f6bdc0d..55c0d23e 100644
--- a/tests/kms_psr_sink_crc.c
+++ b/tests/kms_psr_sink_crc.c
@@ -136,7 +136,7 @@ static void fill_blt(data_t *data, uint32_t handle, unsigned char color)
 	igt_assert(batch);
 
 	COLOR_BLIT_COPY_BATCH_START(0);
-	OUT_BATCH((1 << 24) | (0xf0 << 16) | 0);
+	OUT_BATCH(BIT(24) | (0xf0 << 16) | 0);
 	OUT_BATCH(0);
 	OUT_BATCH(0xfff << 16 | 0xfff);
 	OUT_RELOC(dst, I915_GEM_DOMAIN_RENDER, I915_GEM_DOMAIN_RENDER, 0);
diff --git a/tests/kms_render.c b/tests/kms_render.c
index 72da87f1..55bebeef 100644
--- a/tests/kms_render.c
+++ b/tests/kms_render.c
@@ -211,7 +211,7 @@ static int run_test(const char *test_name, enum test_flags flags)
 			struct kmstest_connector_config cconf;
 
 			if (!kmstest_get_connector_config(drm_fd, connector_id,
-							   1 << j, &cconf))
+							   BIT(j), &cconf))
 				continue;
 
 			test_connector(test_name, &cconf, flags);
diff --git a/tests/kms_setmode.c b/tests/kms_setmode.c
index 430568a1..1c92e2cf 100644
--- a/tests/kms_setmode.c
+++ b/tests/kms_setmode.c
@@ -321,10 +321,9 @@ static void setup_crtcs(drmModeRes *resources, struct connector_config *cconf,
 			igt_assert(encoder);
 
 			config_valid &= !!(encoder->possible_crtcs &
-					  (1 << crtc->crtc_idx));
+					  BIT(crtc->crtc_idx));
 
-			encoder_mask |= 1 << get_encoder_idx(resources,
-							     encoder);
+			encoder_mask |= BIT(get_encoder_idx(resources, encoder));
 			config_valid &= !(encoder_mask &
 					  ~encoder->possible_clones);
 
@@ -622,14 +621,14 @@ static int assign_crtc_to_connectors(const struct test_config *tconf,
 		int crtc_idx = crtc_idxs[i];
 
 		if ((tconf->flags & TEST_SINGLE_CRTC_CLONE) &&
-		    crtc_idx_mask & ~(1 << crtc_idx))
+		    crtc_idx_mask & ~BIT(crtc_idx))
 			return -1;
 
 		if ((tconf->flags & TEST_EXCLUSIVE_CRTC_CLONE) &&
-		    crtc_idx_mask & (1 << crtc_idx))
+		    crtc_idx_mask & BIT(crtc_idx))
 			return -1;
 
-		crtc_idx_mask |= 1 << crtc_idx;
+		crtc_idx_mask |= BIT(crtc_idx);
 
 		cconfs[i].crtc_idx = crtc_idx;
 	}
diff --git a/tests/perf.c b/tests/perf.c
index c9c5c57e..23944973 100644
--- a/tests/perf.c
+++ b/tests/perf.c
@@ -45,31 +45,31 @@ IGT_TEST_DESCRIPTION("Test the i915 perf metrics streaming interface");
 #define GEN6_MI_REPORT_PERF_COUNT ((0x28 << 23) | (3 - 2))
 
 #define GFX_OP_PIPE_CONTROL     ((3 << 29) | (3 << 27) | (2 << 24))
-#define PIPE_CONTROL_CS_STALL	   (1 << 20)
-#define PIPE_CONTROL_GLOBAL_SNAPSHOT_COUNT_RESET	(1 << 19)
-#define PIPE_CONTROL_TLB_INVALIDATE     (1 << 18)
-#define PIPE_CONTROL_SYNC_GFDT	  (1 << 17)
-#define PIPE_CONTROL_MEDIA_STATE_CLEAR  (1 << 16)
+#define PIPE_CONTROL_CS_STALL	   BIT(20)
+#define PIPE_CONTROL_GLOBAL_SNAPSHOT_COUNT_RESET	BIT(19)
+#define PIPE_CONTROL_TLB_INVALIDATE     BIT(18)
+#define PIPE_CONTROL_SYNC_GFDT	  BIT(17)
+#define PIPE_CONTROL_MEDIA_STATE_CLEAR  BIT(16)
 #define PIPE_CONTROL_NO_WRITE	   (0 << 14)
-#define PIPE_CONTROL_WRITE_IMMEDIATE    (1 << 14)
+#define PIPE_CONTROL_WRITE_IMMEDIATE    BIT(14)
 #define PIPE_CONTROL_WRITE_DEPTH_COUNT  (2 << 14)
 #define PIPE_CONTROL_WRITE_TIMESTAMP    (3 << 14)
-#define PIPE_CONTROL_DEPTH_STALL	(1 << 13)
-#define PIPE_CONTROL_RENDER_TARGET_FLUSH (1 << 12)
-#define PIPE_CONTROL_INSTRUCTION_INVALIDATE (1 << 11)
-#define PIPE_CONTROL_TEXTURE_CACHE_INVALIDATE   (1 << 10) /* GM45+ only */
-#define PIPE_CONTROL_ISP_DIS	    (1 << 9)
-#define PIPE_CONTROL_INTERRUPT_ENABLE   (1 << 8)
-#define PIPE_CONTROL_FLUSH_ENABLE       (1 << 7) /* Gen7+ only */
+#define PIPE_CONTROL_DEPTH_STALL	BIT(13)
+#define PIPE_CONTROL_RENDER_TARGET_FLUSH BIT(12)
+#define PIPE_CONTROL_INSTRUCTION_INVALIDATE BIT(11)
+#define PIPE_CONTROL_TEXTURE_CACHE_INVALIDATE   BIT(10) /* GM45+ only */
+#define PIPE_CONTROL_ISP_DIS	    BIT(9)
+#define PIPE_CONTROL_INTERRUPT_ENABLE   BIT(8)
+#define PIPE_CONTROL_FLUSH_ENABLE       BIT(7) /* Gen7+ only */
 /* GT */
-#define PIPE_CONTROL_DATA_CACHE_INVALIDATE      (1 << 5)
-#define PIPE_CONTROL_VF_CACHE_INVALIDATE	(1 << 4)
-#define PIPE_CONTROL_CONST_CACHE_INVALIDATE     (1 << 3)
-#define PIPE_CONTROL_STATE_CACHE_INVALIDATE     (1 << 2)
-#define PIPE_CONTROL_STALL_AT_SCOREBOARD	(1 << 1)
-#define PIPE_CONTROL_DEPTH_CACHE_FLUSH	  (1 << 0)
+#define PIPE_CONTROL_DATA_CACHE_INVALIDATE      BIT(5)
+#define PIPE_CONTROL_VF_CACHE_INVALIDATE	BIT(4)
+#define PIPE_CONTROL_CONST_CACHE_INVALIDATE     BIT(3)
+#define PIPE_CONTROL_STATE_CACHE_INVALIDATE     BIT(2)
+#define PIPE_CONTROL_STALL_AT_SCOREBOARD	BIT(1)
+#define PIPE_CONTROL_DEPTH_CACHE_FLUSH	  BIT(0)
 #define PIPE_CONTROL_PPGTT_WRITE	(0 << 2)
-#define PIPE_CONTROL_GLOBAL_GTT_WRITE   (1 << 2)
+#define PIPE_CONTROL_GLOBAL_GTT_WRITE   BIT(2)
 
 /* Temporarily copy i915-perf uapi here to avoid a dependency on libdrm's
  * i915_drm.h copy being updated with the i915-perf interface before this
@@ -105,9 +105,9 @@ enum drm_i915_perf_property_id {
 
 struct drm_i915_perf_open_param {
        __u32 flags;
-#define I915_PERF_FLAG_FD_CLOEXEC	(1<<0)
-#define I915_PERF_FLAG_FD_NONBLOCK	(1<<1)
-#define I915_PERF_FLAG_DISABLED		(1<<2)
+#define I915_PERF_FLAG_FD_CLOEXEC	BIT(0)
+#define I915_PERF_FLAG_FD_NONBLOCK	BIT(1)
+#define I915_PERF_FLAG_DISABLED		BIT(2)
 
        __u32 num_properties;
        __u64 properties_ptr;
@@ -599,7 +599,7 @@ read_2_oa_reports(int stream_fd,
 	size_t sample_size = (sizeof(struct drm_i915_perf_record_header) +
 			      format_size);
 	const struct drm_i915_perf_record_header *header;
-	uint32_t exponent_mask = (1 << (exponent + 1)) - 1;
+	uint32_t exponent_mask = BIT((exponent + 1)) - 1;
 
 	/* Note: we allocate a large buffer so that each read() iteration
 	 * should scrape *all* pending records.
@@ -688,14 +688,14 @@ read_2_oa_reports(int stream_fd,
 				 * checking if the least significant bits are
 				 * zero and the exponent bit is set.
 				 */
-				if ((report[1] & exponent_mask) != (1 << exponent)) {
+				if ((report[1] & exponent_mask) != BIT(exponent)) {
 					igt_debug("skipping non timer report reason=%x\n",
 						  report[0]);
 
 					/* Also assert our hypothesis about the
 					 * reason bit...
 					 */
-					igt_assert_eq(report[0] & (1 << 10), 0);
+					igt_assert_eq(report[0] & BIT(10), 0);
 					continue;
 				}
 			}
diff --git a/tests/pm_rpm.c b/tests/pm_rpm.c
index d0600d59..c6bbc784 100644
--- a/tests/pm_rpm.c
+++ b/tests/pm_rpm.c
@@ -1680,7 +1680,7 @@ static void planes_subtest(bool universal, bool dpms)
 		plane = drmModeGetPlane(drm_fd, planes->planes[i]);
 		igt_assert(plane);
 
-		if (plane->possible_crtcs & (1 << crtc_idx)) {
+		if (plane->possible_crtcs & BIT(crtc_idx)) {
 			enum plane_type type;
 
 			type = universal ? get_plane_type(plane->plane_id) :
@@ -1722,7 +1722,7 @@ static void pm_test_tiling(void)
 	uint32_t stride = 512;
 
 	/* calculate how many objects we can map */
-	for (i = 1 << off_bit; i <= gtt_obj_max_size; i <<= 1, max_gem_objs++)
+	for (i = BIT(off_bit); i <= gtt_obj_max_size; i <<= 1, max_gem_objs++)
 		;
 
 	gem_bufs = calloc(max_gem_objs, sizeof(*gem_bufs));
@@ -1731,7 +1731,7 @@ static void pm_test_tiling(void)
 	/* try to set different tiling for each handle */
 	for (i = 0; i < ARRAY_SIZE(tiling_modes); i++) {
 
-		for (j = 0, k = 1 << off_bit;
+		for (j = 0, k = BIT(off_bit);
 		     k <= gtt_obj_max_size; k <<= 1, j++) {
 			handles[j] = gem_create(drm_fd, k);
 			gem_bufs[j] = gem_mmap__gtt(drm_fd, handles[j],
@@ -1750,7 +1750,7 @@ static void pm_test_tiling(void)
 
 		enable_one_screen_and_wait(&ms_data);
 
-		for (j = 0, k = 1 << off_bit;
+		for (j = 0, k = BIT(off_bit);
 		     k <= gtt_obj_max_size; k <<= 1, j++) {
 			igt_assert(munmap(gem_bufs[j], k) == 0);
 			gem_close(drm_fd, handles[j]);
diff --git a/tests/pm_rps.c b/tests/pm_rps.c
index 5a54d9d4..484f3419 100644
--- a/tests/pm_rps.c
+++ b/tests/pm_rps.c
@@ -594,11 +594,11 @@ static uint32_t *make_busy(int fd, uint32_t handle)
 	i = 0;
 	batch[i] = MI_BATCH_BUFFER_START;
 	if (gen >= 8) {
-		batch[i] |= 1 << 8 | 1;
+		batch[i] |= BIT(8) | 1;
 		batch[++i] = 0;
 		batch[++i] = 0;
 	} else if (gen >= 6) {
-		batch[i] |= 1 << 8;
+		batch[i] |= BIT(8);
 		batch[++i] = 0;
 	} else {
 		batch[i] |= 2 << 6;
diff --git a/tests/pm_sseu.c b/tests/pm_sseu.c
index 3e60a5f8..d34e074d 100644
--- a/tests/pm_sseu.c
+++ b/tests/pm_sseu.c
@@ -223,7 +223,7 @@ gem_get_target_spins(double dt)
 	/* Double increments until we bound the target time */
 	prev_dt = 0.0;
 	for (i = 0; i < 32; i++) {
-		spins = 1 << i;
+		spins = BIT(i);
 		clock_gettime(CLOCK_MONOTONIC, &tstart);
 
 		gem.spinfunc(gem.batch, &gem.buf, spins);
@@ -242,7 +242,7 @@ gem_get_target_spins(double dt)
 	igt_assert_neq(i, 32);
 
 	/* Linearly interpolate between i and i-1 to get target increments */
-	spins = 1 << (i-1); /* lower bound spins */
+	spins = BIT((i - 1)); /* lower bound spins */
 	spins += spins * (dt - prev_dt)/(cur_dt - prev_dt); /* target spins */
 
 	return spins;
diff --git a/tests/prime_busy.c b/tests/prime_busy.c
index c7ff8720..7ed9211e 100644
--- a/tests/prime_busy.c
+++ b/tests/prime_busy.c
@@ -85,7 +85,7 @@ static void busy(int fd, unsigned ring, unsigned flags)
 		store[count].delta = sizeof(uint32_t) * count;
 		store[count].read_domains = I915_GEM_DOMAIN_INSTRUCTION;
 		store[count].write_domain = I915_GEM_DOMAIN_INSTRUCTION;
-		batch[i] = MI_STORE_DWORD_IMM | (gen < 6 ? 1 << 22 : 0);
+		batch[i] = MI_STORE_DWORD_IMM | (gen < 6 ? BIT(22) : 0);
 		if (gen >= 8) {
 			batch[++i] = 0;
 			batch[++i] = 0;
@@ -110,11 +110,11 @@ static void busy(int fd, unsigned ring, unsigned flags)
 	store[count].write_domain = 0;
 	batch[i] = MI_BATCH_BUFFER_START;
 	if (gen >= 8) {
-		batch[i] |= 1 << 8 | 1;
+		batch[i] |= BIT(8) | 1;
 		batch[++i] = 0;
 		batch[++i] = 0;
 	} else if (gen >= 6) {
-		batch[i] |= 1 << 8;
+		batch[i] |= BIT(8);
 		batch[++i] = 0;
 	} else {
 		batch[i] |= 2 << 6;
diff --git a/tests/prime_nv_pcopy.c b/tests/prime_nv_pcopy.c
index b5ceabf1..9f8823e7 100644
--- a/tests/prime_nv_pcopy.c
+++ b/tests/prime_nv_pcopy.c
@@ -81,8 +81,8 @@ static void nv_bo_alloc(struct nouveau_bo **bo, rect *r,
 			memtype = memtype_intel;
 		tile_y += ((tile_mode & 0xf0)>>4);
 
-		dx = 1 << tile_x;
-		dy = 1 << tile_y;
+		dx = BIT(tile_x);
+		dy = BIT(tile_y);
 		igt_debug("Tiling requirements: x y %u %u\n", dx, dy);
 	}
 
diff --git a/tests/prime_vgem.c b/tests/prime_vgem.c
index 5f8a4dd5..c21b8703 100644
--- a/tests/prime_vgem.c
+++ b/tests/prime_vgem.c
@@ -297,7 +297,7 @@ static void work(int i915, int dmabuf, unsigned ring, uint32_t flags)
 		store[count].delta = sizeof(uint32_t) * count;
 		store[count].read_domains = I915_GEM_DOMAIN_INSTRUCTION;
 		store[count].write_domain = I915_GEM_DOMAIN_INSTRUCTION;
-		batch[i] = MI_STORE_DWORD_IMM | (gen < 6 ? 1 << 22 : 0);
+		batch[i] = MI_STORE_DWORD_IMM | (gen < 6 ? BIT(22) : 0);
 		if (gen >= 8) {
 			batch[++i] = 0;
 			batch[++i] = 0;
@@ -322,11 +322,11 @@ static void work(int i915, int dmabuf, unsigned ring, uint32_t flags)
 	store[count].write_domain = 0;
 	batch[i] = MI_BATCH_BUFFER_START;
 	if (gen >= 8) {
-		batch[i] |= 1 << 8 | 1;
+		batch[i] |= BIT(8) | 1;
 		batch[++i] = 0;
 		batch[++i] = 0;
 	} else if (gen >= 6) {
-		batch[i] |= 1 << 8;
+		batch[i] |= BIT(8);
 		batch[++i] = 0;
 	} else {
 		batch[i] |= 2 << 6;
@@ -564,7 +564,7 @@ static uint32_t set_fb_on_crtc(int fd, int pipe, struct vgem_bo *bo, uint32_t fb
 			memset(&enc, 0, sizeof(enc));
 			enc.encoder_id = encoders[e];
 			drmIoctl(fd, DRM_IOCTL_MODE_GETENCODER, &enc);
-			if (enc.possible_crtcs & (1 << pipe))
+			if (enc.possible_crtcs & BIT(pipe))
 				break;
 		}
 		if (e == conn.count_encoders)
diff --git a/tests/sw_sync.c b/tests/sw_sync.c
index 946592e8..ea1e119c 100644
--- a/tests/sw_sync.c
+++ b/tests/sw_sync.c
@@ -491,7 +491,7 @@ static void test_sync_multi_timeline_wait(void)
 }
 
 #define MULTI_CONSUMER_THREADS 8
-#define MULTI_CONSUMER_ITERATIONS (1 << 14)
+#define MULTI_CONSUMER_ITERATIONS BIT(14)
 static void * test_sync_multi_consumer_thread(void *arg)
 {
 	data_t *data = arg;
@@ -572,7 +572,7 @@ static void test_sync_multi_consumer(void)
 }
 
 #define MULTI_CONSUMER_PRODUCER_THREADS 8
-#define MULTI_CONSUMER_PRODUCER_ITERATIONS (1 << 14)
+#define MULTI_CONSUMER_PRODUCER_ITERATIONS BIT(14)
 static void * test_sync_multi_consumer_producer_thread(void *arg)
 {
 	data_t *data = arg;
@@ -749,7 +749,7 @@ static int mpsc_consumer_thread(void)
  */
 static void test_sync_multi_producer_single_consumer(void)
 {
-	int iterations = 1 << 12;
+	int iterations = BIT(12);
 	int n = 5;
 	int prod_timeline[n];
 	int cons_timeline;
@@ -781,7 +781,7 @@ static void test_sync_multi_producer_single_consumer(void)
 
 static void test_sync_expired_merge(void)
 {
-	int iterations = 1 << 20;
+	int iterations = BIT(20);
 	int timeline;
 	int i;
 	int fence_expired, fence_merged;
diff --git a/tests/testdisplay.c b/tests/testdisplay.c
index a974f42b..12292334 100644
--- a/tests/testdisplay.c
+++ b/tests/testdisplay.c
@@ -495,7 +495,7 @@ int update_display(bool probe)
 
 			if (test_preferred_mode || force_mode ||
 			    specified_mode_num != -1)
-				crtc_idx_mask &= ~(1 << connector->pipe);
+				crtc_idx_mask &= ~BIT(connector->pipe);
 
 		}
 	}
diff --git a/tests/vgem_basic.c b/tests/vgem_basic.c
index 5b54a4a3..ec1d827d 100644
--- a/tests/vgem_basic.c
+++ b/tests/vgem_basic.c
@@ -59,14 +59,14 @@ static void test_create(int fd)
 	bo.height = 1024;
 	bo.bpp = 8;
 	vgem_create(fd, &bo);
-	igt_assert_eq(bo.size, 1<<20);
+	igt_assert_eq(bo.size, BIT(20));
 	gem_close(fd, bo.handle);
 
-	bo.width = 1<<15;
-	bo.height = 1<<15;
+	bo.width = BIT(15);
+	bo.height = BIT(15);
 	bo.bpp = 16;
 	vgem_create(fd, &bo);
-	igt_assert_eq(bo.size, 1<<31);
+	igt_assert_eq(bo.size, BIT(31));
 	gem_close(fd, bo.handle);
 }
 
-- 
2.11.0



More information about the Intel-gfx mailing list