Re: [i-g-t 1/2] tests/kms_hdr: Fix bpc-switch subtests

[Karthik B S <karthik.b.s@intel.com>]

On 6/25/2024 10:01 AM, Bhanuprakash Modem wrote:
> igt@kms_hdr@bpc-switch.* subtests are getting skip due to
> recently merged cleanup patch. This patch will fix the logic
> to run the subtests.
>
> Fixes: (4fe91292b) tests/kms_hdr: Use lib helper to check the pipe/output combo validity
> Cc: Karthik B S <karthik.b.s@intel.com>
> Signed-off-by: Bhanuprakash Modem <bhanuprakash.modem@intel.com>
Reviewed-by: Karthik B S <karthik.b.s@intel.com>
> ---
>   tests/kms_hdr.c | 2 +-
>   1 file changed, 1 insertion(+), 1 deletion(-)
>
> diff --git a/tests/kms_hdr.c b/tests/kms_hdr.c
> index 475463c21..36ebb4f4e 100644
> --- a/tests/kms_hdr.c
> +++ b/tests/kms_hdr.c
> @@ -274,7 +274,7 @@ static void test_bpc_switch(data_t *data, uint32_t flags)
>   
>   		for_each_pipe(display, pipe) {
>   			igt_output_set_pipe(output, pipe);
> -			if (intel_pipe_output_combo_valid(display)) {
> +			if (!intel_pipe_output_combo_valid(display)) {
>   				igt_output_set_pipe(output, PIPE_NONE);
>   				continue;
>   			}