Re: [igt-dev] [i-g-t v5 13/52] tests/kms_cursor_edge_walk: Add support for Bigjoiner

Thasleem, Mohammed
2022-12-27T06:32:04Z

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@intel.com>
LGTM
Reviewed-by: Mohammed Thasleem <mohammed.thasleem@intel.com>

> ---
> tests/kms_cursor_edge_walk.c | 6 ++++++
> 1 file changed, 6 insertions(+)
>
> diff --git a/tests/kms_cursor_edge_walk.c b/tests/kms_cursor_edge_walk.c
> index e653b9ab..86b880d8 100644
> --- a/tests/kms_cursor_edge_walk.c
> +++ b/tests/kms_cursor_edge_walk.c
> @@ -351,6 +351,12 @@ igt_main_args("", long_opts, help_str, opt_handler, &data)
> data.curh = data.curw;
> igt_require(data.curw <= max_curw && data.curh <= max_curh);
>
> + igt_display_reset(&data.display);
> + igt_output_set_pipe(data.output, data.pipe);
> + if (!i915_pipe_output_combo_valid(&data.display))
> + continue;
> +
> + igt_output_set_pipe(data.output, PIPE_NONE);
> igt_dynamic_f("pipe-%s-%s-%dx%d",
> kmstest_pipe_name(data.pipe),
> data.output->name,