== Series Details == Series: Add ops to intel_uc (rev3) URL : https://patchwork.freedesktop.org/series/70716/ State : warning == Summary == $ dim checkpatch origin/drm-tip bb5e4aea2877 drm/i915/uc: Add ops to intel_uc -:142: ERROR:COMPLEX_MACRO: Macros with complex values should be enclosed in parentheses #142: FILE: drivers/gpu/drm/i915/gt/uc/intel_uc.h:73: +#define intel_uc_ops_function(_NAME, _OPS, _TYPE, _RET) \ +static inline _TYPE intel_uc_##_NAME(struct intel_uc *uc) \ +{ \ + if (uc->ops->_OPS) \ + return uc->ops->_OPS(uc); \ + return _RET; \ +} -:142: CHECK:MACRO_ARG_REUSE: Macro argument reuse '_OPS' - possible side-effects? #142: FILE: drivers/gpu/drm/i915/gt/uc/intel_uc.h:73: +#define intel_uc_ops_function(_NAME, _OPS, _TYPE, _RET) \ +static inline _TYPE intel_uc_##_NAME(struct intel_uc *uc) \ +{ \ + if (uc->ops->_OPS) \ + return uc->ops->_OPS(uc); \ + return _RET; \ +} -:142: CHECK:MACRO_ARG_PRECEDENCE: Macro argument '_OPS' may be better as '(_OPS)' to avoid precedence issues #142: FILE: drivers/gpu/drm/i915/gt/uc/intel_uc.h:73: +#define intel_uc_ops_function(_NAME, _OPS, _TYPE, _RET) \ +static inline _TYPE intel_uc_##_NAME(struct intel_uc *uc) \ +{ \ + if (uc->ops->_OPS) \ + return uc->ops->_OPS(uc); \ + return _RET; \ +} -:149: CHECK:LINE_SPACING: Please use a blank line after function/struct/union/enum declarations #149: FILE: drivers/gpu/drm/i915/gt/uc/intel_uc.h:80: +} +intel_uc_ops_function(init_hw, init_hw, int, 0); -:150: ERROR:SPACING: space prohibited before that close parenthesis ')' #150: FILE: drivers/gpu/drm/i915/gt/uc/intel_uc.h:81: +intel_uc_ops_function(fini_hw, fini_hw, void, ); total: 2 errors, 0 warnings, 3 checks, 117 lines checked def878150533 drm/i915/uc: Add init_fw/fini_fw to to intel_uc_ops -:87: CHECK:LINE_SPACING: Please use a blank line after function/struct/union/enum declarations #87: FILE: drivers/gpu/drm/i915/gt/uc/intel_uc.h:80: } +intel_uc_ops_function(fetch_firmwares, init_fw, void, ); -:87: ERROR:SPACING: space prohibited before that close parenthesis ')' #87: FILE: drivers/gpu/drm/i915/gt/uc/intel_uc.h:80: +intel_uc_ops_function(fetch_firmwares, init_fw, void, ); -:88: ERROR:SPACING: space prohibited before that close parenthesis ')' #88: FILE: drivers/gpu/drm/i915/gt/uc/intel_uc.h:81: +intel_uc_ops_function(cleanup_firmwares, fini_fw, void, ); total: 2 errors, 0 warnings, 1 checks, 63 lines checked 71565950a620 drm/i915/uc: Add init/fini to to intel_uc_ops -:89: ERROR:SPACING: space prohibited before that close parenthesis ')' #89: FILE: drivers/gpu/drm/i915/gt/uc/intel_uc.h:82: +intel_uc_ops_function(init, init, void, ); -:90: ERROR:SPACING: space prohibited before that close parenthesis ')' #90: FILE: drivers/gpu/drm/i915/gt/uc/intel_uc.h:83: +intel_uc_ops_function(fini, fini, void, ); total: 2 errors, 0 warnings, 0 checks, 65 lines checked 0960b20d5549 drm/i915/uc: Add sanitize to to intel_uc_ops -:74: CHECK:LINE_SPACING: Please use a blank line after function/struct/union/enum declarations #74: FILE: drivers/gpu/drm/i915/gt/uc/intel_uc.h:80: } +intel_uc_ops_function(sanitize, sanitize, int, 0); total: 0 errors, 0 warnings, 1 checks, 49 lines checked _______________________________________________ Intel-gfx mailing list Intel-gfx@xxxxxxxxxxxxxxxxxxxxx https://lists.freedesktop.org/mailman/listinfo/intel-gfx
