[Intel-gfx] ✗ Fi.CI.CHECKPATCH: warning for series starting with [1/2] drm/i915/selftests: Verify the LRC register layout between init and HW (rev2)

Patchwork patchwork at emeril.freedesktop.org
Sun Sep 22 14:31:29 UTC 2019


== Series Details ==

Series: series starting with [1/2] drm/i915/selftests: Verify the LRC register layout between init and HW (rev2)
URL   : https://patchwork.freedesktop.org/series/67018/
State : warning

== Summary ==

$ dim checkpatch origin/drm-tip
3651aa32b5c0 drm/i915/selftests: Verify the LRC register layout between init and HW
-:61: CHECK:MACRO_ARG_REUSE: Macro argument reuse 'x' - possible side-effects?
#61: FILE: drivers/gpu/drm/i915/gt/intel_lrc.c:474:
+#define REG(x) (((x) >> 2) | BUILD_BUG_ON_ZERO(x >= 0x200))

-:62: ERROR:COMPLEX_MACRO: Macros with complex values should be enclosed in parentheses
#62: FILE: drivers/gpu/drm/i915/gt/intel_lrc.c:475:
+#define REG16(x) \
+	(((x) >> 9) | BIT(7) | BUILD_BUG_ON_ZERO(x >= 0x10000)), \
+	(((x) >> 2) & 0x7f)

-:62: CHECK:MACRO_ARG_REUSE: Macro argument reuse 'x' - possible side-effects?
#62: FILE: drivers/gpu/drm/i915/gt/intel_lrc.c:475:
+#define REG16(x) \
+	(((x) >> 9) | BIT(7) | BUILD_BUG_ON_ZERO(x >= 0x10000)), \
+	(((x) >> 2) & 0x7f)

total: 1 errors, 0 warnings, 2 checks, 1125 lines checked
d35131b8ca31 drm/i915/tgl: Swap engines for rc6/powersaving



More information about the Intel-gfx mailing list