Re: [PATCH 12/29] drm/armada: Drop struct_mutex from cursor paths

On Mon, Nov 23, 2015 at 10:32:45AM +0100, Daniel Vetter wrote:

@@ -957,11 +955,9 @@ static int armada_drm_crtc_cursor_move(struct drm_crtc *crtc, int x, int y) if (!dcrtc->variant->has_spu_adv_reg) return -EFAULT;

• mutex_lock(&dev->struct_mutex); dcrtc->cursor_x = x; dcrtc->cursor_y = y; ret = armada_drm_crtc_cursor_update(dcrtc, false);

• mutex_unlock(&dev->struct_mutex);

  return ret;

}

drivers/gpu/drm/armada/armada_crtc.c: In function 'armada_drm_crtc_cursor_move': drivers/gpu/drm/armada/armada_crtc.c:916:21: warning: unused variable 'dev' [-Wunused-variable]

Shall I add to your patch a change to remove that variable too? :)