Re: [igt-dev] [PATCH i-g-t 2/2] tests/kms_universal_plane: update testplan doc

["Modem, Bhanuprakash" <bhanuprakash.modem@intel.com>]

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 <swati2.sharma@intel.com>

Reviewed-by: Bhanuprakash Modem <bhanuprakash.modem@intel.com>

> ---
>   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