== Series Details == Series: series starting with [CI,1/2] drm/i915/step: Add macro magic for handling steps URL : https://patchwork.freedesktop.org/series/92735/ State : warning == Summary == $ dim checkpatch origin/drm-tip 29663fa4daa2 drm/i915/step: Add macro magic for handling steps -:24: ERROR:MULTISTATEMENT_MACRO_USE_DO_WHILE: Macros with multiple statements should be enclosed in a do - while loop #24: FILE: drivers/gpu/drm/i915/intel_step.c:186: +#define STEP_NAME_CASE(name) \ + case STEP_##name: \ + return #name; -:24: WARNING:TRAILING_SEMICOLON: macros should not use a trailing semicolon #24: FILE: drivers/gpu/drm/i915/intel_step.c:186: +#define STEP_NAME_CASE(name) \ + case STEP_##name: \ + return #name; -:47: ERROR:COMPLEX_MACRO: Macros with complex values should be enclosed in parentheses #47: FILE: drivers/gpu/drm/i915/intel_step.h:20: +#define STEP_NAME_LIST(func) \ + func(A0) \ + func(A1) \ + func(A2) \ + func(B0) \ + func(B1) \ + func(B2) \ + func(C0) \ + func(C1) \ + func(D0) \ + func(D1) \ + func(E0) \ + func(F0) \ + func(G0) \ + func(H0) \ + func(I0) \ + func(I1) \ + func(J0) -:47: CHECK:MACRO_ARG_REUSE: Macro argument reuse 'func' - possible side-effects? #47: FILE: drivers/gpu/drm/i915/intel_step.h:20: +#define STEP_NAME_LIST(func) \ + func(A0) \ + func(A1) \ + func(A2) \ + func(B0) \ + func(B1) \ + func(B2) \ + func(C0) \ + func(C1) \ + func(D0) \ + func(D1) \ + func(E0) \ + func(F0) \ + func(G0) \ + func(H0) \ + func(I0) \ + func(I1) \ + func(J0) total: 2 errors, 1 warnings, 1 checks, 70 lines checked e9092fbd9b75 drm/i915/dmc: Change intel_get_stepping_info() _______________________________________________ Intel-gfx mailing list Intel-gfx@xxxxxxxxxxxxxxxxxxxxx https://lists.freedesktop.org/mailman/listinfo/intel-gfx
