[PATCH i-g-t v3 2/2] tests/kms_content_protection: Skip empty vendor names in blocklist

[Kandpal, Suraj]

> -----Original Message-----
> From: Santhosh Reddy Guddati <santhosh.reddy.guddati at intel.com>
> Sent: Thursday, June 12, 2025 10:43 AM
> To: igt-dev at lists.freedesktop.org
> Cc: Nautiyal, Ankit K <ankit.k.nautiyal at intel.com>; Kandpal, Suraj
> <suraj.kandpal at intel.com>; Thasleem, Mohammed
> <mohammed.thasleem at intel.com>; Reddy Guddati, Santhosh
> <santhosh.reddy.guddati at intel.com>
> Subject: [PATCH i-g-t v3 2/2] tests/kms_content_protection: Skip empty
> vendor names in blocklist
> 
> Move the blocklist check after hdcp support check on a connector.
> This will avoid unnecessary skips when multi displays are connected and one
> of the outputs is blocklisted.
> 
> V2: Add Fixes tag (Suraj)
> V3: Split to separate commits and fix indentation (Kamil)
> 
> Fixes: 6a256bfe3025 ("tests/kms_content_protection: Whitelist check for
> panel support")
> Signed-off-by: Santhosh Reddy Guddati <santhosh.reddy.guddati at intel.com>
> ---
>  tests/kms_content_protection.c | 9 ++++++---
>  1 file changed, 6 insertions(+), 3 deletions(-)
> 
> diff --git a/tests/kms_content_protection.c b/tests/kms_content_protection.c
> index b0d0b0234..51fc1d3be 100644
> --- a/tests/kms_content_protection.c
> +++ b/tests/kms_content_protection.c
> @@ -637,9 +637,6 @@ test_content_protection(enum igt_commit_style
> commit_style, int content_type)
>  	}
> 
>  	for_each_connected_output(display, output) {
> -		igt_require_f(!is_output_hdcp_test_exempt(output),
> -			      "Skipped as the panel is blacklisted");
> -
>  		for_each_pipe(display, pipe) {
>  			igt_display_reset(display);
> 
> @@ -652,6 +649,12 @@ test_content_protection(enum igt_commit_style
> commit_style, int content_type)
>  			if (!output_hdcp_capable(output, content_type))
>  				continue;
> 
> +			if (is_output_hdcp_test_exempt(output)) {
> +				igt_info("Skipping HDCP test on %s, as the
> panel is blocklisted\n",
> +					  output->name);
> +				continue;
> +			}
> +
>  			igt_dynamic_f("pipe-%s-%s",
> kmstest_pipe_name(pipe), output->name)
>  				test_content_protection_on_output(output,
> pipe, commit_style, content_type);

LGTM,
Reviewed-by: Suraj Kandpal <suraj.kandpal at intel.com>

> 
> --
> 2.34.1