Re: [PATCH i-g-t] tools: rename HDMI compliance utility

[Harry Wentland <harry.wentland@amd.com>]

On 2026-08-27 14:53, Swati Sharma wrote:
> Rename the AMD-specific HDMI compliance utility to a generic
> igt_hdmi_compliance binary and update the build rule.
> 
> Keep the existing AMD debugfs flow for pixel-format forcing. Skip
> that operation on Intel devices until a driver-supported interface
> for selecting the pixel encoding is available.
> 
> Assisted-by: GitHub Copilot:Claude Opus 4.6
> Signed-off-by: Swati Sharma <swati2.sharma@intel.com>

Reviewed-by: Harry Wentland <harry.wentland@amd.com>

Harry

> ---
>  tools/{amd_hdmi_compliance.c => igt_hdmi_compliance.c} | 9 ++++++---
>  tools/meson.build                                      | 2 +-
>  2 files changed, 7 insertions(+), 4 deletions(-)
>  rename tools/{amd_hdmi_compliance.c => igt_hdmi_compliance.c} (99%)
> 
> diff --git a/tools/amd_hdmi_compliance.c b/tools/igt_hdmi_compliance.c
> similarity index 99%
> rename from tools/amd_hdmi_compliance.c
> rename to tools/igt_hdmi_compliance.c
> index 2af4c0bab..ece0abd1d 100644
> --- a/tools/amd_hdmi_compliance.c
> +++ b/tools/igt_hdmi_compliance.c
> @@ -833,12 +833,15 @@ static void force_pixel_format(data_t *data, int pixel_format, int conn_id)
>  
>  	test_init(data, conn_id);
>  
> -	fd = igt_debugfs_connector_dir(data->fd, data->output->name, O_RDONLY);
> -	igt_assert(fd >= 0);
> -
>  	igt_info("Setting %d on connector id %d\n",
>  			pixel_format, data->output->config.connector->connector_id);
>  
> +	igt_skip_on_f(is_intel_device(data->fd),
> +			      "pixel-format forcing is not supported on Intel devices\n");
> +
> +	fd = igt_debugfs_connector_dir(data->fd, data->output->name, O_RDONLY);
> +	igt_assert(fd >= 0);
> +
>  	switch (pixel_format) {
>  	case 1:
>  		entry_name = "force_rgb_output";
> diff --git a/tools/meson.build b/tools/meson.build
> index fbc6c0a00..68c31da92 100644
> --- a/tools/meson.build
> +++ b/tools/meson.build
> @@ -183,7 +183,7 @@ executable('intel_gpu_top', 'intel_gpu_top.c',
>  	   dependencies : [lib_igt_perf,lib_igt_device_scan,lib_igt_drm_clients,lib_igt_drm_fdinfo,
>  	                   lib_igt_drm_stub, lib_igt_tools_stub, math])
>  
> -executable('amd_hdmi_compliance', 'amd_hdmi_compliance.c',
> +executable('igt_hdmi_compliance', 'igt_hdmi_compliance.c',
>  	   dependencies : [tool_deps],
>  	   install_rpath : bindir_rpathdir,
>  	   install : true)