[igt-dev] [PATCH i-g-t v7 2/8] kms_content_protection: Link Integrity Check subtest

[Shankar, Uma]

>-----Original Message-----
>From: C, Ramalingam
>Sent: Thursday, May 30, 2019 8:31 PM
>To: Shankar, Uma <uma.shankar at intel.com>; Vetter, Daniel
><daniel.vetter at intel.com>; igt-dev at lists.freedesktop.org
>Cc: Mun, Gwan-gyeong <gwan-gyeong.mun at intel.com>; C, Ramalingam
><ramalingam.c at intel.com>
>Subject: [PATCH i-g-t v7 2/8] kms_content_protection: Link Integrity Check subtest
>
>Existing Link integrity check test is moved into dedicated subtest.
>This helps to reduce the execution time of other tests by removing the repeated Link
>integrity check on every other tests.
>
>v2:
>  s/reduced/reduce [Uma]

Looks good to me.
Reviewed-by: Uma Shankar <uma.shankar at intel.com>

>Signed-off-by: Ramalingam C <ramalingam.c at intel.com>
>Acked-by: Daniel Vetter <daniel.vetter at ffwll.ch>
>---
> tests/kms_content_protection.c | 11 ++++++++++-
> 1 file changed, 10 insertions(+), 1 deletion(-)
>
>diff --git a/tests/kms_content_protection.c b/tests/kms_content_protection.c index
>5f7d8ff9b52b..c86fe936f12f 100644
>--- a/tests/kms_content_protection.c
>+++ b/tests/kms_content_protection.c
>@@ -38,6 +38,7 @@ struct data {
> } data;
>
> #define CP_DPMS					(1 << 0)
>+#define CP_LIC					(1 << 1)
>
> #define CP_UNDESIRED				0
> #define CP_DESIRED				1
>@@ -265,7 +266,9 @@ static void test_content_protection_on_output(igt_output_t
>*output,
>
> 		modeset_with_fb(pipe, output, s);
> 		test_cp_enable_with_retry(output, s, 3);
>-		test_cp_lic(output);
>+
>+		if (data.cp_tests & CP_LIC)
>+			test_cp_lic(output);
>
> 		if (data.cp_tests & CP_DPMS) {
> 			igt_pipe_set_prop_value(display, pipe, @@ -377,6 +380,12
>@@ igt_main
> 		test_content_protection(COMMIT_ATOMIC);
> 	}
>
>+	igt_subtest("LIC") {
>+		igt_require(data.display.is_atomic);
>+		data.cp_tests = CP_LIC;
>+		test_content_protection(COMMIT_ATOMIC);
>+	}
>+
> 	igt_fixture
> 		igt_display_fini(&data.display);
> }
>--
>2.19.1