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

Ramalingam C
2019-07-03T15:01:22Z

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]

Signed-off-by: Ramalingam C <ramalingam.c@intel.com>
Acked-by: Daniel Vetter <daniel.vetter@ffwll.ch>
Reviewed-by: Uma Shankar <uma.shankar@intel.com>
---
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);