✗ Fi.CI.CHECKPATCH: warning for series starting with [1/2] Revert "drm/i915/perf: add a parameter to control the size of OA buffer"

From: Patchwork <patchwork@emeril.freedesktop.org>
To: Joonas Lahtinen <joonas.lahtinen@linux.intel.com>
Cc: intel-gfx@lists.freedesktop.org
Subject: ✗ Fi.CI.CHECKPATCH: warning for series starting with [1/2] Revert "drm/i915/perf: add a parameter to control the size of OA buffer"
Date: Fri, 16 Nov 2018 14:00:27 -0000	[thread overview]
Message-ID: <20181116140027.4917.91391@emeril.freedesktop.org> (raw)
In-Reply-To: <20181116135510.13807-1-joonas.lahtinen@linux.intel.com>

== Series Details ==

Series: series starting with [1/2] Revert "drm/i915/perf: add a parameter to control the size of OA buffer"
URL   : https://patchwork.freedesktop.org/series/52612/
State : warning

== Summary ==

$ dim checkpatch origin/drm-tip
b808c374682f Revert "drm/i915/perf: add a parameter to control the size of OA buffer"
-:42: CHECK:MACRO_ARG_PRECEDENCE: Macro argument 'tail' may be better as '(tail)' to avoid precedence issues
#42: FILE: drivers/gpu/drm/i915/i915_perf.c:221:
+#define OA_TAKEN(tail, head)	((tail - head) & (OA_BUFFER_SIZE - 1))

-:42: CHECK:MACRO_ARG_PRECEDENCE: Macro argument 'head' may be better as '(head)' to avoid precedence issues
#42: FILE: drivers/gpu/drm/i915/i915_perf.c:221:
+#define OA_TAKEN(tail, head)	((tail - head) & (OA_BUFFER_SIZE - 1))

total: 0 errors, 0 warnings, 2 checks, 272 lines checked
11e9d279512d Revert "drm/i915/perf: Fix warning in documentation"

_______________________________________________
Intel-gfx mailing list
Intel-gfx@lists.freedesktop.org
https://lists.freedesktop.org/mailman/listinfo/intel-gfx