[Intel-gfx] ✗ Fi.CI.CHECKPATCH: warning for drm/i915/perf: add OA interrupt support (rev2)
Patchwork
patchwork at emeril.freedesktop.org
Thu Jan 17 08:09:07 UTC 2019
== Series Details ==
Series: drm/i915/perf: add OA interrupt support (rev2)
URL : https://patchwork.freedesktop.org/series/54280/
State : warning
== Summary ==
$ dim checkpatch origin/drm-tip
f0460c1ade9e drm/i915/perf: rework aging tail workaround
-:241: CHECK:SPACING: No space is necessary after a cast
#241: FILE: drivers/gpu/drm/i915/i915_perf.c:500:
+ u32 *report32 = (void *) report;
-:319: CHECK:MULTIPLE_ASSIGNMENTS: multiple assignments should be avoided
#319: FILE: drivers/gpu/drm/i915/i915_perf.c:795:
+ report32[0] = report32[1] = 0;
-:363: CHECK:MULTIPLE_ASSIGNMENTS: multiple assignments should be avoided
#363: FILE: drivers/gpu/drm/i915/i915_perf.c:998:
+ report32[0] = report32[1] = 0;
total: 0 errors, 0 warnings, 3 checks, 354 lines checked
94b8c039703f drm/i915/perf: reset pollin when perf stream is enabled
47f65f25c7c0 drm/i915/perf: only append status when data is available
4fbe19755ae5 drm/i915/perf: add new open param to configure polling of OA buffer
3bbe259283a3 drm/i915: handle interrupts from the OA unit
0c7a8458e075 drm/i915/perf: add interrupt enabling parameter
-:7: WARNING:TYPO_SPELLING: 'conjuction' may be misspelled - perhaps 'conjunction'?
#7:
mechanism of the HW. In conjuction with long periods for checks for
-:30: CHECK:LINE_SPACING: Please don't use multiple blank lines
#30: FILE: drivers/gpu/drm/i915/i915_perf.c:421:
+
-:122: CHECK:UNNECESSARY_PARENTHESES: Unnecessary parentheses around 'dev_priv->perf.oa.half_full_count_last !=
atomic64_read(&dev_priv->perf.oa.half_full_count)'
#122: FILE: drivers/gpu/drm/i915/i915_perf.c:2344:
+ if (stream->oa_interrupt_monitor &&
+ (dev_priv->perf.oa.half_full_count_last !=
+ atomic64_read(&dev_priv->perf.oa.half_full_count))) {
-:170: WARNING:TYPO_SPELLING: 'conjuction' may be misspelled - perhaps 'conjunction'?
#170: FILE: include/uapi/drm/i915_drm.h:1553:
+ * buffer in i915. This option in conjuction with a long polling delay
total: 0 errors, 2 warnings, 2 checks, 145 lines checked
49538d21dc79 drm/i915/perf: add flushing ioctl
More information about the Intel-gfx
mailing list