== Series Details == Series: i915: Simplify mmio handling & add new DG2 shadow table (rev3) URL : https://patchwork.freedesktop.org/series/94534/ State : warning == Summary == $ dim checkpatch origin/drm-tip 20b20ad7947c drm/i915/uncore: Convert gen6/gen7 read operations to fwtable e6b2795f4353 drm/i915/uncore: Associate shadow table with uncore -:43: CHECK:LINE_SPACING: Please don't use multiple blank lines #43: FILE: drivers/gpu/drm/i915/intel_uncore.c:1041: + -:87: CHECK:LINE_SPACING: Please don't use multiple blank lines #87: FILE: drivers/gpu/drm/i915/intel_uncore.c:1710: + -:95: CHECK:MACRO_ARG_REUSE: Macro argument reuse 'uncore' - possible side-effects? #95: FILE: drivers/gpu/drm/i915/intel_uncore.c:1975: +#define ASSIGN_SHADOW_TABLE(uncore, d) \ +{ \ + (uncore)->shadowed_reg_table = d; \ + (uncore)->shadowed_reg_table_entries = ARRAY_SIZE((d)); \ +} -:95: CHECK:MACRO_ARG_REUSE: Macro argument reuse 'd' - possible side-effects? #95: FILE: drivers/gpu/drm/i915/intel_uncore.c:1975: +#define ASSIGN_SHADOW_TABLE(uncore, d) \ +{ \ + (uncore)->shadowed_reg_table = d; \ + (uncore)->shadowed_reg_table_entries = ARRAY_SIZE((d)); \ +} total: 0 errors, 0 warnings, 4 checks, 128 lines checked a8dd5ae3188f drm/i915/uncore: Replace gen8 write functions with general fwtable 3554b1cc5b33 drm/i915/uncore: Drop gen11/gen12 mmio write handlers 31b02534892f drm/i915/uncore: Drop gen11 mmio read handlers -:40: ERROR:COMPLEX_MACRO: Macros with complex values should be enclosed in parentheses #40: FILE: drivers/gpu/drm/i915/intel_uncore.c:1570: +#define __gen_fwtable_read(x) \ static u##x \ +fwtable_read##x(struct intel_uncore *uncore, i915_reg_t reg, bool trace) \ +{ \ enum forcewake_domains fw_engine; \ GEN6_READ_HEADER(x); \ + fw_engine = __fwtable_reg_read_fw_domains(uncore, offset); \ if (fw_engine) \ __force_wake_auto(uncore, fw_engine); \ val = __raw_uncore_read##x(uncore, reg); \ GEN6_READ_FOOTER; \ } -:65: ERROR:OPEN_BRACE: open brace '{' following function definitions go on the next line #65: FILE: drivers/gpu/drm/i915/intel_uncore.c:1583: +static enum forcewake_domains +fwtable_reg_read_fw_domains(struct intel_uncore *uncore, i915_reg_t reg) { total: 2 errors, 0 warnings, 0 checks, 81 lines checked 6d85e55215cb drm/i915/dg2: Add DG2-specific shadow register table
