The drm_atomic_connector_check() function 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.

Signed-off-by: Maxime Ripard <mrip...@kernel.org>
---
 drivers/gpu/drm/drm_atomic.c | 4 ++--
 1 file changed, 2 insertions(+), 2 deletions(-)

diff --git a/drivers/gpu/drm/drm_atomic.c b/drivers/gpu/drm/drm_atomic.c
index 
1147fa890a03ef2c3283972c816796d8de6307c9..6d92f13d489f299aea5cd14f900ae766588462da
 100644
--- a/drivers/gpu/drm/drm_atomic.c
+++ b/drivers/gpu/drm/drm_atomic.c
@@ -478,12 +478,12 @@ static int drm_atomic_connector_check(struct 
drm_connector *connector,
                               connector->base.id, connector->name);
                return -EINVAL;
        }
 
        if (state->crtc)
-               crtc_state = drm_atomic_get_existing_crtc_state(state->state,
-                                                               state->crtc);
+               crtc_state = drm_atomic_get_new_crtc_state(state->state,
+                                                          state->crtc);
 
        if (writeback_job->fb && !crtc_state->active) {
                drm_dbg_atomic(connector->dev,
                               "[CONNECTOR:%d:%s] has framebuffer, but 
[CRTC:%d] is off\n",
                               connector->base.id, connector->name,

-- 
2.50.1

Reply via email to