I've forgotten to remove that in my per-plane locking patch.

Reported-by: Rob Clark <robdclark at gmail.com>
Signed-off-by: Daniel Vetter <daniel.vetter at intel.com>
---
 drivers/gpu/drm/drm_atomic.c | 6 ------
 1 file changed, 6 deletions(-)

diff --git a/drivers/gpu/drm/drm_atomic.c b/drivers/gpu/drm/drm_atomic.c
index d3b46746b611..ba49b5ca822f 100644
--- a/drivers/gpu/drm/drm_atomic.c
+++ b/drivers/gpu/drm/drm_atomic.c
@@ -243,12 +243,6 @@ drm_atomic_get_plane_state(struct drm_atomic_state *state,
        if (state->plane_states[index])
                return state->plane_states[index];

-       /*
-        * TODO: We currently don't have per-plane mutexes. So instead of trying
-        * crazy tricks with deferring plane->crtc and hoping for the best just
-        * grab all crtc locks. Once we have per-plane locks we must update this
-        * to only take the plane mutex.
-        */
        ret = drm_modeset_lock(&plane->mutex, state->acquire_ctx);
        if (ret)
                return ERR_PTR(ret);
-- 
2.1.1

Reply via email to