[Intel-gfx] ✗ Fi.CI.CHECKPATCH: warning for drm/i915/perf: add OA interrupt support (rev3)
Patchwork
patchwork at emeril.freedesktop.org
Mon Feb 18 18:53:09 UTC 2019
== Series Details ==
Series: drm/i915/perf: add OA interrupt support (rev3)
URL : https://patchwork.freedesktop.org/series/54280/
State : warning
== Summary ==
$ dim checkpatch origin/drm-tip
3b4560b8c649 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
aabeed256cf1 drm/i915/perf: move pollin setup to non hw specific code
966e0440dd80 drm/i915/perf: only append status when data is available
e4ffcd8eb379 drm/i915/perf: add new open param to configure polling of OA buffer
70333ed90cac drm/i915: handle interrupts from the OA unit
8ded9c4b56f3 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:2342:
+ 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:1637:
+ * buffer in i915. This option in conjuction with a long polling delay
total: 0 errors, 2 warnings, 2 checks, 145 lines checked
6da84143a465 drm/i915/perf: add flushing ioctl
More information about the Intel-gfx
mailing list