[Intel-gfx] ✗ Fi.CI.CHECKPATCH: warning for i915: Simplify mmio handling & add new DG2 shadow table (rev3)

[Patchwork]

== 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