[PATCH i-g-t v2] tests/kms_bw: Restrict linear-tiling subtest to HDMI outputs and improve skip logic

[Naladala, Ramanaidu]

On 6/26/2025 3:49 PM, Pranay Samala wrote:
> This patch updates the linear-tiling subtest in kms_bw to only use HDMI
> connector. If the platform does not have enough HDMI outputs to match the
> number of requested pipes, the test is skipped with a clear message
> indicating the requirement and the number of HDMI outputs found.
>
> This avoids attempting to force modes on non-HDMI connectors and makes
> the test results clearer and more reliable on systems that don’t have
> enough HDMI connectivity.
>
> v2: Skip the test instead of continue (Rama)
>      Remove hdmi_count (Rama)
> Signed-off-by: Pranay Samala<pranay.samala at intel.com>
> ---
>   tests/kms_bw.c | 8 ++++++++
>   1 file changed, 8 insertions(+)
>
> diff --git a/tests/kms_bw.c b/tests/kms_bw.c
> index ca05695d3..3ebfdcebd 100644
> --- a/tests/kms_bw.c
> +++ b/tests/kms_bw.c
> @@ -195,6 +195,7 @@ static void run_test_linear_tiling(data_t *data, int pipe, const drmModeModeInfo
>   	int i = 0, num_pipes = 0;
>   	enum pipe p;
>   	int ret;
> +	uint32_t conn_type;
>   
>   	/* Cannot use igt_display_get_n_pipes() due to fused pipes on i915 where they do
>   	 * not give the numver of valid crtcs and always return IGT_MAX_PIPES */
> @@ -215,6 +216,13 @@ static void run_test_linear_tiling(data_t *data, int pipe, const drmModeModeInfo
>   			continue;
>   		}
>   
> +		conn_type = output->config.connector->connector_type;
> +
> +		igt_skip_on_f(!physical &&
> +			      conn_type != DRM_MODE_CONNECTOR_HDMIA &&
> +			      conn_type != DRM_MODE_CONNECTOR_HDMIB,
> +			      "Cannot force mode, connected output is not HDMI\n");
> +

LGTM,

Reviewed-by: Naladala Ramanaidu <ramanaidu.naladala at intel.com> 
<mailto:ramanaidu.naladala at intel.com>

>   		force_output_mode(data, output, mode);
>   
>   		igt_create_color_fb(display->drm_fd, mode->hdisplay,
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <https://lists.freedesktop.org/archives/igt-dev/attachments/20250626/de930230/attachment-0001.htm>