From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: Received: from mga09.intel.com (mga09.intel.com [134.134.136.24]) by gabe.freedesktop.org (Postfix) with ESMTPS id 7EB7F10E1AF for ; Fri, 30 Sep 2022 06:45:41 +0000 (UTC) Message-ID: <5b840167-5ba2-a77e-4ce4-20c2e1fb4012@intel.com> Date: Fri, 30 Sep 2022 12:14:44 +0530 Content-Language: en-US To: Bhanuprakash Modem , References: <20220922160004.2041598-1-bhanuprakash.modem@intel.com> <20220922160004.2041598-20-bhanuprakash.modem@intel.com> From: "Thasleem, Mohammed" In-Reply-To: <20220922160004.2041598-20-bhanuprakash.modem@intel.com> Content-Type: text/plain; charset="UTF-8"; format=flowed Content-Transfer-Encoding: 7bit MIME-Version: 1.0 Subject: Re: [igt-dev] [i-g-t V4 19/52] tests/kms_lease: 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: 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 > --- > 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