== Series Details == Series: Allow using dyndbg to replace drm_debug_enabled URL : https://patchwork.freedesktop.org/series/92438/ State : warning == Summary == $ dim checkpatch origin/drm-tip 05ade4d62336 drm_print.h: rewrap __DRM_DEFINE_DBG_RATELIMITED macro -:32: CHECK:LOGICAL_CONTINUATIONS: Logical continuations should be on the previous line #32: FILE: include/drm/drm_print.h:535: + if (drm_debug_enabled(DRM_UT_ ## category) \ + && __ratelimit(&rs_)) \ -:45: ERROR:MISSING_SIGN_OFF: Missing Signed-off-by: line(s) total: 1 errors, 0 warnings, 1 checks, 32 lines checked b347ee940ec2 drm: fixup comment spelling -:9: WARNING:COMMIT_MESSAGE: Missing commit description - Add an appropriate one -:20: ERROR:MISSING_SIGN_OFF: Missing Signed-off-by: line(s) total: 1 errors, 1 warnings, 0 checks, 8 lines checked c3d19ae5f037 drm: RFC add choice to use dynamic debug in drm-debug -:69: WARNING:COMMIT_LOG_LONG_LINE: Possible unwrapped commit description (prefer a maximum 75 chars per line) #69: commit 7911902129a8 ("drm/print: Handle potentially NULL drm_devices in drm_dbg_*) -:69: ERROR:GIT_COMMIT_ID: Please use git commit description style 'commit <12+ chars of sha1> ("<title line>")' - ie: 'commit 7911902129a8 ("drm/print: Handle potentially NULL drm_devices in drm_dbg_*")' #69: commit 7911902129a8 ("drm/print: Handle potentially NULL drm_devices in drm_dbg_*) -:141: CHECK:BRACES: Blank lines aren't necessary after an open brace '{' #141: FILE: drivers/gpu/drm/drm_print.c:93: + for_each_set_bit(bitpos, &changes, ARRAY_SIZE(format_class_prefixes)) { + -:164: CHECK:LINE_SPACING: Please use a blank line after function/struct/union/enum declarations #164: FILE: drivers/gpu/drm/drm_print.c:116: +} +static const struct kernel_param_ops param_ops_debug = { -:181: CHECK:PARENTHESIS_ALIGNMENT: Alignment should match open parenthesis #181: FILE: drivers/gpu/drm/drm_print.c:327: +void _drm_dev_dbg(const struct device *dev, enum drm_debug_category category, const char *format, ...) -:275: CHECK:PARENTHESIS_ALIGNMENT: Alignment should match open parenthesis #275: FILE: include/drm/drm_print.h:384: +void _drm_dev_dbg(const struct device *dev, enum drm_debug_category category, const char *format, ...); -:354: CHECK:LINE_SPACING: Please don't use multiple blank lines #354: FILE: include/drm/drm_print.h:549: + -:409: ERROR:MISSING_SIGN_OFF: Missing Signed-off-by: line(s) total: 2 errors, 1 warnings, 5 checks, 313 lines checked 64eb8b1471cb i915: map gvt pr_debug categories to bits in parameters/debug_gvt -:68: CHECK:LINE_SPACING: Please use a blank line after function/struct/union/enum declarations #68: FILE: drivers/gpu/drm/i915/i915_params.c:284: +}; +#define NUM_CLASSES ARRAY_SIZE(format_prefix_classes) -:91: CHECK:BRACES: Blank lines aren't necessary after an open brace '{' #91: FILE: drivers/gpu/drm/i915/i915_params.c:307: + for_each_set_bit(bitpos, &changes, NUM_CLASSES) { + -:104: CHECK:LINE_SPACING: Please use a blank line after function/struct/union/enum declarations #104: FILE: drivers/gpu/drm/i915/i915_params.c:320: +} +static int param_get_dyndbg(char *buffer, const struct kernel_param *kp) -:109: CHECK:LINE_SPACING: Please use a blank line after function/struct/union/enum declarations #109: FILE: drivers/gpu/drm/i915/i915_params.c:325: +} +static const struct kernel_param_ops param_ops_dyndbg = { -:127: ERROR:MISSING_SIGN_OFF: Missing Signed-off-by: line(s) total: 1 errors, 0 warnings, 4 checks, 86 lines checked _______________________________________________ Intel-gfx mailing list Intel-gfx@xxxxxxxxxxxxxxxxxxxxx https://lists.freedesktop.org/mailman/listinfo/intel-gfx
