[igt-dev] [i-g-t v6 44/52] tests/i915/kms_flip_scaled_crc: Add support for Bigjoiner

[Swati Sharma]

This patch LGTM.

Reviewed-by: Swati Sharma <swati2.sharma at intel.com>

On 29-Dec-22 10:47 AM, Bhanuprakash Modem wrote:
> This patch will add a check to Skip the subtest if a selected pipe/output
> combo won't support Bigjoiner or 8K mode.
> 
> Example:
> * Pipe-D wont support a mode > 5K
> * To use 8K mode on a pipe then consecutive pipe must be available & free.
> 
> V2: - Use updated helper name
> V3: - Fix dangling pointer
> 
> Signed-off-by: Bhanuprakash Modem <bhanuprakash.modem at intel.com>
> ---
>   tests/i915/kms_flip_scaled_crc.c | 14 +++++++++++++-
>   1 file changed, 13 insertions(+), 1 deletion(-)
> 
> diff --git a/tests/i915/kms_flip_scaled_crc.c b/tests/i915/kms_flip_scaled_crc.c
> index 10465ca7..7f7a59b9 100644
> --- a/tests/i915/kms_flip_scaled_crc.c
> +++ b/tests/i915/kms_flip_scaled_crc.c
> @@ -648,8 +648,13 @@ igt_main
>   			for_each_pipe(&data.display, pipe) {
>   				bool found = false;
>   				for_each_valid_output_on_pipe(&data.display, pipe, output) {
> +					igt_display_reset(&data.display);
> +
>   					modetoset = find_mode(&data, output);
> -					if (modetoset) {
> +					igt_output_set_pipe(output, pipe);
> +					igt_output_override_mode(output, modetoset);
> +
> +					if (modetoset && i915_pipe_output_combo_valid(&data.display)) {
>   						found = true;
>   						igt_dynamic_f("pipe-%s-valid-mode", kmstest_pipe_name(pipe))
>   							run_tests(&data, index, pipe, output, modetoset);
> @@ -658,6 +663,13 @@ igt_main
>   				}
>   				if (!found) {
>   					for_each_valid_output_on_pipe(&data.display, pipe, output) {
> +						igt_display_reset(&data.display);
> +
> +						igt_output_set_pipe(output, pipe);
> +						if (!i915_pipe_output_combo_valid(&data.display))
> +							continue;
> +
> +						modetoset = NULL;
>   						igt_dynamic_f("pipe-%s-default-mode", kmstest_pipe_name(pipe))
>   							run_tests(&data, index, pipe, output, modetoset);
>   					}

-- 
~Swati Sharma