drm/ast: Call drm_atomic_helper_check_plane_state() unconditionally
authorThomas Zimmermann <tzimmermann@suse.de>
Thu, 13 Oct 2022 11:29:17 +0000 (13:29 +0200)
committerThomas Zimmermann <tzimmermann@suse.de>
Sun, 16 Oct 2022 09:15:45 +0000 (11:15 +0200)
commit0432a5044bb39fc542516cdec58e5041afad486c
tree02fe19054ceed81ebf1b8d743f23090558aeb3e1
parent1fe182154984fa7942f8aafc268e9922e553cb13
drm/ast: Call drm_atomic_helper_check_plane_state() unconditionally

Always call drm_atomic_helper_check_plane_state() in each plane's
atomic_check function. At the minimum, it needs to set or clear the
plane state's 'visible' field. Otherwise the plane-state handling
is bogus and would keep updating planes that have been disabled.

While at it, also warn if the primary plane has been enabled, but is
not visible. This cannot legally happen as the plane always covers
the entire screen.

Signed-off-by: Thomas Zimmermann <tzimmermann@suse.de>
Reviewed-by: Jocelyn Falempe <jfalempe@redhat.com>
Tested-by: Jocelyn Falempe <jfalempe@redhat.com>
Link: https://patchwork.freedesktop.org/patch/msgid/20221013112923.769-3-tzimmermann@suse.de
drivers/gpu/drm/ast/ast_mode.c