diff options
author | Matt Roper <matthew.d.roper@intel.com> | 2015-03-05 15:01:00 -0800 |
---|---|---|
committer | Daniel Vetter <daniel.vetter@ffwll.ch> | 2015-03-06 18:06:10 +0100 |
commit | 07be8fec156ba8405f2aafa3a37fe29d06893f05 (patch) | |
tree | 0591bf1aca05c2c8f5f89bb0400d948e8a2d0fe6 /tests/kms_flip.c | |
parent | 14a3d88a0967718124fc3b6c111a877203da025f (diff) |
igt.cocci: Replace igt_assert() with igt_assert_CMP() where possible
The integer comparison macros give us better error output by including
the actual values that failed the comparison.
Signed-off-by: Matt Roper <matthew.d.roper@intel.com>
Signed-off-by: Daniel Vetter <daniel.vetter@intel.com>
Diffstat (limited to 'tests/kms_flip.c')
-rw-r--r-- | tests/kms_flip.c | 8 |
1 files changed, 4 insertions, 4 deletions
diff --git a/tests/kms_flip.c b/tests/kms_flip.c index cae02e9b..30139caa 100644 --- a/tests/kms_flip.c +++ b/tests/kms_flip.c @@ -771,7 +771,7 @@ static void eat_error_state(void) int fd; fd = igt_debugfs_open(dfs_entry_error, O_WRONLY); - igt_assert(fd >= 0); + igt_assert_lte(0, fd); igt_assert(write(fd, data, sizeof(data)) == sizeof(data)); close(fd); @@ -1348,7 +1348,7 @@ static void run_test_on_crtc_set(struct test_output *o, int *crtc_idxs, if (!o->mode_valid) return; - igt_assert(o->count == crtc_count); + igt_assert_eq(o->count, crtc_count); last_connector = o->kconnector[0]; @@ -1407,10 +1407,10 @@ static void run_test_on_crtc_set(struct test_output *o, int *crtc_idxs, sleep(1); if (o->flags & TEST_BO_TOOBIG) { - igt_assert(do_page_flip(o, o->fb_ids[1], true) == -E2BIG); + igt_assert_eq(do_page_flip(o, o->fb_ids[1], true), -E2BIG); goto out; } else - igt_assert(do_page_flip(o, o->fb_ids[1], true) == 0); + igt_assert_eq(do_page_flip(o, o->fb_ids[1], true), 0); wait_for_events(o); o->current_fb_id = 1; |