From 282482db70188a3cbc653cebbf407715373255d1 Mon Sep 17 00:00:00 2001 From: Daniel Vetter Date: Thu, 10 Jul 2014 20:04:19 +0200 Subject: tests: Run igt.cocci New stuff caught. Plus manually simplify the massive igt_fail_on_f(file == NULL, ...) to a simple igt_assert(file). We already print the errno (if applicapable) and the condition, which is equally informative. Cc: Yi Sun Cc: Matt Roper Cc: Wendy Wang Signed-off-by: Daniel Vetter --- tests/kms_universal_plane.c | 8 +++----- 1 file changed, 3 insertions(+), 5 deletions(-) (limited to 'tests/kms_universal_plane.c') diff --git a/tests/kms_universal_plane.c b/tests/kms_universal_plane.c index c64cdbb9..94d89b87 100644 --- a/tests/kms_universal_plane.c +++ b/tests/kms_universal_plane.c @@ -62,9 +62,8 @@ functional_test_init(functional_test_t *test, igt_output_t *output, enum pipe pi drmModeModeInfo *mode; test->pipe_crc = igt_pipe_crc_new(pipe, INTEL_PIPE_CRC_SOURCE_AUTO); - if (!test->pipe_crc) - igt_skip("auto crc not supported on this connector with pipe %i\n", - pipe); + igt_skip_on_f(!test->pipe_crc, + "auto crc not supported on this connector with pipe %i\n", pipe); igt_output_set_pipe(output, pipe); @@ -136,8 +135,7 @@ functional_test_pipe(data_t *data, enum pipe pipe, igt_output_t *output) igt_assert(data->display.has_universal_planes); igt_skip_on(pipe >= display->n_pipes); - fprintf(stdout, "Testing connector %s using pipe %c\n", - igt_output_name(output), pipe_name(pipe)); + igt_info("Testing connector %s using pipe %c\n", igt_output_name(output), pipe_name(pipe)); functional_test_init(&test, output, pipe); -- cgit v1.2.3