== Series Details ==
Series: Remaining patches for basic GuC submission
URL : https://patchwork.freedesktop.org/series/92912/
State : warning
== Summary ==
$ dim checkpatch origin/drm-tip
c4c2a95bc42c drm/i915/guc: GuC virtual engines
-:595: CHECK:LINE_SPACING: Please don't use multiple blank lines
#595: FILE: drivers/gpu/drm/i915/gt/uc/intel_guc_submission.c:1895:
+
+
total: 0 errors, 0 warnings, 1 checks, 537 lines checked
ed2ce8f94165 drm/i915/guc: Make hangcheck work with GuC virtual engines
abe5c9520ece drm/i915: Hold reference to intel_context over life of i915_request
095ca97f853a drm/i915/guc: Disable bonding extension with GuC submission
9b220ffb651e drm/i915/guc: Direct all breadcrumbs for a class to single
breadcrumbs
e7bbc081df3c drm/i915: Add i915_sched_engine destroy vfunc
96e05d821dc0 drm/i915: Move active request tracking to a vfunc
2fe085686ac0 drm/i915/guc: Reset implementation for new GuC interface
-:198: CHECK:COMPARISON_TO_NULL: Comparison to NULL could be written
"guc->lrc_desc_pool_vaddr"
#198: FILE: drivers/gpu/drm/i915/gt/uc/intel_guc_submission.c:248:
+ return guc->lrc_desc_pool_vaddr != NULL;
-:851: CHECK:BRACES: braces {} should be used on all arms of this statement
#851: FILE: drivers/gpu/drm/i915/gt/uc/intel_guc_submission.c:1263:
+ if (unlikely(ret == -EBUSY)) {
[...]
+ } else if (unlikely(ret == -ENODEV))
[...]
-:1075: CHECK:LINE_SPACING: Please don't use multiple blank lines
#1075: FILE: drivers/gpu/drm/i915/gt/uc/intel_uc.c:568:
+
total: 0 errors, 0 warnings, 3 checks, 1029 lines checked
7cfdea9a7cca drm/i915: Reset GPU immediately if submission is disabled
-:94: CHECK:BRACES: Blank lines aren't necessary before a close brace '}'
#94: FILE: drivers/gpu/drm/i915/gt/intel_engine_heartbeat.c:223:
+
+}
total: 0 errors, 0 warnings, 1 checks, 181 lines checked
8bc28306ef3c drm/i915/guc: Add disable interrupts to guc sanitize
2d8442c0bcb5 drm/i915/guc: Suspend/resume implementation for new interface
93fcd3d3f15f drm/i915/guc: Handle context reset notification
a6a5b8bd25ca drm/i915/guc: Handle engine reset failure notification
39fc22487a99 drm/i915/guc: Enable the timer expired interrupt for GuC
6642efd29a52 drm/i915/guc: Provide mmio list to be saved/restored on engine
reset
-:360: CHECK:PARENTHESIS_ALIGNMENT: Alignment should match open parenthesis
#360: FILE: drivers/gpu/drm/i915/gt/uc/intel_guc_ads.c:212:
+ temp_set.registers = kmalloc_array(temp_set.size,
+ sizeof(*temp_set.registers),
-:389: CHECK:SPACING: No space is necessary after a cast
#389: FILE: drivers/gpu/drm/i915/gt/uc/intel_guc_ads.c:241:
+ temp_set.registers = (struct guc_mmio_reg *) (((u8 *) blob) + offset);
-:464: WARNING:FROM_SIGN_OFF_MISMATCH: From:/Signed-off-by: email name
mismatch: 'From: "Signed-off-by: John Harrison" ' !=
'Signed-off-by: John Harrison '
total: 0 errors, 1 warnings, 2 checks, 400 lines checked
5e642e904723 drm/i915/guc: Don't complain about reset races
ee7a77ae81e6 drm/i915/guc: Enable GuC engine reset
3a6d2423e625 drm/i915/guc: Capture error state on context reset
b4ed2cab4b35 drm/i915/guc: Fix for error capture after full GPU reset with GuC
-:120: CHECK:BRACES: braces {} should be used on all arms of this statement
#120: FILE: drivers/gpu/drm/i915/gt/intel_engine_cs.c:1716:
+ if (guc) {
[...]
+ } else
[...]
-:124: CHECK:BRACES: Unbalanced braces around else statement
#124: FILE: drivers/gpu/drm/i915/gt/intel_engine_cs.c:1720:
+ } else
total: 0 errors, 0 warnings, 2 checks, 347 lines checked
43ba142e24f9 drm/i915/guc: Hook GuC scheduling policies up
078bbab61932 drm/i915/guc: Connect reset modparam updates to GuC policy flags
-:53: CHECK:MACRO_ARG_REUSE: Macro argument reuse 'i915' - possible
side-effects?
#53: FILE: drivers/gpu/drm/i915/i915_debugfs_params.c:17:
+#define GET_I915(i915, name, ptr) \
+ do {\
+ struct i915_params *params; \
+ params = container_of(((void *) (ptr)), typeof(*params), name);
\
+ (i915) = container_of(params, typeof(*(i915)), params); \
+ } while (0)
-:56: CHECK:SPACING: No space is necessary after a cast
#56: FILE: drivers/gpu/drm/i915/i915_debugfs_params.c:20:
+ params = container_of(((void *) (ptr)), typeof(*params), name);
\
total: 0 errors, 0 warnings, 2 checks, 69 lines checked
bce9b2362ee8 drm/i915/guc: Include scheduling policies in the debugfs state dump
e1639b04b6cc drm/i915/guc: Add golden context to GuC ADS
-:262: CHECK:SPACING: No space is necessary after a cast
#262: FILE: drivers/gpu/drm/i915/gt/uc/intel_guc_ads.c:455:
+ ptr = ((u8 *) blob) + offset;
total: 0 errors, 0 warnings, 1 checks, 345 lines checked
5901f74f1b1c drm/i915/guc: Implement banned contexts for GuC submission
-:382: CHECK:PARENTHESIS_ALIGNMENT: Alignment should match open parenthesis
#382: FILE: drivers/gpu/drm/i915/gt/uc/intel_guc_submission.c:1412:
+ if (