From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: Received: from mga18.intel.com (mga18.intel.com [134.134.136.126]) by gabe.freedesktop.org (Postfix) with ESMTPS id 1238910E096 for ; Fri, 17 Mar 2023 06:33:59 +0000 (UTC) Message-ID: <07dae8d0-72ff-7206-e5e7-236779789e13@intel.com> Date: Fri, 17 Mar 2023 12:03:09 +0530 Content-Language: en-US To: Karthik B S , References: <20230223151018.2565040-1-bhanuprakash.modem@intel.com> <20230223151018.2565040-11-bhanuprakash.modem@intel.com> <33dec22f-7723-7bae-aec1-30bc95fae46d@intel.com> From: "Modem, Bhanuprakash" In-Reply-To: <33dec22f-7723-7bae-aec1-30bc95fae46d@intel.com> Content-Type: text/plain; charset="UTF-8"; format=flowed Content-Transfer-Encoding: 8bit MIME-Version: 1.0 Subject: Re: [igt-dev] [i-g-t v8 10/51] tests/kms_content_protection: 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: HI Karthik, On Fri-17-03-2023 12:01 pm, Karthik B S wrote: > > On 2/23/2023 8:39 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. > > Please add the revision history details. Thanks for the review, will add those details while pushing the patch. - Bhanu > > With this added, > > Reviewed-by: Karthik B S > >> >> Signed-off-by: Bhanuprakash Modem >> --- >>   tests/kms_content_protection.c | 6 +++++- >>   1 file changed, 5 insertions(+), 1 deletion(-) >> >> diff --git a/tests/kms_content_protection.c >> b/tests/kms_content_protection.c >> index aa28b7bfd..638e1301a 100644 >> --- a/tests/kms_content_protection.c >> +++ b/tests/kms_content_protection.c >> @@ -174,7 +174,6 @@ static void modeset_with_fb(const enum pipe pipe, >> igt_output_t *output, >>       igt_plane_t *primary; >>       mode = igt_output_get_mode(output); >> -    igt_output_set_pipe(output, pipe); >>       primary = igt_output_get_plane_type(output, >> DRM_PLANE_TYPE_PRIMARY); >>       igt_plane_set_fb(primary, &data.red); >> @@ -502,6 +501,11 @@ test_content_protection(enum igt_commit_style s, >> int content_type) >>                   continue; >>               igt_display_reset(display); >> + >> +            igt_output_set_pipe(output, pipe); >> +            if (!i915_pipe_output_combo_valid(display)) >> +                continue; >> + >>               modeset_with_fb(pipe, output, s); >>               if (!output_hdcp_capable(output, content_type))