[PATCH i-g-t 3/3] test/kms_content_protection: Move test lic mst code.

[B, Jeevan]

LGTM.

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

> -----Original Message-----
> From: Kandpal, Suraj <suraj.kandpal at intel.com>
> Sent: Tuesday, September 10, 2024 3:15 PM
> To: igt-dev at lists.freedesktop.org
> Cc: Thasleem, Mohammed <mohammed.thasleem at intel.com>; B S, Karthik
> <karthik.b.s at intel.com>; B, Jeevan <jeevan.b at intel.com>; Kandpal, Suraj
> <suraj.kandpal at intel.com>
> Subject: [PATCH i-g-t 3/3] test/kms_content_protection: Move test lic mst
> code.
> 
> Move mst lic code before screen color change to avoid upsetting lic test as it
> ends up failing due to the commit that's done without seeting CP property.
> 
> Signed-off-by: Suraj Kandpal <suraj.kandpal at intel.com>
> ---
>  tests/kms_content_protection.c | 6 +++---
>  1 file changed, 3 insertions(+), 3 deletions(-)
> 
> diff --git a/tests/kms_content_protection.c
> b/tests/kms_content_protection.c index c3771fad8..cb4d825cd 100644
> --- a/tests/kms_content_protection.c
> +++ b/tests/kms_content_protection.c
> @@ -760,6 +760,9 @@ test_content_protection_mst(int content_type)
> 
>  	ret = test_mst_cp_enable_with_retry(hdcp_mst_output,
> valid_outputs, 2, content_type);
> 
> +	if (data.cp_tests & CP_LIC)
> +		test_cp_lic_on_mst(hdcp_mst_output, valid_outputs, 0);
> +
>  	if (ret) {
>  		for (i = 0; i < valid_outputs; i++)
> 
> 	prepare_modeset_on_mst_output(hdcp_mst_output[i], true); @@ -
> 767,9 +770,6 @@ test_content_protection_mst(int content_type)
>  		igt_display_commit2(display, COMMIT_ATOMIC);
>  	}
> 
> -	if (data.cp_tests & CP_LIC)
> -		test_cp_lic_on_mst(hdcp_mst_output, valid_outputs, 0);
> -
>  	/*
>  	 * Verify if CP is still enabled on other outputs by disabling CP on the
> first output.
>  	 */
> --
> 2.43.2