Re: [PATCH 2/2] tools/amd_hdmi_compliance: Fix for HDMI Compliance Automation

Reviewed-by: Alex Hung <[email protected]>

On 7/21/26 13:56, Fangzhi Zuo wrote:
> From: Mark Broadworth <[email protected]>
> 
> It is to fix hand shaking issue between DUT and the box.
> Need to block until SIGNALRM triggered.
> 
> Signed-off-by: Mark Broadworth <[email protected]>
> Signed-off-by: Fangzhi Zuo <[email protected]>
> Tested-by: Mark Broadworth <[email protected]>
> ---
>   tools/amd_hdmi_compliance.c | 8 ++++++--
>   1 file changed, 6 insertions(+), 2 deletions(-)
> 
> diff --git a/tools/amd_hdmi_compliance.c b/tools/amd_hdmi_compliance.c
> index a60176c0e..2861b440e 100644
> --- a/tools/amd_hdmi_compliance.c
> +++ b/tools/amd_hdmi_compliance.c
> @@ -903,8 +903,12 @@ static void test_vic_mode(data_t *data, int vic, int conn_id)
>   	igt_plane_set_fb(data->primary, &afb);
>   	igt_display_commit_atomic(display, DRM_MODE_ATOMIC_ALLOW_MODESET, NULL);
>   
> -	igt_info("Press [Enter] to finish\n");
> -	wait_for_keypress();
> +	if (data->timeout_seconds > 0)
> +		pause(); /* block until SIGALRM fires (works when detached) */
> +	else {
> +		igt_info("Press [Enter] to finish\n");
> +		wait_for_keypress();
> +	}
>   
>   	test_fini(data);
>   }