Re: [PATCH i-g-t, v6 1/4] tests/intel/kms_joiner_helper: Add igt_get_joined_pipes_name()

[Karthik B S <karthik.b.s@intel.com>]

On 5/1/2026 12:39 AM, Swati Sharma wrote:
> Add function to transform the enum into a string.
>
> v2: -Improved func() doc (Jeevan)
> v3: -Return names without '-' dash prefix (Santhosh)
> v4: -Move from lib/igt_kms to tests/intel/kms_joiner_helper (Jani)
>
> Signed-off-by: Swati Sharma <swati2.sharma@intel.com>
Reviewed-by: Karthik B S <karthik.b.s@intel.com>
> ---
>   tests/intel/kms_joiner_helper.c | 22 ++++++++++++++++++++++
>   tests/intel/kms_joiner_helper.h |  1 +
>   2 files changed, 23 insertions(+)
>
> diff --git a/tests/intel/kms_joiner_helper.c b/tests/intel/kms_joiner_helper.c
> index e24d7ce94..0aa409ac2 100644
> --- a/tests/intel/kms_joiner_helper.c
> +++ b/tests/intel/kms_joiner_helper.c
> @@ -183,3 +183,25 @@ bool igt_assign_pipes_for_outputs(int drm_fd,
>   	}
>   	return true;
>   }
> +
> +/**
> + * igt_get_joined_pipes_name:
> + * @val: joined_pipes enum value
> + *
> + * Returns: A readable string for the given joined_pipes enum value.
> + */
> +const char *igt_get_joined_pipes_name(enum joined_pipes val)
> +{
> +	switch (val) {
> +	case JOINED_PIPES_DEFAULT:
> +		return "";
> +	case JOINED_PIPES_NONE:
> +		return "none";
> +	case JOINED_PIPES_BIG_JOINER:
> +		return "bigjoiner";
> +	case JOINED_PIPES_ULTRA_JOINER:
> +		return "ultrajoiner";
> +	default:
> +		igt_assert(false);
> +	}
> +}
> diff --git a/tests/intel/kms_joiner_helper.h b/tests/intel/kms_joiner_helper.h
> index 6d21e7eb0..1ea871a3b 100644
> --- a/tests/intel/kms_joiner_helper.h
> +++ b/tests/intel/kms_joiner_helper.h
> @@ -17,6 +17,7 @@ bool igt_assign_pipes_for_outputs(int drm_fd,
>   				  uint32_t *used_pipes_mask,
>   				  uint32_t master_pipes_mask,
>   				  uint32_t valid_pipes_mask);
> +const char *igt_get_joined_pipes_name(enum joined_pipes val);
>   
>   enum force_joiner_mode {
>   	FORCE_JOINER_ENABLE = 0,