[PATCH v5 2/7] drm/atomic-helper: Add format-conversion state to shadow-plane state
Javier Martinez Canillas
javierm at redhat.com
Mon Oct 9 14:22:29 UTC 2023
Thomas Zimmermann <tzimmermann at suse.de> writes:
Hello Thomas,
> Store an instance of struct drm_format_conv_state in the shadow-plane
> state struct drm_shadow_plane_state. Many drivers with shadow planes
> use DRM's format helpers to copy or convert the framebuffer data to
> backing storage in the scanout buffer. The shadow plane provides the
> necessary state and manages the conversion's intermediate buffer memory.
>
I would argue this is one of the primary selling points for this series
(since Maxime was asking about the motivation). As you point out, many
drivers that use a shadow-buffer can also expose a different format than
the native one to user-space, so makes sense to have the buffer used for
conversion as a part of the shadow-plane state.
Just mentioning this because you are not including that in the paragraph
talking about the motivation for the patch series in its cover letter.
--
Best regards,
Javier Martinez Canillas
Core Platforms
Red Hat
More information about the dri-devel
mailing list