Re: [PATCH i-g-t v1 05/25] lib/igt_kms: Add function to get valid pipe for specific output

[Jani Nikula <jani.nikula@intel.com>]

On Tue, 28 Apr 2026, Mohammed Bilal <mohammed.bilal@intel.com> wrote:
> From: Louis Chauvet <louis.chauvet@bootlin.com>
>
> Introduces a new function igt_get_pipe_for_output in igt_kms. The function
> is designed to retrieve a valid pipe for a specific output in a display.

Please avoid using pipes. Why would you need the pipe here?

The function isn't even used in the entire series.

>
> Signed-off-by: Louis Chauvet <louis.chauvet@bootlin.com>
> ---
>  lib/igt_kms.c | 22 ++++++++++++++++++++++
>  lib/igt_kms.h |  1 +
>  2 files changed, 23 insertions(+)
>
> diff --git a/lib/igt_kms.c b/lib/igt_kms.c
> index aa086e96c..73d65f70a 100644
> --- a/lib/igt_kms.c
> +++ b/lib/igt_kms.c
> @@ -8426,3 +8426,25 @@ uint32_t igt_get_connector_id_from_mst_path(int drm_fd, const void *mst_path)
>  
>  	return 0;
>  }
> +
> +/**
> + * igt_get_pipe_for_output:
> + * @display: display to fetch the pipes
> + * @output: output to use
> + *
> + * Get a valid pipe for a specific output. The return value is the pipe first valid pipe for a
> + * specific output.
> + */
> +enum pipe igt_get_pipe_for_output(igt_display_t *display,
> +				  igt_output_t *output)
> +{
> +        igt_crtc_t *crtc;
> +
> +        for_each_crtc(display, crtc) {
> +                if (igt_output_is_connected(output) &&
> +                     (output->config.valid_crtc_index_mask & (1 << crtc->crtc_index)))
> +                        return crtc->pipe;
> +        }
> +
> +        return PIPE_NONE;
> +}
> diff --git a/lib/igt_kms.h b/lib/igt_kms.h
> index bc921f4ff..70b7d1a58 100644
> --- a/lib/igt_kms.h
> +++ b/lib/igt_kms.h
> @@ -1305,5 +1305,6 @@ int igt_get_connected_connectors(int drm_fd, uint32_t **connector_ids);
>  drmModeConnectorPtr igt_get_connector_from_name(int drm_fd, const char *port_name);
>  uint32_t igt_get_connector_id_from_name(int drm_fd, const char *port_name);
>  uint32_t igt_get_connector_id_from_mst_path(int drm_fd, const void *mst_path);
> +enum pipe igt_get_pipe_for_output(igt_display_t *display, igt_output_t *output);
>  
>  #endif /* __IGT_KMS_H__ */

-- 
Jani Nikula, Intel