[igt-dev] [i-g-t v5 11/52] tests/kms_content_protection: Add support for Bigjoiner

[Karthik B S]

On 11/15/2022 10:38 PM, 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
>
> Signed-off-by: Bhanuprakash Modem <bhanuprakash.modem at intel.com>
> ---
>   tests/kms_content_protection.c | 15 +++++++++++++++
>   1 file changed, 15 insertions(+)
>
> diff --git a/tests/kms_content_protection.c b/tests/kms_content_protection.c
> index aa28b7bf..0aa2a8a2 100644
> --- a/tests/kms_content_protection.c
> +++ b/tests/kms_content_protection.c
> @@ -502,6 +502,13 @@ test_content_protection(enum igt_commit_style s, int content_type)
>   				continue;
>   
>   			igt_display_reset(display);
> +
> +			igt_output_set_pipe(output, pipe);
> +			if (!i915_pipe_output_combo_valid(display)) {
> +				igt_output_set_pipe(output, PIPE_NONE);
> +				continue;
> +			}
> +
>   			modeset_with_fb(pipe, output, s);
>   
>   			if (!output_hdcp_capable(output, content_type))
> @@ -623,6 +630,14 @@ test_content_protection_mst(int content_type)
>   		igt_assert_f(pipe_found, "No valid pipe found for %s\n", output->name);
>   
>   		igt_output_set_pipe(output, pipe);
> +		if (!i915_pipe_output_combo_valid(display)){
> +			igt_output_set_pipe(output, PIPE_NONE);
> +			continue;
> +		}

Hi,

Can this check be moved into the previous loop for finding pipe? So that 
even if a combo is invalid we still try with the next pipe. With current 
logic the an output will skip if this check fails.

> +
> +		igt_info("Using (pipe %s + %s) to run the subtest.\n",
> +			 kmstest_pipe_name(pipe), igt_output_name(output));
> +

Please remove this as in mst case 2 or more displays are used in the 
subtest.

This info could be even updated to be more specific, but I would prefer 
that to be done in a separate clean up patch.

Thanks,
Karthik.B.S
>   		prepare_modeset_on_mst_output(output);
>   		mst_output[dp_mst_outputs++] = output;
>   	}