From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: Received: from mga01.intel.com (mga01.intel.com [192.55.52.88]) by gabe.freedesktop.org (Postfix) with ESMTPS id 1A34F10E0E9 for ; Fri, 30 Dec 2022 15:28:14 +0000 (UTC) From: "Gupta, Nidhi1" To: "igt-dev@lists.freedesktop.org" Date: Fri, 30 Dec 2022 15:27:26 +0000 Message-ID: References: In-Reply-To: Content-Language: en-US Content-Type: multipart/alternative; boundary="_000_PH7PR11MB60062C2F089B2C515A683E97C5F09PH7PR11MB6006namp_" MIME-Version: 1.0 Subject: Re: [igt-dev] [i-g-t v5 33/52] tests/kms_sequence: Add support for Bigjoiner List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , Errors-To: igt-dev-bounces@lists.freedesktop.org Sender: "igt-dev" List-ID: --_000_PH7PR11MB60062C2F089B2C515A683E97C5F09PH7PR11MB6006namp_ Content-Type: text/plain; charset="us-ascii" Content-Transfer-Encoding: quoted-printable On Tue, 2022-11-15 at 08:43 +0530, 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 >--- Reviewed-by: Nidhi Gupta > tests/kms_sequence.c | 6 ++++++ > 1 file changed, 6 insertions(+) >diff --git a/tests/kms_sequence.c b/tests/kms_sequence.c >index c72857a3..c8a3d6bb 100644 >--- a/tests/kms_sequence.c >+++ b/tests/kms_sequence.c >@@ -285,6 +285,12 @@ igt_main > "drmCrtcQue= ueSequence"); > igt_subtest_with_dynamic_f("%s-= %s", f->name, m->name) { > for_each_pipe_wi= th_valid_output(&data.display, p, output) { >+ ig= t_display_reset(&data.display); >+ >+ ig= t_output_set_pipe(output, p); >+ if= (!i915_pipe_output_combo_valid(&data.display)) >+ = continue; >+ > i= gt_dynamic_f("%s-pipe-%s", igt_output_name(output), kmstest_pipe_name(p)) { > = data.pipe =3D p; > = data.output =3D output; --_000_PH7PR11MB60062C2F089B2C515A683E97C5F09PH7PR11MB6006namp_ Content-Type: text/html; charset="us-ascii" Content-Transfer-Encoding: quoted-printable

On Tue, 2022-11-15 at 08:43 +0530, Bhanuprakash M= odem wrote:

>This patch will add a check to Skip the subte= st 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 p= ipe must be available & free.

> 

>V2: - Use updated helper name

> 

>Signed-off-by: Bhanuprakash Modem <bhanupr= akash.modem@intel.com>

>---

Reviewed-by: Nidhi Gupta <nidhi1.gupta@intel.c= om>

> tests/kms_sequence.c | 6 ++++++

> 1 file changed, 6 insertions(+)

>diff --git a/tests/kms_sequence.c b/tests/kms= _sequence.c

>index c72857a3..c8a3d6bb 100644

>--- a/tests/kms_sequence.c

>+++ b/tests/kms_sequence.c

>@@ -285,6 +285,12 @@  igt_main

>        &n= bsp;            = ;            &n= bsp;            = ;             &= nbsp;   "drmCrtcQueueSequence");

>        &n= bsp;            = ;            &n= bsp;         igt_subtest_with_dynam= ic_f("%s-%s", f->name, m->name) {

>        &n= bsp;            = ;            &n= bsp;            = ;            for_eac= h_pipe_with_valid_output(&data.display, p, output) {

>+       &n= bsp;            = ;            &n= bsp;            = ;            &n= bsp;            = ; igt_display_reset(&data.display);

>+

>+       &n= bsp;            = ;            &n= bsp;            = ;            &n= bsp;            = ; igt_output_set_pipe(output, p);

>+       &n= bsp;            = ;            &n= bsp;            = ;            &n= bsp;            = ; if (!i915_pipe_output_combo_valid(&data.display))

>+       &n= bsp;            = ;            &n= bsp;            = ;            &n= bsp;            = ;            &n= bsp;   continue;

>+

>        &n= bsp;            = ;            &n= bsp;            = ;            &n= bsp;            = ;  igt_dynamic_f("%s-pipe-%s", igt_output_name(output), kmst= est_pipe_name(p)) {

>        &n= bsp;            = ;            &n= bsp;            = ;            &n= bsp;            = ;            &n= bsp;    data.pipe =3D p;

>        &n= bsp;            = ;            &n= bsp;            = ;            &n= bsp;            = ;            &n= bsp;    data.output =3D output;

--_000_PH7PR11MB60062C2F089B2C515A683E97C5F09PH7PR11MB6006namp_--