The armada atomic_check implementation uses the deprecated
drm_atomic_get_existing_crtc_state() helper.

This hook is called as part of the global atomic_check, thus before the
states are swapped. The existing state thus points to the new state, and
we can use drm_atomic_get_new_crtc_state() instead.

Reviewed-by: Ville Syrjälä <[email protected]>
Signed-off-by: Maxime Ripard <[email protected]>
---
To: Russell King <[email protected]>
---
 drivers/gpu/drm/armada/armada_plane.c | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

diff --git a/drivers/gpu/drm/armada/armada_plane.c 
b/drivers/gpu/drm/armada/armada_plane.c
index 
12a01a51d6fb7dc0e9264d4e4bf6dc4e1b472cb6..dae81ebafdb462b57478f7182e24fe7bc2915698
 100644
--- a/drivers/gpu/drm/armada/armada_plane.c
+++ b/drivers/gpu/drm/armada/armada_plane.c
@@ -92,11 +92,11 @@ int armada_drm_plane_atomic_check(struct drm_plane *plane,
        if (!new_plane_state->fb || WARN_ON(!new_plane_state->crtc)) {
                new_plane_state->visible = false;
                return 0;
        }
 
-       crtc_state = drm_atomic_get_existing_crtc_state(state, crtc);
+       crtc_state = drm_atomic_get_new_crtc_state(state, crtc);
        ret = drm_atomic_helper_check_plane_state(new_plane_state, crtc_state,
                                                  0,
                                                  INT_MAX, true, false);
        if (ret)
                return ret;

-- 
2.51.0

Reply via email to