Re: [PATCH i-g-t] tests/kms: Fix documentation

[Karthik B S <karthik.b.s@intel.com>]

On 3/12/2025 2:12 PM, Swati Sharma wrote:
> Missed few KMS binaries, drop 'Functionality' and 'Test category'
> from them too.
>
> Fixes: 25039689b609 ("tests/kms: Drop 'Functionality' and 'Test category' from
> documentation")
> Signed-off-by: Swati Sharma <swati2.sharma@intel.com>
Reviewed-by: Karthik B S <karthik.b.s@intel.com>
> ---
>   tests/intel/kms_psr2_su.c   | 3 ---
>   tests/kms_prime.c           | 3 ---
>   tests/kms_rmfb.c            | 2 --
>   tests/kms_universal_plane.c | 3 ---
>   4 files changed, 11 deletions(-)
>
> diff --git a/tests/intel/kms_psr2_su.c b/tests/intel/kms_psr2_su.c
> index 0099daf1d..e9bc6f6c6 100644
> --- a/tests/intel/kms_psr2_su.c
> +++ b/tests/intel/kms_psr2_su.c
> @@ -28,7 +28,6 @@
>    * Description: Test PSR2 selective update
>    * Driver requirement: i915, xe
>    * Mega feature: PSR
> - * Test category: functionality test
>    */
>   
>   #include "igt.h"
> @@ -43,11 +42,9 @@
>   /**
>    * SUBTEST: frontbuffer-XRGB8888
>    * Description: Test that selective update works when screen changes
> - * Functionality: fbt, psr2, selective_update
>    *
>    * SUBTEST: page_flip-%s
>    * Description: Test the selective update with %arg[1] when screen changes
> - * Functionality: pixel_formats, psr2, selective_update
>    *
>    * arg[1]:
>    *
> diff --git a/tests/kms_prime.c b/tests/kms_prime.c
> index 7f46c6842..1d011327c 100644
> --- a/tests/kms_prime.c
> +++ b/tests/kms_prime.c
> @@ -26,9 +26,7 @@
>    * Category: Display
>    * Description: Prime tests, focusing on KMS side
>    * Driver requirement: i915, xe
> - * Functionality: hybrid, kms_gem_interop
>    * Mega feature: General Display Features
> - * Test category: functionality test
>    */
>   
>   #include "igt.h"
> @@ -51,7 +49,6 @@
>    * SUBTEST: basic-crc-%s
>    * Description: Make a dumb color buffer, export to another device and compare
>    *              the CRCs with a buffer native to that device
> - * Functionality: crc, hybrid, kms_gem_interop
>    *
>    * arg[1]:
>    *
> diff --git a/tests/kms_rmfb.c b/tests/kms_rmfb.c
> index e4203bdd4..acda4b528 100644
> --- a/tests/kms_rmfb.c
> +++ b/tests/kms_rmfb.c
> @@ -27,9 +27,7 @@
>    * Description: This tests rmfb and close-fd behavior. In these cases the
>    *              framebuffers should be removed from the crtc.
>    * Driver requirement: i915, xe
> - * Functionality: kms_gem_interop
>    * Mega feature: General Display Features
> - * Test category: functionality test
>    */
>   
>   #include "igt.h"
> diff --git a/tests/kms_universal_plane.c b/tests/kms_universal_plane.c
> index 7e80b1389..8e0b4dda0 100644
> --- a/tests/kms_universal_plane.c
> +++ b/tests/kms_universal_plane.c
> @@ -26,9 +26,7 @@
>    * Category: Display
>    * Description: Check pageflip & modeset on universal plane
>    * Driver requirement: i915, xe
> - * Functionality: plane
>    * Mega feature: General Display Features
> - * Test category: functionality test
>    */
>   
>   #include "igt.h"
> @@ -41,7 +39,6 @@
>   /**
>    * SUBTEST: cursor-fb-leak
>    * Description: Check for cursor leaks after performing cursor operations
> - * Functionality: cursor, plane
>    *
>    * SUBTEST: disable-primary-vs-flip
>    * Description: Check pageflips while primary plane is disabled before IOCTL or