On Tue, Jan 20, 2015 at 11:09:14PM +0100, Daniel Vetter wrote: [...] > diff --git a/include/drm/drm_crtc.h b/include/drm/drm_crtc.h > index 4d3f3b874dd6..8b626ff3d3bd 100644 > --- a/include/drm/drm_crtc.h > +++ b/include/drm/drm_crtc.h > @@ -929,6 +929,7 @@ struct drm_bridge { > * struct struct drm_atomic_state - the global state object for atomic updates > * @dev: parent DRM device > * @allow_modeset: allow full modeset > + * @allow_modeset: hint to enforce legacy cursor ioctl semantics This should be legacy_cursor_update. With that fixed: Reviewed-by: Thierry Reding