diff options
author | Arkadiusz Hiler <arkadiusz.hiler@intel.com> | 2019-04-09 16:48:00 +0300 |
---|---|---|
committer | Arkadiusz Hiler <arkadiusz.hiler@intel.com> | 2019-04-10 18:18:29 +0300 |
commit | a636371c0aac1c9119900634d4a56f53e923b9b7 (patch) | |
tree | a5c31d7e2a0331c47828c53198b19b7e4b38c0bc | |
parent | a558c2e2405473f4157ca71638e5a111ebfa80d1 (diff) |
tests/kms_plane_multiple: Run only on the first valid output
There is no real need for testing on each output, as they do not affect
CRC coming out from the pipe. Let's use first viable one.
Cc: Maarten Lankhorst <maarten.lankhorst@linux.intel.com>
Cc: Martin Peres <martin.peres@linux.intel.com>
Signed-off-by: Arkadiusz Hiler <arkadiusz.hiler@intel.com>
Reviewed-by: Ville Syrjälä <ville.syrjala@linux.intel.com>
-rw-r--r-- | tests/kms_plane_multiple.c | 26 |
1 files changed, 5 insertions, 21 deletions
diff --git a/tests/kms_plane_multiple.c b/tests/kms_plane_multiple.c index bfaeeded..d2d02a5f 100644 --- a/tests/kms_plane_multiple.c +++ b/tests/kms_plane_multiple.c @@ -302,41 +302,25 @@ static void test_plane_position(data_t *data, enum pipe pipe, uint64_t tiling) { igt_output_t *output; - int connected_outs; int n_planes = data->display.pipes[pipe].n_planes; + output = igt_get_single_output_for_pipe(&data->display, pipe); + igt_require(output); + if (!opt.user_seed) opt.seed = time(NULL); srand(opt.seed); - connected_outs = 0; - for_each_valid_output_on_pipe(&data->display, pipe, output) { - test_plane_position_with_output(data, pipe, - output, - n_planes, - tiling); - connected_outs++; - } - - igt_skip_on(connected_outs == 0); - + test_plane_position_with_output(data, pipe, output, + n_planes, tiling); } static void run_tests_for_pipe(data_t *data, enum pipe pipe) { - igt_output_t *output; - igt_fixture { - int valid_tests = 0; - igt_skip_on(pipe >= data->display.n_pipes); - - for_each_valid_output_on_pipe(&data->display, pipe, output) - valid_tests++; - - igt_require_f(valid_tests, "no valid crtc/connector combinations found\n"); igt_require(data->display.pipes[pipe].n_planes > 0); } |