Re: [igt-dev] [i-g-t V4 19/52] tests/kms_lease: Add support for Bigjoiner

["Thasleem, Mohammed" <mohammed.thasleem@intel.com>]

On 9/22/2022 9:29 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@intel.com>
> ---
>   tests/kms_lease.c | 8 ++++++--
>   1 file changed, 6 insertions(+), 2 deletions(-)
> 
> diff --git a/tests/kms_lease.c b/tests/kms_lease.c
> index 67e8d3be..a52d8bfc 100644
> --- a/tests/kms_lease.c
> +++ b/tests/kms_lease.c
> @@ -1213,10 +1213,14 @@ igt_main
>   			igt_describe(f->desc);
>   			igt_subtest_with_dynamic_f("%s", f->name) {
>   				for_each_pipe_with_valid_output(display, data.pipe, output) {
> +					igt_display_reset(display);
> +
> +					igt_output_set_pipe(output, data.pipe);
> +					if (!i915_pipe_output_combo_valid(display))
> +						continue;
> +
>   					igt_dynamic_f("pipe-%s-%s", kmstest_pipe_name(data.pipe),
>   						      igt_output_name(output)) {
> -						igt_display_reset(display);
> -						igt_display_commit(display);
>   						data.crtc_id = display->pipes[data.pipe].crtc_id;
>   						data.connector_id = output->id;
>   						data.plane_id =
Reviewed-by: Mohammed Thasleem <mohammed.thasleem@intel.com>