Re: [PATCH i-g-t 14/25] lib/kms: Fill plane->format_mods_async for planes without IN_FORMATS

["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>
> 
> Fill plane->format_mods_async for drivers that don't provide
> the IN_FORMATS blob. Since we don't know any better we simply
> make the assumption that all formats are async flip capable.
> 
> This should make it possible for tests to rely on
> plane->format_mods_async without having to do additonal async
> flip capability checks.
> 
> And if some driver can't deal with this then the proper solution
> is to have it expose IN_FORMATS and IN_FORMATS_ASYNC.
> 
> Signed-off-by: Ville Syrjälä <ville.syrjala@linux.intel.com>
LGTM
Reviewed-by: Santhosh Reddy Guddati <santhosh.reddy.guddati@intel.com>

> ---
>   lib/igt_kms.c | 3 +++
>   1 file changed, 3 insertions(+)
> 
> diff --git a/lib/igt_kms.c b/lib/igt_kms.c
> index ebc13063dfb0..b81be73978a1 100644
> --- a/lib/igt_kms.c
> +++ b/lib/igt_kms.c
> @@ -6246,6 +6246,9 @@ static void igt_fill_plane_format_mod(igt_display_t *display, igt_plane_t *plane
>   	if (!igt_plane_has_prop(plane, IGT_PLANE_IN_FORMATS)) {
>   		fill_plane_default_format_mods(plane, &plane->format_mods);
>   
> +		if (igt_has_drm_cap(display->drm_fd, DRM_CAP_ASYNC_PAGE_FLIP))
> +			fill_plane_default_format_mods(plane, &plane->format_mods_async);
> +
>   		return;
>   	}
>