Re: [PATCH drm-misc-next v4 0/4] drm: verisilicon: convert drm_format to vs_format in atomic_check

Thomas Zimmermann
2026-03-31T06:27:14Z

Hi,

I only looked briefly over the update, but it seems good now.

Best regards
Thomas

Am 31.03.26 um 08:01 schrieb Icenowy Zheng:
> This patchset tries to move the process of converting drm_format to
> vs_format into plane's atomic_check callback (instead of the
> atomic_commit one, which the process currently sits) for more proper
> error handling (because atomic_commit cannot fail).
>
> In addition, because of the original flow of primary plane's
> atomic_check seems to be flawed, it's also updated before filling
> vs_format is added there.
>
> Icenowy Zheng (4):
> drm: verisilicon: make vs_format conversion function return int
> drm: verisilicon: subclass drm_plane_state
> drm: verisilicon: call atomic helper's plane state check even if no
> CRTC
> drm: verisilicon: fill plane's vs_format in atomic_check
>
> drivers/gpu/drm/verisilicon/vs_plane.c | 54 ++++++++++++++++++-
> drivers/gpu/drm/verisilicon/vs_plane.h | 18 ++++++-
> .../gpu/drm/verisilicon/vs_primary_plane.c | 48 ++++++++++-------
> 3 files changed, 98 insertions(+), 22 deletions(-)
>

--
--
Thomas Zimmermann
Graphics Driver Developer
SUSE Software Solutions Germany GmbH
Frankenstr. 146, 90461 Nürnberg, Germany, www.suse.com
GF: Jochen Jaser, Andrew McDonald, Werner Knoblich, (HRB 36809, AG Nürnberg)