Re: [PATCH i-g-t] tests/kms_content_protection: Add LIC Type 1 scenario

["Sharma, Swati2" <swati2.sharma@intel.com>]

LGTM

Reviewed-by: Swati Sharma <swati2.sharma@intel.com>

On 18-Jan-24 12:57 PM, Suraj Kandpal wrote:
> Currently LIC is done only for type0 content this patch
> adds LIC for type1 content too.
> 
> --v2
> -Use arg[1] [Swati]
> -Mention Functionality [Swati]
> 
> --v3
> -Move back to prev version of comment as type 0 and type 1
> dont have the same functionality [Swati]
> 
> Cc: Swati Sharma <swati2.sharma@intel.com>
> Signed-off-by: Suraj Kandpal <suraj.kandpal@intel.com>
> Reviewed-by: Ankit Nautiyal <ankit.k.nautiyal@intel.com>
> ---
>   tests/kms_content_protection.c | 18 ++++++++++++++----
>   1 file changed, 14 insertions(+), 4 deletions(-)
> 
> diff --git a/tests/kms_content_protection.c b/tests/kms_content_protection.c
> index 6d472ee02..3fc575634 100644
> --- a/tests/kms_content_protection.c
> +++ b/tests/kms_content_protection.c
> @@ -43,8 +43,13 @@
>   #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.
> + * Functionality: hdcp1.4, hdcp2.2
> + *
> + * SUBTEST: lic-type-1
> + * Description: Test for the integrity of link for type-1 content.
> + * Functionality: hdcp2.2
>    *
>    * SUBTEST: content-type-change
>    * Description: Test the content type change when the content protection already
> @@ -783,11 +788,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",