diff options
author | Bhanuprakash Modem <bhanuprakash.modem@intel.com> | 2024-06-18 11:54:01 +0530 |
---|---|---|
committer | Bhanuprakash Modem <bhanuprakash.modem@intel.com> | 2024-06-18 15:51:51 +0530 |
commit | 02cb86f2338e9c6f23b348463f656d3e993af81d (patch) | |
tree | 8dd54ddf4c4476f7067a46a62401ec5087e166d8 /tests/kms_vrr.c | |
parent | 6dcde1dfbb8e7cba11e406b0bcc7980dadacd192 (diff) |
tests/kms: Drop redundant check for pipe/output combo validity
As intel_pipe_output_combo_valid() is taking care of pipe/output
combo validity, drop other redundant checks.
Signed-off-by: Bhanuprakash Modem <bhanuprakash.modem@intel.com>
Reviewed-by: Jeevan B <jeevan.b@intel.com>
Diffstat (limited to 'tests/kms_vrr.c')
-rw-r--r-- | tests/kms_vrr.c | 22 |
1 files changed, 10 insertions, 12 deletions
diff --git a/tests/kms_vrr.c b/tests/kms_vrr.c index 7f64d6806..5af220b5b 100644 --- a/tests/kms_vrr.c +++ b/tests/kms_vrr.c @@ -794,22 +794,20 @@ run_vrr_test(data_t *data, test_t test, uint32_t flags) continue; for_each_pipe(&data->display, pipe) { - if (igt_pipe_connector_valid(pipe, output)) { - igt_output_set_pipe(output, pipe); + igt_output_set_pipe(output, pipe); - if (!intel_pipe_output_combo_valid(&data->display)) { - igt_output_set_pipe(output, PIPE_NONE); - continue; - } + if (!intel_pipe_output_combo_valid(&data->display)) { + igt_output_set_pipe(output, PIPE_NONE); + continue; + } - igt_dynamic_f("pipe-%s-%s", - kmstest_pipe_name(pipe), output->name) - test(data, pipe, output, flags); + igt_dynamic_f("pipe-%s-%s", + kmstest_pipe_name(pipe), output->name) + test(data, pipe, output, flags); - test_cleanup(data, pipe, output); + test_cleanup(data, pipe, output); - break; - } + break; } } } |