[Intel-gfx] ✗ Fi.CI.CHECKPATCH: warning for drm/ioctl: Ditch DRM_UNLOCKED except for the legacy vblank ioctl

Patchwork patchwork at emeril.freedesktop.org
Wed May 29 14:51:42 UTC 2019


== Series Details ==

Series: drm/ioctl: Ditch DRM_UNLOCKED except for the legacy vblank ioctl
URL   : https://patchwork.freedesktop.org/series/61299/
State : warning

== Summary ==

$ dim checkpatch origin/drm-tip
82c3759de56a drm/ioctl: Ditch DRM_UNLOCKED except for the legacy vblank ioctl
-:11: ERROR:GIT_COMMIT_ID: Please use git commit description style 'commit <12+ chars of sha1> ("<title line>")' - ie: 'commit ea487835e887 ("drm: Enforce unlocked ioctl operation for kms driver ioctls")'
#11: 
commit ea487835e8876abf7ad909636e308c801a2bcda6

-:21: ERROR:GIT_COMMIT_ID: Please use git commit description style 'commit <12+ chars of sha1> ("<title line>")' - ie: 'commit ed8b67040965 ("drm: convert drm_ioctl to unlocked_ioctl")'
#21: 
commit ed8b67040965e4fe695db333d5914e18ea5f146f

-:39: ERROR:GIT_COMMIT_ID: Please use git commit description style 'commit <12+ chars of sha1> ("<title line>")' - ie: 'commit 8f4ff2b06afc ("drm: do not sleep on vblank while holding a mutex")'
#39: 
commit 8f4ff2b06afcd6f151868474a432c603057eaf56

-:49: ERROR:GIT_COMMIT_ID: Please use git commit description style 'commit <12+ chars of sha1> ("<title line>")' - ie: 'commit fdd5b877e9eb ("drm: Enforce BKL-less ioctls for modern drivers")'
#49: 
commit fdd5b877e9ebc2029e1373b4a3cd057329a9ab7a

-:164: WARNING:LONG_LINE: line over 100 characters
#164: FILE: drivers/gpu/drm/drm_ioctl.c:650:
+	DRM_IOCTL_DEF(DRM_IOCTL_PRIME_HANDLE_TO_FD, drm_prime_handle_to_fd_ioctl, DRM_AUTH|DRM_RENDER_ALLOW),

-:164: CHECK:SPACING: spaces preferred around that '|' (ctx:VxV)
#164: FILE: drivers/gpu/drm/drm_ioctl.c:650:
+	DRM_IOCTL_DEF(DRM_IOCTL_PRIME_HANDLE_TO_FD, drm_prime_handle_to_fd_ioctl, DRM_AUTH|DRM_RENDER_ALLOW),
 	                                                                                  ^

-:165: WARNING:LONG_LINE: line over 100 characters
#165: FILE: drivers/gpu/drm/drm_ioctl.c:651:
+	DRM_IOCTL_DEF(DRM_IOCTL_PRIME_FD_TO_HANDLE, drm_prime_fd_to_handle_ioctl, DRM_AUTH|DRM_RENDER_ALLOW),

-:165: CHECK:SPACING: spaces preferred around that '|' (ctx:VxV)
#165: FILE: drivers/gpu/drm/drm_ioctl.c:651:
+	DRM_IOCTL_DEF(DRM_IOCTL_PRIME_FD_TO_HANDLE, drm_prime_fd_to_handle_ioctl, DRM_AUTH|DRM_RENDER_ALLOW),
 	                                                                                  ^

-:268: WARNING:NO_AUTHOR_SIGN_OFF: Missing Signed-off-by: line by nominal patch author 'Daniel Vetter <daniel.vetter at ffwll.ch>'

total: 4 errors, 3 warnings, 2 checks, 191 lines checked



More information about the Intel-gfx mailing list