diff options
| author | Arkadiusz Hiler <arkadiusz.hiler@intel.com> | 2019-03-11 15:31:50 +0200 |
|---|---|---|
| committer | Arkadiusz Hiler <arkadiusz.hiler@intel.com> | 2019-03-12 15:21:20 +0200 |
| commit | b29af69f2ce6135bb1793b70a906ab5ef4fc91c0 (patch) | |
| tree | bc3f7ccd976fd9ea09ef549968bb4ec4be95f8ad /tests/kms_plane_multiple.c | |
| parent | e0537c6a995b753e7c44b933fc52b307866bd876 (diff) | |
tests/kms_plane_multiple: Do not iterate over output twice
test_plane_position already iterates over outputs using
for_each_valid_output_on_pipe so there is no need to do that twice,
especially with unused *output.
Cc: Mika Kahola <mika.kahola@intel.com>
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: Martin Peres <martin.peres@linux.intel.com>
Reviewed-by: Daniel Vetter <daniel.vetter@ffwll.ch>
Diffstat (limited to 'tests/kms_plane_multiple.c')
| -rw-r--r-- | tests/kms_plane_multiple.c | 12 |
1 files changed, 4 insertions, 8 deletions
diff --git a/tests/kms_plane_multiple.c b/tests/kms_plane_multiple.c index 617054d9..bfaeeded 100644 --- a/tests/kms_plane_multiple.c +++ b/tests/kms_plane_multiple.c @@ -341,20 +341,16 @@ run_tests_for_pipe(data_t *data, enum pipe pipe) } igt_subtest_f("atomic-pipe-%s-tiling-x", kmstest_pipe_name(pipe)) - for_each_valid_output_on_pipe(&data->display, pipe, output) - test_plane_position(data, pipe, LOCAL_I915_FORMAT_MOD_X_TILED); + test_plane_position(data, pipe, LOCAL_I915_FORMAT_MOD_X_TILED); igt_subtest_f("atomic-pipe-%s-tiling-y", kmstest_pipe_name(pipe)) - for_each_valid_output_on_pipe(&data->display, pipe, output) - test_plane_position(data, pipe, LOCAL_I915_FORMAT_MOD_Y_TILED); + test_plane_position(data, pipe, LOCAL_I915_FORMAT_MOD_Y_TILED); igt_subtest_f("atomic-pipe-%s-tiling-yf", kmstest_pipe_name(pipe)) - for_each_valid_output_on_pipe(&data->display, pipe, output) - test_plane_position(data, pipe, LOCAL_I915_FORMAT_MOD_Yf_TILED); + test_plane_position(data, pipe, LOCAL_I915_FORMAT_MOD_Yf_TILED); igt_subtest_f("atomic-pipe-%s-tiling-none", kmstest_pipe_name(pipe)) - for_each_valid_output_on_pipe(&data->display, pipe, output) - test_plane_position(data, pipe, LOCAL_DRM_FORMAT_MOD_NONE); + test_plane_position(data, pipe, LOCAL_DRM_FORMAT_MOD_NONE); } static data_t data; |