[igt-dev] [PATCH i-g-t v2] tests/kms_atomic_interruptible:add test description

[B, Jeevan]

>-----Original Message-----
>From: Arundhati Hagaragi <arundhati.hagaragi at intel.com>
>Sent: Monday, January 11, 2021 1:34 AM
>To: igt-dev at lists.freedesktop.org
>Cc: Hagaragi, Arundhati <arundhati.hagaragi at intel.com>; Gupta; Gupta,
>Nidhi1 <nidhi1.gupta at intel.com>; B; B, Jeevan <jeevan.b at intel.com>
>Subject: [PATCH i-g-t v2] tests/kms_atomic_interruptible:add test
>description
>
>Added Test description for test case
>
>Signed-off-by: Arundhati Hagaragi <arundhati.hagaragi at intel.com>
>Cc: Gupta, Nidhi1 <nidhi1.gupta at intel.com>
>Cc: B, Jeevan <Jeevan.B at intel.com>
>
>v2: modified description to be more specific. (Jeevan)
>---
> tests/kms_atomic_interruptible.c | 7 +++++++
> 1 file changed, 7 insertions(+)
>
>diff --git a/tests/kms_atomic_interruptible.c
>b/tests/kms_atomic_interruptible.c
>index 3ba75089..9d19055c 100644
>--- a/tests/kms_atomic_interruptible.c
>+++ b/tests/kms_atomic_interruptible.c
>@@ -284,6 +284,7 @@ igt_main
> 		igt_require_sw_sync();
> 	}
>
>+	igt_describe("Tests the interrupt properties of legacy modeset");
> 	igt_subtest_with_dynamic("legacy-setmode") {
> 		for_each_pipe_with_valid_output(&display, pipe, output) {
> 			igt_dynamic_f("%s-pipe-%s",
>igt_output_name(output), kmstest_pipe_name(pipe)) @@ -292,6 +293,7
>@@ igt_main
> 		}
> 	}
>
>+	igt_describe("Tests the interrupt properties of atomic modeset");
> 	igt_subtest_with_dynamic("atomic-setmode") {
> 		for_each_pipe_with_valid_output(&display, pipe, output) {
> 			igt_dynamic_f("%s-pipe-%s",
>igt_output_name(output), kmstest_pipe_name(pipe)) @@ -300,6 +302,7
>@@ igt_main
> 		}
> 	}
>
>+	igt_describe("Tests the interrupt properties for DPMS");
> 	igt_subtest_with_dynamic("legacy-dpms") {
> 		for_each_pipe_with_valid_output(&display, pipe, output) {
> 			igt_dynamic_f("%s-pipe-%s",
>igt_output_name(output), kmstest_pipe_name(pipe)) @@ -308,6 +311,7
>@@ igt_main
> 		}
> 	}
>
>+	igt_describe("Tests the interrupt properties for pageflip");
> 	igt_subtest_with_dynamic("legacy-pageflip") {
> 		for_each_pipe_with_valid_output(&display, pipe, output) {
> 			igt_dynamic_f("%s-pipe-%s",
>igt_output_name(output), kmstest_pipe_name(pipe)) @@ -316,6 +320,7
>@@ igt_main
> 		}
> 	}
>
>+	igt_describe("Tests the interrupt properties for cursor");
> 	igt_subtest_with_dynamic("legacy-cursor") {
> 		for_each_pipe_with_valid_output(&display, pipe, output) {
> 			igt_dynamic_f("%s-pipe-%s",
>igt_output_name(output), kmstest_pipe_name(pipe)) @@ -324,6 +329,7
>@@ igt_main
> 		}
> 	}
>
>+	igt_describe("Tests the interrupt properties for primary plane");
> 	igt_subtest_with_dynamic("universal-setplane-primary") {
> 		for_each_pipe_with_valid_output(&display, pipe, output) {
> 			igt_dynamic_f("%s-pipe-%s",
>igt_output_name(output), kmstest_pipe_name(pipe)) @@ -332,6 +338,7
>@@ igt_main
> 		}
> 	}
>
>+	igt_describe("Tests the interrupt properties for cursor plane");
> 	igt_subtest_with_dynamic("universal-setplane-cursor") {
> 		for_each_pipe_with_valid_output(&display, pipe, output) {
> 			igt_dynamic_f("%s-pipe-%s",
>igt_output_name(output), kmstest_pipe_name(pipe))

Looks fine. 
Reviewed-by: Jeevan B <jeevan.b at intel.com>
>--
>2.25.1