[igt-dev] [i-g-t V4 29/52] tests/kms_properties: Add support for Bigjoiner

[Thasleem, Mohammed]

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 at intel.com>
> ---
>   tests/kms_properties.c | 18 ++++++++++++++++++
>   1 file changed, 18 insertions(+)
> 
> diff --git a/tests/kms_properties.c b/tests/kms_properties.c
> index dd5a93aa..953fb3fb 100644
> --- a/tests/kms_properties.c
> +++ b/tests/kms_properties.c
> @@ -241,6 +241,12 @@ static void plane_properties(igt_display_t *display, bool atomic)
>   		found = false;
>   
>   		for_each_valid_output_on_pipe(display, pipe, output) {
> +			igt_display_reset(display);
> +
> +			igt_output_set_pipe(output, pipe);
> +			if (!i915_pipe_output_combo_valid(display))
> +				continue;
> +
>   			found_any = found = true;
>   
>   			run_plane_property_tests(display, pipe, output, atomic);
> @@ -264,6 +270,12 @@ static void crtc_properties(igt_display_t *display, bool atomic)
>   		found = false;
>   
>   		for_each_valid_output_on_pipe(display, pipe, output) {
> +			igt_display_reset(display);
> +
> +			igt_output_set_pipe(output, pipe);
> +			if (!i915_pipe_output_combo_valid(display))
> +				continue;
> +
>   			found_any_valid_pipe = found = true;
>   
>   			run_crtc_property_tests(display, pipe, output, atomic);
> @@ -290,6 +302,12 @@ static void connector_properties(igt_display_t *display, bool atomic)
>   			if (!igt_pipe_connector_valid(pipe, output))
>   				continue;
>   
> +			igt_display_reset(display);
> +
> +			igt_output_set_pipe(output, pipe);
> +			if (!i915_pipe_output_combo_valid(display))
> +				continue;
> +
>   			found = true;
>   			run_connector_property_tests(display, pipe, output, atomic);
>   			break;
Reviewed-by: Mohammed Thasleem <mohammed.thasleem at intel.com>