== Series Details ==

Series: series starting with drm/atomic: document and enforce rules around 
"spurious" EBUSY (rev2)
URL   : https://patchwork.freedesktop.org/series/82023/
State : warning

== Summary ==

$ dim checkpatch origin/drm-tip
2ae63b2182d7 drm/atomic: document and enforce rules around "spurious" EBUSY
-:47: WARNING:COMMIT_LOG_LONG_LINE: Possible unwrapped commit description 
(prefer a maximum 75 chars per line)
#47: 
References: 
https://lists.freedesktop.org/archives/dri-devel/2018-July/182281.html

-:74: WARNING:UNSPECIFIED_INT: Prefer 'unsigned int' to bare use of 'unsigned'
#74: FILE: drivers/gpu/drm/drm_atomic.c:1269:
+       unsigned requested_crtc = 0;

-:75: WARNING:UNSPECIFIED_INT: Prefer 'unsigned int' to bare use of 'unsigned'
#75: FILE: drivers/gpu/drm/drm_atomic.c:1270:
+       unsigned affected_crtc = 0;

-:112: WARNING:NO_AUTHOR_SIGN_OFF: Missing Signed-off-by: line by nominal patch 
author 'Daniel Vetter <daniel.vet...@ffwll.ch>'

total: 0 errors, 4 warnings, 0 checks, 51 lines checked
126d6b5ccc51 drm/atomic: debug output for EBUSY
-:52: CHECK:PARENTHESIS_ALIGNMENT: Alignment should match open parenthesis
#52: FILE: drivers/gpu/drm/drm_atomic_helper.c:1741:
+               DRM_DEBUG_ATOMIC("[PLANE:%d:%s] inflight previous commit 
preventing async commit\n",
+                       plane->base.id, plane->name);

-:63: CHECK:PARENTHESIS_ALIGNMENT: Alignment should match open parenthesis
#63: FILE: drivers/gpu/drm/drm_atomic_helper.c:1962:
+                               DRM_DEBUG_ATOMIC("[CRTC:%d:%s] busy with a 
previous commit\n",
+                                       crtc->base.id, crtc->name);

-:75: CHECK:PARENTHESIS_ALIGNMENT: Alignment should match open parenthesis
#75: FILE: drivers/gpu/drm/drm_atomic_helper.c:2140:
+                       DRM_DEBUG_ATOMIC("[CONNECTOR:%d:%s] busy with a 
previous commit\n",
+                               conn->base.id, conn->name);

-:89: CHECK:PARENTHESIS_ALIGNMENT: Alignment should match open parenthesis
#89: FILE: drivers/gpu/drm/drm_atomic_helper.c:2159:
+                       DRM_DEBUG_ATOMIC("[PLANE:%d:%s] busy with a previous 
commit\n",
+                               plane->base.id, plane->name);

-:95: WARNING:NO_AUTHOR_SIGN_OFF: Missing Signed-off-by: line by nominal patch 
author 'Daniel Vetter <daniel.vet...@ffwll.ch>'

total: 0 errors, 1 warnings, 4 checks, 63 lines checked


_______________________________________________
Intel-gfx mailing list
Intel-gfx@lists.freedesktop.org
https://lists.freedesktop.org/mailman/listinfo/intel-gfx

Reply via email to