== Series Details == Series: drm/i915/dsc: Fix the macro that calculates DSCC_/DSCA_ PPS reg address URL : https://patchwork.freedesktop.org/series/129486/ State : warning == Summary == Error: dim checkpatch failed d490bac8db7c drm/i915/dsc: Fix the macro that calculates DSCC_/DSCA_ PPS reg address -:7: WARNING:COMMIT_LOG_LONG_LINE: Prefer a maximum 75 chars per line (possible unwrapped commit description?) #7: Commit bd077259d0a9 ("drm/i915/vdsc: Add function to read any PPS register") defines -:43: WARNING:LONG_LINE: line length of 104 exceeds 100 columns #43: FILE: drivers/gpu/drm/i915/display/intel_vdsc_regs.h:54: +#define DSCA_PPS(pps) _MMIO(_DSCA_PPS_0 + ((pps < 12) ? (pps):(pps + 12)) * 4) -:43: CHECK:SPACING: spaces required around that ':' (ctx:VxV) #43: FILE: drivers/gpu/drm/i915/display/intel_vdsc_regs.h:54: +#define DSCA_PPS(pps) _MMIO(_DSCA_PPS_0 + ((pps < 12) ? (pps):(pps + 12)) * 4) ^ -:43: CHECK:MACRO_ARG_REUSE: Macro argument reuse 'pps' - possible side-effects? #43: FILE: drivers/gpu/drm/i915/display/intel_vdsc_regs.h:54: +#define DSCA_PPS(pps) _MMIO(_DSCA_PPS_0 + ((pps < 12) ? (pps):(pps + 12)) * 4) -:43: CHECK:MACRO_ARG_PRECEDENCE: Macro argument 'pps' may be better as '(pps)' to avoid precedence issues #43: FILE: drivers/gpu/drm/i915/display/intel_vdsc_regs.h:54: +#define DSCA_PPS(pps) _MMIO(_DSCA_PPS_0 + ((pps < 12) ? (pps):(pps + 12)) * 4) -:44: WARNING:LONG_LINE: line length of 104 exceeds 100 columns #44: FILE: drivers/gpu/drm/i915/display/intel_vdsc_regs.h:55: +#define DSCC_PPS(pps) _MMIO(_DSCC_PPS_0 + ((pps < 12) ? (pps):(pps + 12)) * 4) -:44: CHECK:SPACING: spaces required around that ':' (ctx:VxV) #44: FILE: drivers/gpu/drm/i915/display/intel_vdsc_regs.h:55: +#define DSCC_PPS(pps) _MMIO(_DSCC_PPS_0 + ((pps < 12) ? (pps):(pps + 12)) * 4) ^ -:44: CHECK:MACRO_ARG_REUSE: Macro argument reuse 'pps' - possible side-effects? #44: FILE: drivers/gpu/drm/i915/display/intel_vdsc_regs.h:55: +#define DSCC_PPS(pps) _MMIO(_DSCC_PPS_0 + ((pps < 12) ? (pps):(pps + 12)) * 4) -:44: CHECK:MACRO_ARG_PRECEDENCE: Macro argument 'pps' may be better as '(pps)' to avoid precedence issues #44: FILE: drivers/gpu/drm/i915/display/intel_vdsc_regs.h:55: +#define DSCC_PPS(pps) _MMIO(_DSCC_PPS_0 + ((pps < 12) ? (pps):(pps + 12)) * 4) total: 0 errors, 3 warnings, 6 checks, 10 lines checked
