✗ Fi.CI.CHECKPATCH: warning for drm/i915: GTT remapping for display

From: Patchwork <patchwork@emeril.freedesktop.org>
To: Ville Syrjala <ville.syrjala@linux.intel.com>
Cc: intel-gfx@lists.freedesktop.org
Subject: ✗ Fi.CI.CHECKPATCH: warning for drm/i915: GTT remapping for display
Date: Tue, 25 Sep 2018 20:05:54 -0000	[thread overview]
Message-ID: <20180925200554.24225.10413@emeril.freedesktop.org> (raw)
In-Reply-To: <20180925193714.25280-1-ville.syrjala@linux.intel.com>

== Series Details ==

Series: drm/i915: GTT remapping for display
URL   : https://patchwork.freedesktop.org/series/50165/
State : warning

== Summary ==

$ dim checkpatch origin/drm-tip
fd6c4b3fa1e9 drm/i915: Make sure fb gtt offsets stay within 32bits
dafb1135ed8c drm/i915: Decouple SKL stride units from intel_fb_stride_alignment()
96eb024e5885 drm/i915: Add a new "remapped" gtt_view
-:190: CHECK:SPACING: spaces preferred around that '*' (ctx:VxV)
#190: FILE: drivers/gpu/drm/i915/i915_gem_gtt.h:193:
+	BUILD_BUG_ON(sizeof(struct intel_remapped_info) != 9*sizeof(unsigned int));
 	                                                    ^

total: 0 errors, 0 warnings, 1 checks, 248 lines checked
fa6cf412dafd drm/i915/selftests: Add mock selftest for remapped vmas
-:111: WARNING:SUSPECT_CODE_INDENT: suspect code indent for conditional statements (8, 8)
#111: FILE: drivers/gpu/drm/i915/selftests/i915_vma.c:493:
+	for (t = types; *t; t++) {
 	for (a = planes; a->width; a++) {

-:129: WARNING:DEEP_INDENTATION: Too many leading tabs - consider code refactoring
#129: FILE: drivers/gpu/drm/i915/selftests/i915_vma.c:554:
+						if (view.type == I915_GGTT_VIEW_ROTATED)

-:130: WARNING:LONG_LINE: line over 100 characters
#130: FILE: drivers/gpu/drm/i915/selftests/i915_vma.c:555:
+							sg = assert_rotated(obj, &view.rotated, n, sg);

-:131: WARNING:DEEP_INDENTATION: Too many leading tabs - consider code refactoring
#131: FILE: drivers/gpu/drm/i915/selftests/i915_vma.c:556:
+						else

-:132: WARNING:LONG_LINE: line over 100 characters
#132: FILE: drivers/gpu/drm/i915/selftests/i915_vma.c:557:
+							sg = assert_remapped(obj, &view.remapped, n, sg);

total: 0 errors, 5 warnings, 0 checks, 124 lines checked
762ba8d98203 drm/i915/selftests: Add live vma selftest
43dcb570fdec drm/i915: Overcome display engine stride limits via GTT remapping
8b2eedbf6d7d drm/i915: Bump gen4+ fb stride limit to 256KiB
-:40: CHECK:SPACING: spaces preferred around that '*' (ctx:VxV)
#40: FILE: drivers/gpu/drm/i915/intel_display.c:2536:
+		return 256*1024;
 		          ^

total: 0 errors, 0 warnings, 1 checks, 19 lines checked
0dea7c2d42cf drm/i915: Bump gen7+ fb size limits to 16kx16k

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