Re: [PATCH 3/5] tools/amd_hdmi_compliance: Fix VIC 0 not working

[Rodrigo Siqueira Jordao <Rodrigo.Siqueira@amd.com>]

On 10/24/24 3:03 PM, Alex Hung wrote:
> From: Stylon Wang <stylon.wang@amd.com>
> 
> [WHY]
> VIC 0 is not working with -t option
> 
> [HOW]
> Make 0 pass as a valid VIC
> 
> Signed-off-by: Alex Hung <alex.hung@amd.com>
> Signed-off-by: Stylon Wang <stylon.wang@amd.com>
> ---
>   tools/amd_hdmi_compliance.c | 2 +-
>   1 file changed, 1 insertion(+), 1 deletion(-)
> 
> diff --git a/tools/amd_hdmi_compliance.c b/tools/amd_hdmi_compliance.c
> index c0a324eb9..f06af72b0 100644
> --- a/tools/amd_hdmi_compliance.c
> +++ b/tools/amd_hdmi_compliance.c
> @@ -527,7 +527,7 @@ int main(int argc, char **argv)
>   	if (max_bpc)
>   		set_max_bpc(&data, max_bpc, conn_id);
>   
> -	if (vic) {
> +	if (vic >= 0) {

Maybe you should squash this commit in the first one.

>   		if (vic > ARRAY_SIZE(test_modes) ||
>   		!test_modes[vic].name[0]) {
>   			igt_warn("VIC %d is not supported\n", vic);