✗ Fi.CI.CHECKPATCH: warning for drm/i915: header cleanup of the day

From: Patchwork <patchwork@emeril.freedesktop.org>
To: Jani Nikula <jani.nikula@intel.com>
Cc: intel-gfx@lists.freedesktop.org
Subject: ✗ Fi.CI.CHECKPATCH: warning for drm/i915: header cleanup of the day
Date: Thu, 08 Aug 2019 14:51:20 -0000	[thread overview]
Message-ID: <20190808145120.4817.12701@emeril.freedesktop.org> (raw)
In-Reply-To: <cover.1565271681.git.jani.nikula@intel.com>

== Series Details ==

Series: drm/i915: header cleanup of the day
URL   : https://patchwork.freedesktop.org/series/64910/
State : warning

== Summary ==

$ dim checkpatch origin/drm-tip
263e77d0ad54 drm/i915: remove unused dev_priv->no_aux_handshake
-:7: ERROR:GIT_COMMIT_ID: Please use git commit description style 'commit <12+ chars of sha1> ("<title line>")' - ie: 'commit 3cf2efb1a7c6 ("Revert "drm/i915/dp: use VBT provided eDP params if available"")'
#7: 
3cf2efb1a7c6 ("Revert "drm/i915/dp: use VBT provided eDP params if

total: 1 errors, 0 warnings, 0 checks, 19 lines checked
51bcb5cc50ae drm/i915: move add_taint_for_CI() to i915_utils.h
967569ca7446 drm/i915: move I915_STATE_WARN() and _ON() to intel_display.h
-:33: CHECK:MACRO_ARG_REUSE: Macro argument reuse 'x' - possible side-effects?
#33: FILE: drivers/gpu/drm/i915/display/intel_display.h:589:
+#define I915_STATE_WARN_ON(x)						\
+	I915_STATE_WARN((x), "%s", "WARN_ON(" __stringify(x) ")")

total: 0 errors, 0 warnings, 1 checks, 46 lines checked
27978f72232d drm/i915: move printing and load error inject to i915_utils.[ch]
-:150: WARNING:FILE_PATH_CHANGES: added, moved or deleted file(s), does MAINTAINERS need updating?
#150: 
new file mode 100644

total: 0 errors, 1 warnings, 0 checks, 242 lines checked
b139c4e58bec drm/i915: extract i915_perf.h from i915_drv.h
-:91: WARNING:FILE_PATH_CHANGES: added, moved or deleted file(s), does MAINTAINERS need updating?
#91: 
new file mode 100644

total: 0 errors, 1 warnings, 0 checks, 81 lines checked
17f4e89e6e0d drm/i915: extract i915_sysfs.h from i915_drv.h
-:60: WARNING:FILE_PATH_CHANGES: added, moved or deleted file(s), does MAINTAINERS need updating?
#60: 
new file mode 100644

total: 0 errors, 1 warnings, 0 checks, 38 lines checked
f5ea7f575fdb drm/i915: extract i915_suspend.h from i915_drv.h
-:60: WARNING:FILE_PATH_CHANGES: added, moved or deleted file(s), does MAINTAINERS need updating?
#60: 
new file mode 100644

total: 0 errors, 1 warnings, 0 checks, 38 lines checked
f7eab6b0a831 drm/i915: extract i915_memcpy.h from i915_drv.h
-:115: WARNING:FILE_PATH_CHANGES: added, moved or deleted file(s), does MAINTAINERS need updating?
#115: 
new file mode 100644

total: 0 errors, 1 warnings, 0 checks, 96 lines checked
f38bad2b2252 drm/i915: extract gem/i915_gem_stolen.h from i915_drv.h
-:33: WARNING:FILE_PATH_CHANGES: added, moved or deleted file(s), does MAINTAINERS need updating?
#33: 
new file mode 100644

-:54: WARNING:UNSPECIFIED_INT: Prefer 'unsigned int' to bare use of 'unsigned'
#54: FILE: drivers/gpu/drm/i915/gem/i915_gem_stolen.h:17:
+				unsigned alignment);

-:57: WARNING:UNSPECIFIED_INT: Prefer 'unsigned int' to bare use of 'unsigned'
#57: FILE: drivers/gpu/drm/i915/gem/i915_gem_stolen.h:20:
+					 unsigned alignment, u64 start,

total: 0 errors, 3 warnings, 0 checks, 92 lines checked
22892c935c97 drm/i915: extract i915_gem_shrinker.h from i915_drv.h
-:21: WARNING:FILE_PATH_CHANGES: added, moved or deleted file(s), does MAINTAINERS need updating?
#21: 
new file mode 100644

-:43: WARNING:UNSPECIFIED_INT: Prefer 'unsigned int' to bare use of 'unsigned'
#43: FILE: drivers/gpu/drm/i915/gem/i915_gem_shrinker.h:18:
+			      unsigned flags);

-:54: CHECK:UNCOMMENTED_DEFINITION: struct mutex definition without comment
#54: FILE: drivers/gpu/drm/i915/gem/i915_gem_shrinker.h:29:
+				    struct mutex *mutex);

total: 0 errors, 2 warnings, 1 checks, 61 lines checked

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