Re: [igt-dev] [i-g-t v8 10/51] tests/kms_content_protection: Add support for Bigjoiner

Karthik B S
2023-03-17T06:31:18Z

On 2/23/2023 8:39 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.

Please add the revision history details.

With this added,

Reviewed-by: Karthik B S <karthik.b.s@intel.com>

>
> Signed-off-by: Bhanuprakash Modem <bhanuprakash.modem@intel.com>
> ---
> tests/kms_content_protection.c | 6 +++++-
> 1 file changed, 5 insertions(+), 1 deletion(-)
>
> diff --git a/tests/kms_content_protection.c b/tests/kms_content_protection.c
> index aa28b7bfd..638e1301a 100644
> --- a/tests/kms_content_protection.c
> +++ b/tests/kms_content_protection.c
> @@ -174,7 +174,6 @@ static void modeset_with_fb(const enum pipe pipe, igt_output_t *output,
> igt_plane_t *primary;
>
> mode = igt_output_get_mode(output);
> - igt_output_set_pipe(output, pipe);
>
> primary = igt_output_get_plane_type(output, DRM_PLANE_TYPE_PRIMARY);
> igt_plane_set_fb(primary, &data.red);
> @@ -502,6 +501,11 @@ 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))
> + continue;
> +
> modeset_with_fb(pipe, output, s);
>
> if (!output_hdcp_capable(output, content_type))