Re: [PATCH i-g-t 13/25] lib/kms: Add display->format_mods_async

["Reddy Guddati, Santhosh" <santhosh.reddy.guddati@intel.com>]

On 12-05-2026 17:51, Ville Syrjala wrote:
> From: Ville Syrjälä <ville.syrjala@linux.intel.com>
> 
> Create a display wide union of the planes' async format+modifier
> lists, just as we already have done for the sync format+modifiers.

LGTM
Reviewed-by: Santhosh Reddy Guddati <santhosh.reddy.guddati@intel.com>

Regards,
Santhosh
> 
> Signed-off-by: Ville Syrjälä <ville.syrjala@linux.intel.com>
> ---
>   lib/igt_kms.c | 6 ++++++
>   lib/igt_kms.h | 1 +
>   2 files changed, 7 insertions(+)
> 
> diff --git a/lib/igt_kms.c b/lib/igt_kms.c
> index dc3486a795a4..ebc13063dfb0 100644
> --- a/lib/igt_kms.c
> +++ b/lib/igt_kms.c
> @@ -6385,9 +6385,15 @@ static const struct igt_format_mods *plane_format_mods(igt_plane_t *plane)
>   	return &plane->format_mods;
>   }
>   
> +static const struct igt_format_mods *plane_format_mods_async(igt_plane_t *plane)
> +{
> +	return &plane->format_mods_async;
> +}
> +
>   static void igt_fill_display_format_mod(igt_display_t *display)
>   {
>   	display_format_mods_fill(display, &display->format_mods, plane_format_mods);
> +	display_format_mods_fill(display, &display->format_mods_async, plane_format_mods_async);
>   }
>   
>   /**
> diff --git a/lib/igt_kms.h b/lib/igt_kms.h
> index 35923f94d7ff..e02c5dd25d16 100644
> --- a/lib/igt_kms.h
> +++ b/lib/igt_kms.h
> @@ -535,6 +535,7 @@ struct _igt_display {
>   	bool first_commit;
>   
>   	struct igt_format_mods format_mods;
> +	struct igt_format_mods format_mods_async;
>   };
>   
>   typedef struct {