Re: [igt-dev] [i-g-t v6 03/52] tests/kms_async_flips: Add support for Bigjoiner

Karthik B S
2022-12-23T08:05:25Z

On 12/23/2022 11:59 AM, 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
> V3: - Drop unrelated changes
>
> Signed-off-by: Bhanuprakash Modem <bhanuprakash.modem@intel.com>
Reviewed-by: Karthik B S <karthik.b.s@intel.com>
> ---
> tests/kms_async_flips.c | 11 +++++++++--
> 1 file changed, 9 insertions(+), 2 deletions(-)
>
> diff --git a/tests/kms_async_flips.c b/tests/kms_async_flips.c
> index 7cb71f6b..e32ae914 100644
> --- a/tests/kms_async_flips.c
> +++ b/tests/kms_async_flips.c
> @@ -542,9 +542,16 @@ static void run_test(data_t *data, void (*test)(data_t *))
> {
> igt_output_t *output;
> enum pipe pipe;
> + igt_display_t *display = &data->display;
> +
> + for_each_pipe(display, pipe) {
> + 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;
>
> - for_each_pipe(&data->display, pipe) {
> - for_each_valid_output_on_pipe(&data->display, pipe, output) {
> igt_dynamic_f("pipe-%s-%s", kmstest_pipe_name(pipe), output->name) {
> data->output = output;
> data->pipe = pipe;