== Series Details == Series: mtl: add support for pmdemand (rev15) URL : https://patchwork.freedesktop.org/series/116949/ State : warning == Summary == Error: dim checkpatch failed 595a7ad922f2 drm/i915: fix the derating percentage for MTL 6c642218f0cc drm/i915: update the QGV point frequency calculations f73e6f3d3a89 drm/i915: store the peak bw per QGV point a12a6f28079a drm/i915: extract intel_bw_check_qgv_points() e1dbf9927a1c drm/i915: modify max_bw to return index to intel_bw_info 49909f81b052 drm/i915/mtl: find the best QGV point for the SAGV configuration 11ac7a3d546d drm/i915/mtl: Add support for PM DEMAND Traceback (most recent call last): File "scripts/spdxcheck.py", line 6, in <module> from ply import lex, yacc ModuleNotFoundError: No module named 'ply' Traceback (most recent call last): File "scripts/spdxcheck.py", line 6, in <module> from ply import lex, yacc ModuleNotFoundError: No module named 'ply' -:425: WARNING:FILE_PATH_CHANGES: added, moved or deleted file(s), does MAINTAINERS need updating? #425: new file mode 100644 -:920: CHECK:MACRO_ARG_REUSE: Macro argument reuse 'reg' - possible side-effects? #920: FILE: drivers/gpu/drm/i915/display/intel_pmdemand.c:491: +#define update_reg(reg, field, mask) do { \ + u32 current_val = serialized ? 0 : REG_FIELD_GET((mask), *(reg)); \ + u32 old_val = old ? old->params.field : 0; \ + u32 new_val = new->params.field; \ +\ + *(reg) &= ~(mask); \ + *(reg) |= REG_FIELD_PREP((mask), max3(old_val, new_val, current_val)); \ +} while (0) -:920: CHECK:MACRO_ARG_REUSE: Macro argument reuse 'field' - possible side-effects? #920: FILE: drivers/gpu/drm/i915/display/intel_pmdemand.c:491: +#define update_reg(reg, field, mask) do { \ + u32 current_val = serialized ? 0 : REG_FIELD_GET((mask), *(reg)); \ + u32 old_val = old ? old->params.field : 0; \ + u32 new_val = new->params.field; \ +\ + *(reg) &= ~(mask); \ + *(reg) |= REG_FIELD_PREP((mask), max3(old_val, new_val, current_val)); \ +} while (0) -:920: CHECK:MACRO_ARG_REUSE: Macro argument reuse 'mask' - possible side-effects? #920: FILE: drivers/gpu/drm/i915/display/intel_pmdemand.c:491: +#define update_reg(reg, field, mask) do { \ + u32 current_val = serialized ? 0 : REG_FIELD_GET((mask), *(reg)); \ + u32 old_val = old ? old->params.field : 0; \ + u32 new_val = new->params.field; \ +\ + *(reg) &= ~(mask); \ + *(reg) |= REG_FIELD_PREP((mask), max3(old_val, new_val, current_val)); \ +} while (0) total: 0 errors, 1 warnings, 3 checks, 1016 lines checked