diff options
author | Ville Syrjälä <ville.syrjala@linux.intel.com> | 2014-12-09 14:44:02 +0200 |
---|---|---|
committer | Ville Syrjälä <ville.syrjala@linux.intel.com> | 2014-12-09 20:41:18 +0200 |
commit | b6f68c10b4ee2b730baeb8129aba36b5210ac291 (patch) | |
tree | 625856a15d71713add592fd8844e6ce93b56b490 /tests/kms_mmio_vs_cs_flip.c | |
parent | 1b6eb414f36f3728f5c0393d8f4eefee74abdce9 (diff) |
tests/kms_mmio_vs_cs_flip: Count valid tests for all crtcs
Report SKIP only if none of the pipe/connector combos worked, instead of
trying to report for each pipe separately.
Signed-off-by: Ville Syrjälä <ville.syrjala@linux.intel.com>
Diffstat (limited to 'tests/kms_mmio_vs_cs_flip.c')
-rw-r--r-- | tests/kms_mmio_vs_cs_flip.c | 36 |
1 files changed, 18 insertions, 18 deletions
diff --git a/tests/kms_mmio_vs_cs_flip.c b/tests/kms_mmio_vs_cs_flip.c index e50a9112..00557aa5 100644 --- a/tests/kms_mmio_vs_cs_flip.c +++ b/tests/kms_mmio_vs_cs_flip.c @@ -474,31 +474,37 @@ test_crtc(data_t *data, igt_output_t *output, enum pipe pipe) } static void -run_plane_test_for_pipe(data_t *data, enum pipe pipe) +run_plane_test(data_t *data) { igt_output_t *output; enum igt_plane plane = 1; /* testing with one sprite is enough */ int valid_tests = 0; - - igt_require(data->display.pipes[pipe].n_planes > 2); + enum pipe pipe; for_each_connected_output(&data->display, output) { - if (test_plane(data, output, pipe, plane)) - valid_tests++; + for_each_pipe(&data->display, pipe) { + igt_require(data->display.pipes[pipe].n_planes > 2); + + if (test_plane(data, output, pipe, plane)) + valid_tests++; + } } igt_require_f(valid_tests, "no valid crtc/connector combinations found\n"); } static void -run_crtc_test_for_pipe(data_t *data, enum pipe pipe) +run_crtc_test(data_t *data) { igt_output_t *output; int valid_tests = 0; + enum pipe pipe; for_each_connected_output(&data->display, output) { - if (test_crtc(data, output, pipe)) - valid_tests++; + for_each_pipe(&data->display, pipe) { + if (test_crtc(data, output, pipe)) + valid_tests++; + } } igt_require_f(valid_tests, "no valid crtc/connector combinations found\n"); @@ -508,8 +514,6 @@ static data_t data; igt_main { - int pipe; - igt_skip_on_simulation(); igt_fixture { @@ -531,15 +535,11 @@ igt_main gem_set_tiling(data.drm_fd, data.busy_bo->handle, 0, 4096); } - igt_subtest_f("setplane_vs_cs_flip") { - for (pipe = 0; pipe < data.display.n_pipes; pipe++) - run_plane_test_for_pipe(&data, pipe); - } + igt_subtest_f("setplane_vs_cs_flip") + run_plane_test(&data); - igt_subtest_f("setcrtc_vs_cs_flip") { - for (pipe = 0; pipe < data.display.n_pipes; pipe++) - run_crtc_test_for_pipe(&data, pipe); - } + igt_subtest_f("setcrtc_vs_cs_flip") + run_crtc_test(&data); igt_fixture { drm_intel_bo_unreference(data.busy_bo); |