From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: Received: from mgamail.intel.com (mgamail.intel.com [192.55.52.43]) by gabe.freedesktop.org (Postfix) with ESMTPS id 9C00C10E452 for ; Tue, 16 Jan 2024 09:34:23 +0000 (UTC) Message-ID: <89c29c53-e4b9-403b-826b-a681bad2d9ab@intel.com> Date: Tue, 16 Jan 2024 15:04:11 +0530 Subject: Re: [PATCH i-g-t] tests/kms_content_protection: Add LIC Type 1 scenario To: Suraj Kandpal , References: <20240112103330.169688-1-suraj.kandpal@intel.com> Content-Language: en-US From: "Nautiyal, Ankit K" In-Reply-To: <20240112103330.169688-1-suraj.kandpal@intel.com> Content-Type: text/plain; charset="UTF-8"; format=flowed Content-Transfer-Encoding: 7bit MIME-Version: 1.0 List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , Errors-To: igt-dev-bounces@lists.freedesktop.org Sender: "igt-dev" List-ID: LGTM. Reviewed-by: Ankit Nautiyal On 1/12/2024 4:03 PM, Suraj Kandpal wrote: > Currently LIC is done only for type0 content this patch > adds LIC for type1 content too. > > Signed-off-by: Suraj Kandpal > --- > tests/kms_content_protection.c | 16 ++++++++++++---- > 1 file changed, 12 insertions(+), 4 deletions(-) > > diff --git a/tests/kms_content_protection.c b/tests/kms_content_protection.c > index 6d472ee02..38e4143c8 100644 > --- a/tests/kms_content_protection.c > +++ b/tests/kms_content_protection.c > @@ -43,8 +43,11 @@ > #include "igt_kmod.h" > > /** > - * SUBTEST: lic > - * Description: Test for the integrity of link. > + * SUBTEST: lic-type-0 > + * Description: Test for the integrity of link for type 0 content. > + * > + * SUBTEST: lic-type-1 > + * Description: Test for the integrity of link for type 1 content. > * > * SUBTEST: content-type-change > * Description: Test the content type change when the content protection already > @@ -783,11 +786,16 @@ static const struct { > .cp_tests = CP_DPMS, > .content_type = HDCP_CONTENT_TYPE_0 > }, > - { .desc = "Test for the integrity of link.", > - .name = "lic", > + { .desc = "Test for the integrity of link with type 0 content.", > + .name = "lic-type-0", > .cp_tests = CP_LIC, > .content_type = HDCP_CONTENT_TYPE_0, > }, > + { .desc = "Test for the integrity of link with type 1 content", > + .name = "lic-type-1", > + .cp_tests = CP_LIC, > + .content_type = HDCP_CONTENT_TYPE_1, > + }, > { .desc = "Test content protection with content type 1 " > "that can be handled only through HDCP2.2.", > .name = "type1", From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: X-Spam-Checker-Version: SpamAssassin 3.4.0 (2014-02-07) on aws-us-west-2-korg-lkml-1.web.codeaurora.org Message-ID: <89c29c53-e4b9-403b-826b-a681bad2d9ab@intel.com> Date: Tue, 16 Jan 2024 15:04:11 +0530 Subject: Re: [PATCH i-g-t] tests/kms_content_protection: Add LIC Type 1 scenario References: <20240112103330.169688-1-suraj.kandpal@intel.com> Content-Language: en-US From: "Nautiyal, Ankit K" In-Reply-To: <20240112103330.169688-1-suraj.kandpal@intel.com> Content-Type: text/plain; charset="UTF-8"; format=flowed Content-Transfer-Encoding: 7bit MIME-Version: 1.0 List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , Errors-To: igt-dev-bounces@lists.freedesktop.org Sender: "igt-dev" List-ID: To: Suraj Kandpal , igt-dev@lists.freedesktop.org Message-ID: <20240116093411.wL6VqAVVznNx_4yua02bTAo0NJIXV8aVRVbi-vGYds0@z> LGTM. Reviewed-by: Ankit Nautiyal On 1/12/2024 4:03 PM, Suraj Kandpal wrote: > Currently LIC is done only for type0 content this patch > adds LIC for type1 content too. > > Signed-off-by: Suraj Kandpal > --- > tests/kms_content_protection.c | 16 ++++++++++++---- > 1 file changed, 12 insertions(+), 4 deletions(-) > > diff --git a/tests/kms_content_protection.c b/tests/kms_content_protection.c > index 6d472ee02..38e4143c8 100644 > --- a/tests/kms_content_protection.c > +++ b/tests/kms_content_protection.c > @@ -43,8 +43,11 @@ > #include "igt_kmod.h" > > /** > - * SUBTEST: lic > - * Description: Test for the integrity of link. > + * SUBTEST: lic-type-0 > + * Description: Test for the integrity of link for type 0 content. > + * > + * SUBTEST: lic-type-1 > + * Description: Test for the integrity of link for type 1 content. > * > * SUBTEST: content-type-change > * Description: Test the content type change when the content protection already > @@ -783,11 +786,16 @@ static const struct { > .cp_tests = CP_DPMS, > .content_type = HDCP_CONTENT_TYPE_0 > }, > - { .desc = "Test for the integrity of link.", > - .name = "lic", > + { .desc = "Test for the integrity of link with type 0 content.", > + .name = "lic-type-0", > .cp_tests = CP_LIC, > .content_type = HDCP_CONTENT_TYPE_0, > }, > + { .desc = "Test for the integrity of link with type 1 content", > + .name = "lic-type-1", > + .cp_tests = CP_LIC, > + .content_type = HDCP_CONTENT_TYPE_1, > + }, > { .desc = "Test content protection with content type 1 " > "that can be handled only through HDCP2.2.", > .name = "type1",