✗ Fi.CI.CHECKPATCH: warning for series starting with [CI,1/9] drm/i915/perf: Replace global wakeref tracking with engine-pm

[Patchwork <patchwork@emeril.freedesktop.org>]

== Series Details ==

Series: series starting with [CI,1/9] drm/i915/perf: Replace global wakeref tracking with engine-pm
URL   : https://patchwork.freedesktop.org/series/67874/
State : warning

== Summary ==

$ dim checkpatch origin/drm-tip
4aa72c4ef98d drm/i915/perf: Replace global wakeref tracking with engine-pm
de5e384f91f5 drm/i915/perf: introduce a versioning of the i915-perf uapi
ac5a1dbba730 drm/i915/perf: allow for CS OA configs to be created lazily
15679a893373 drm/i915: add support for perf configuration queries
f4fc96f15f7a drm/i915/perf: implement active wait for noa configurations
-:46: CHECK:SPACING: spaces preferred around that '<<' (ctx:VxV)
#46: FILE: drivers/gpu/drm/i915/gt/intel_gpu_commands.h:228:
+#define   PIPE_CONTROL_WRITE_TIMESTAMP			(3<<14)
                                       			  ^

-:168: ERROR:CODE_INDENT: code indent should use tabs where possible
#168: FILE: drivers/gpu/drm/i915/i915_perf.c:1555:
+^I^I^I^I^I       ^Ioffset) + 4 * d;$

-:168: WARNING:SPACE_BEFORE_TAB: please, no space before tabs
#168: FILE: drivers/gpu/drm/i915/i915_perf.c:1555:
+^I^I^I^I^I       ^Ioffset) + 4 * d;$

-:181: CHECK:OPEN_ENDED_LINE: Lines should not end with a '('
#181: FILE: drivers/gpu/drm/i915/i915_perf.c:1568:
+		DIV64_U64_ROUND_UP(

-:215: CHECK:MULTIPLE_ASSIGNMENTS: multiple assignments should be avoided
#215: FILE: drivers/gpu/drm/i915/i915_perf.c:1602:
+	batch = cs = i915_gem_object_pin_map(bo, I915_MAP_WB);

-:223: CHECK:OPEN_ENDED_LINE: Lines should not end with a '('
#223: FILE: drivers/gpu/drm/i915/i915_perf.c:1610:
+		cs = save_restore_register(

-:226: CHECK:OPEN_ENDED_LINE: Lines should not end with a '('
#226: FILE: drivers/gpu/drm/i915/i915_perf.c:1613:
+	cs = save_restore_register(

-:328: CHECK:OPEN_ENDED_LINE: Lines should not end with a '('
#328: FILE: drivers/gpu/drm/i915/i915_perf.c:1715:
+		cs = save_restore_register(

-:331: CHECK:OPEN_ENDED_LINE: Lines should not end with a '('
#331: FILE: drivers/gpu/drm/i915/i915_perf.c:1718:
+	cs = save_restore_register(

-:448: WARNING:FILE_PATH_CHANGES: added, moved or deleted file(s), does MAINTAINERS need updating?
#448: 
new file mode 100644

-:453: WARNING:SPDX_LICENSE_TAG: Missing or malformed SPDX-License-Identifier tag in line 1
#453: FILE: drivers/gpu/drm/i915/selftests/i915_perf.c:1:
+/*

-:454: WARNING:SPDX_LICENSE_TAG: Misplaced SPDX-License-Identifier tag - use line 1 instead
#454: FILE: drivers/gpu/drm/i915/selftests/i915_perf.c:2:
+ * SPDX-License-Identifier: MIT

total: 1 errors, 4 warnings, 7 checks, 590 lines checked
2245bacfcea6 drm/i915/perf: execute OA configuration from command stream
def0ed974f64 drm/i915/perf: Allow dynamic reconfiguration of the OA stream
c04dded670da drm/i915/perf: allow holding preemption on filtered ctx
e7a841a6b27d drm/i915/execlists: Prevent merging requests with conflicting flags
-:14: WARNING:COMMIT_LOG_LONG_LINE: Possible unwrapped commit description (prefer a maximum 75 chars per line)
#14: 
References: 2a98f4e65bba ("drm/i915: add infrastructure to hold off preemption on a request")

-:14: ERROR:GIT_COMMIT_ID: Please use git commit description style 'commit <12+ chars of sha1> ("<title line>")' - ie: 'commit 2a98f4e65bba ("drm/i915: add infrastructure to hold off preemption on a request")'
#14: 
References: 2a98f4e65bba ("drm/i915: add infrastructure to hold off preemption on a request")

total: 1 errors, 1 warnings, 0 checks, 9 lines checked

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