diff options
| author | Daniel Vetter <daniel.vetter@ffwll.ch> | 2014-05-14 09:56:53 +0200 |
|---|---|---|
| committer | Daniel Vetter <daniel.vetter@ffwll.ch> | 2014-05-14 09:56:53 +0200 |
| commit | 0b7ce4ac2929ea56dfde508d551163a5304c66d8 (patch) | |
| tree | 7af13f919437cc28ab61dbae3fee42a159fcfb6f /tests/gem_render_tiled_blits.c | |
| parent | e624fa8a2e1a31494c6e47a65e378e480882422d (diff) | |
tests: Use igt macros more
Often just folding together of the common if (cond) printf;
abort|igt_skip|igt_fail; pattern. But in a few cases I've ripped out
more since the igt macros will already print the condition and errno.
A few tests where more work (like ripping out return codes en masse)
is needed left as-is.
Signed-off-by: Daniel Vetter <daniel.vetter@ffwll.ch>
Diffstat (limited to 'tests/gem_render_tiled_blits.c')
| -rw-r--r-- | tests/gem_render_tiled_blits.c | 15 |
1 files changed, 5 insertions, 10 deletions
diff --git a/tests/gem_render_tiled_blits.c b/tests/gem_render_tiled_blits.c index a9a88866..8b74e2b4 100644 --- a/tests/gem_render_tiled_blits.c +++ b/tests/gem_render_tiled_blits.c @@ -85,12 +85,10 @@ check_bo(struct intel_batchbuffer *batch, struct igt_buf *buf, uint32_t val) ptr = data; } for (i = 0; i < WIDTH*HEIGHT; i++) { - if (ptr[i] != val) { - fprintf(stderr, "Expected 0x%08x, found 0x%08x " - "at offset 0x%08x\n", - val, ptr[i], i * 4); - abort(); - } + igt_assert_f(ptr[i] == val, + "Expected 0x%08x, found 0x%08x " + "at offset 0x%08x\n", + val, ptr[i], i * 4); val++; } if (ptr != data) @@ -115,10 +113,7 @@ int main(int argc, char **argv) devid = intel_get_drm_devid(fd); render_copy = igt_get_render_copyfunc(devid); - if (render_copy == NULL) { - printf("no render-copy function, doing nothing\n"); - return 77; - } + igt_require(render_copy); snoop = 1; if (IS_GEN2(devid)) /* chipset only handles cached -> uncached */ |