== Series Details == Series: drm_managed, leftovers URL : https://patchwork.freedesktop.org/series/81371/ State : warning == Summary == $ dim checkpatch origin/drm-tip 7e79337308df drm/armada: Use devm_drm_dev_alloc -:68: WARNING:NO_AUTHOR_SIGN_OFF: Missing Signed-off-by: line by nominal patch author 'Daniel Vetter <daniel.vetter@xxxxxxxx>' total: 0 errors, 1 warnings, 0 checks, 45 lines checked 26f00c197367 drm/armada: Don't use drm_device->dev_private -:165: WARNING:NO_AUTHOR_SIGN_OFF: Missing Signed-off-by: line by nominal patch author 'Daniel Vetter <daniel.vetter@xxxxxxxx>' total: 0 errors, 1 warnings, 0 checks, 112 lines checked 987c9338ca7b drm/aspeed: Use managed drmm_mode_config_cleanup -:64: WARNING:NO_AUTHOR_SIGN_OFF: Missing Signed-off-by: line by nominal patch author 'Daniel Vetter <daniel.vetter@xxxxxxxx>' total: 0 errors, 1 warnings, 0 checks, 40 lines checked 86684c96f0be drm/vgem: Use devm_drm_dev_alloc -:13: WARNING:COMMIT_LOG_LONG_LINE: Possible unwrapped commit description (prefer a maximum 75 chars per line) #13: This also somewhat untangles the load code, since the drm and platform device total: 0 errors, 1 warnings, 0 checks, 88 lines checked 07047c4d2081 drm/vkms: Use devm_drm_dev_alloc -:13: WARNING:COMMIT_LOG_LONG_LINE: Possible unwrapped commit description (prefer a maximum 75 chars per line) #13: This also somewhat untangles the load code, since the drm and platform device -:136: WARNING:NO_AUTHOR_SIGN_OFF: Missing Signed-off-by: line by nominal patch author 'Daniel Vetter <daniel.vetter@xxxxxxxx>' total: 0 errors, 2 warnings, 0 checks, 104 lines checked d837a22748b5 drm/xlnx: Use devm_drm_dev_alloc -:63: WARNING:NO_AUTHOR_SIGN_OFF: Missing Signed-off-by: line by nominal patch author 'Daniel Vetter <daniel.vetter@xxxxxxxx>' total: 0 errors, 1 warnings, 0 checks, 39 lines checked 796ee493a8a3 drm/i915/selftest: Create mock_destroy_device -:184: WARNING:NO_AUTHOR_SIGN_OFF: Missing Signed-off-by: line by nominal patch author 'Daniel Vetter <daniel.vetter@xxxxxxxx>' total: 0 errors, 1 warnings, 0 checks, 106 lines checked 124bff980d17 drm/i915/selftests: align more to real device lifetimes -:92: WARNING:NO_AUTHOR_SIGN_OFF: Missing Signed-off-by: line by nominal patch author 'Daniel Vetter <daniel.vetter@xxxxxxxx>' total: 0 errors, 1 warnings, 0 checks, 66 lines checked 9fa6eafa5dc7 drm/dev: Remove drm_dev_init c15b464eaef1 drm/arc: Switch to devm_drm_dev_alloc -:103: WARNING:NO_AUTHOR_SIGN_OFF: Missing Signed-off-by: line by nominal patch author 'Daniel Vetter <daniel.vetter@xxxxxxxx>' total: 0 errors, 1 warnings, 0 checks, 75 lines checked 746009dbe330 drm/arc: Stop using drm_device->dev_private -:69: WARNING:NO_AUTHOR_SIGN_OFF: Missing Signed-off-by: line by nominal patch author 'Daniel Vetter <daniel.vetter@xxxxxxxx>' total: 0 errors, 1 warnings, 0 checks, 40 lines checked fe06aeaae2bd drm/arc: Delete arcpgu_priv->fb be543cb2845f drm/arc: Embedded a drm_simple_display_pipe -:135: WARNING:NO_AUTHOR_SIGN_OFF: Missing Signed-off-by: line by nominal patch author 'Daniel Vetter <daniel.vetter@xxxxxxxx>' total: 0 errors, 1 warnings, 0 checks, 93 lines checked a17149e448e8 drm/arc: Embedd a drm_connector for sim case -:64: WARNING:NO_AUTHOR_SIGN_OFF: Missing Signed-off-by: line by nominal patch author 'Daniel Vetter <daniel.vetter@xxxxxxxx>' total: 0 errors, 1 warnings, 0 checks, 39 lines checked ba16b70d40cc drm/arc: Drop surplus connector registration 5c8bfba805ad drm/arc: Use drmm_mode_config_cleanup -:122: WARNING:NO_AUTHOR_SIGN_OFF: Missing Signed-off-by: line by nominal patch author 'Daniel Vetter <daniel.vetter@xxxxxxxx>' total: 0 errors, 1 warnings, 0 checks, 83 lines checked d11e78dcb76b drm/arc: Align with simple pipe helpers -:60: WARNING:NO_AUTHOR_SIGN_OFF: Missing Signed-off-by: line by nominal patch author 'Daniel Vetter <daniel.vetter@xxxxxxxx>' total: 0 errors, 1 warnings, 0 checks, 35 lines checked 2431d7d612ac drm/arc: Convert to drm_simple_kms_pipe_helper -:294: WARNING:NO_AUTHOR_SIGN_OFF: Missing Signed-off-by: line by nominal patch author 'Daniel Vetter <daniel.vetter@xxxxxxxx>' total: 0 errors, 1 warnings, 0 checks, 242 lines checked 5bb83c222569 drm/arc: Drop crtc check in arc_pgu_update -:28: WARNING:NO_AUTHOR_SIGN_OFF: Missing Signed-off-by: line by nominal patch author 'Daniel Vetter <daniel.vetter@xxxxxxxx>' total: 0 errors, 1 warnings, 0 checks, 8 lines checked 5e3a85368a5d drm/arc: Inline arcpgu_crtc.c -:43: WARNING:FILE_PATH_CHANGES: added, moved or deleted file(s), does MAINTAINERS need updating? #43: deleted file mode 100644 -:235: CHECK:PREFER_KERNEL_TYPES: Prefer kernel type 'u32' over 'uint32_t' #235: FILE: drivers/gpu/drm/arc/arcpgu_drv.c:39: + uint32_t pixel_format = fb->format->format; -:264: CHECK:UNNECESSARY_PARENTHESES: Unnecessary parentheses around 'rate > 0' #264: FILE: drivers/gpu/drm/arc/arcpgu_drv.c:68: + if ((max(rate, clk_rate) - min(rate, clk_rate) < diff) && (rate > 0)) -:331: CHECK:PARENTHESIS_ALIGNMENT: Alignment should match open parenthesis #331: FILE: drivers/gpu/drm/arc/arcpgu_drv.c:135: + arc_pgu_write(arcpgu, ARCPGU_REG_CTRL, + arc_pgu_read(arcpgu, ARCPGU_REG_CTRL) & -:373: WARNING:NO_AUTHOR_SIGN_OFF: Missing Signed-off-by: line by nominal patch author 'Daniel Vetter <daniel.vetter@xxxxxxxx>' total: 0 errors, 2 warnings, 3 checks, 170 lines checked 8b0c4a52a621 drm/arc: Inline arcpgu_drm_hdmi_init -:58: WARNING:FILE_PATH_CHANGES: added, moved or deleted file(s), does MAINTAINERS need updating? #58: deleted file mode 100644 -:89: WARNING:NO_AUTHOR_SIGN_OFF: Missing Signed-off-by: line by nominal patch author 'Daniel Vetter <daniel.vetter@xxxxxxxx>' total: 0 errors, 2 warnings, 0 checks, 29 lines checked a561f0af936b drm/arc: Inline remaining files -:26: WARNING:FILE_PATH_CHANGES: added, moved or deleted file(s), does MAINTAINERS need updating? #26: deleted file mode 100644 -:171: CHECK:PARENTHESIS_ALIGNMENT: Alignment should match open parenthesis #171: FILE: drivers/gpu/drm/arc/arcpgu_drv.c:111: + ret = drm_connector_init(drm, connector, &arcpgu_drm_connector_funcs, + DRM_MODE_CONNECTOR_VIRTUAL); -:304: WARNING:NO_AUTHOR_SIGN_OFF: Missing Signed-off-by: line by nominal patch author 'Daniel Vetter <daniel.vetter@xxxxxxxx>' total: 0 errors, 2 warnings, 1 checks, 117 lines checked e22fc3f4bb1b drm/arc: Initialize sim connector before display pipe -:111: WARNING:NO_AUTHOR_SIGN_OFF: Missing Signed-off-by: line by nominal patch author 'Daniel Vetter <daniel.vetter@xxxxxxxx>' total: 0 errors, 1 warnings, 0 checks, 87 lines checked 686a533fcde6 drm/arc: Move to drm/tiny -:53: WARNING:FILE_PATH_CHANGES: added, moved or deleted file(s), does MAINTAINERS need updating? #53: deleted file mode 100644 -:84: WARNING:CONFIG_DESCRIPTION: please write a paragraph that describes the config symbol fully #84: FILE: drivers/gpu/drm/tiny/Kconfig:3: +config DRM_ARCPGU -:111: WARNING:NO_AUTHOR_SIGN_OFF: Missing Signed-off-by: line by nominal patch author 'Daniel Vetter <daniel.vetter@xxxxxxxx>' total: 0 errors, 3 warnings, 0 checks, 44 lines checked _______________________________________________ Intel-gfx mailing list Intel-gfx@xxxxxxxxxxxxxxxxxxxxx https://lists.freedesktop.org/mailman/listinfo/intel-gfx