From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: Received: from mgamail.intel.com (mgamail.intel.com [134.134.136.100]) by gabe.freedesktop.org (Postfix) with ESMTPS id A535810E012 for ; Tue, 26 Sep 2023 16:15:35 +0000 (UTC) Message-ID: <1a8fefd9-ee54-0c8f-5deb-bc966c584f4d@intel.com> Date: Tue, 26 Sep 2023 21:44:55 +0530 Content-Language: en-US To: Swati Sharma , References: <20230918095033.257372-1-swati2.sharma@intel.com> <20230918095033.257372-3-swati2.sharma@intel.com> From: "Modem, Bhanuprakash" In-Reply-To: <20230918095033.257372-3-swati2.sharma@intel.com> Content-Type: text/plain; charset="UTF-8"; format=flowed Content-Transfer-Encoding: 7bit MIME-Version: 1.0 Subject: Re: [igt-dev] [PATCH i-g-t 2/2] tests/kms_universal_plane: update testplan doc List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , Errors-To: igt-dev-bounces@lists.freedesktop.org Sender: "igt-dev" List-ID: On Mon-18-09-2023 03:20 pm, Swati Sharma wrote: > After converting tests to dynamic subtests, hardcoded pipe > names are removed. Update testplan documentation accordingly. > > Signed-off-by: Swati Sharma Reviewed-by: Bhanuprakash Modem > --- > tests/kms_universal_plane.c | 38 ++++++++----------------------------- > 1 file changed, 8 insertions(+), 30 deletions(-) > > diff --git a/tests/kms_universal_plane.c b/tests/kms_universal_plane.c > index 45278916d..85d986456 100644 > --- a/tests/kms_universal_plane.c > +++ b/tests/kms_universal_plane.c > @@ -33,43 +33,32 @@ > * Category: Display > * Description: Check pageflip & modeset on universal plane > * > - * SUBTEST: cursor-fb-leak-pipe-%s > - * Description: Check for cursor leaks after performing cursor operations on %arg[1] > + * SUBTEST: cursor-fb-leak > + * Description: Check for cursor leaks after performing cursor operations > * Driver requirement: i915, xe > * Functionality: cursor, plane > * Mega feature: General Display Features > * Test category: functionality test > * > - * SUBTEST: disable-primary-vs-flip-pipe-%s > + * SUBTEST: disable-primary-vs-flip > * Description: Check pageflips while primary plane is disabled before IOCTL or > - * between IOCTL and pageflip execution on %arg[1] > + * between IOCTL and pageflip execution > * Driver requirement: i915, xe > * Functionality: plane > * Mega feature: General Display Features > * Test category: functionality test > * > - * SUBTEST: universal-plane-pageflip-windowed-pipe-%s > - * Description: Check if pageflip succeeds in windowed setting on %arg[1] > + * SUBTEST: universal-plane-pageflip-windowed > + * Description: Check if pageflip succeeds in windowed setting > * Driver requirement: i915, xe > * Functionality: plane > * Mega feature: General Display Features > * Test category: functionality test > - * > - * arg[1]: > - * > - * @A: pipe A > - * @B: pipe B > - * @C: pipe C > - * @D: pipe D > - * @E: pipe E > - * @F: pipe F > - * @G: pipe G > - * @H: pipe H > */ > > /** > - * SUBTEST: universal-plane-pipe-%s-%s > - * Description: Check %arg[2] on %arg[1] > + * SUBTEST: universal-plane-%s > + * Description: Check %arg[1] > * Driver requirement: i915, xe > * Functionality: plane > * Mega feature: General Display Features > @@ -77,17 +66,6 @@ > * > * arg[1]: > * > - * @A: pipe A > - * @B: pipe B > - * @C: pipe C > - * @D: pipe D > - * @E: pipe E > - * @F: pipe F > - * @G: pipe G > - * @H: pipe H > - * > - * arg[2]: > - * > * @functional: the switching between different primary plane fbs with CRTC off > * @sanity: for scale-up or scale-down using universal plane API without > * covering CRTC