[Intel-gfx] ✗ Fi.CI.CHECKPATCH: warning for drm/i915: GTT remapping for display

2019-05-09 Thread Patchwork 
== Series Details ==

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

== Summary ==

$ dim checkpatch origin/drm-tip
e1a44e3ca92e drm/i915: Add a new "remapped" gtt_view
-:98: CHECK:LINE_SPACING: Please don't use multiple blank lines
#98: FILE: drivers/gpu/drm/i915/i915_gem.c:5106:
+
+

-:246: CHECK:SPACING: spaces preferred around that '*' (ctx:VxV)
#246: FILE: drivers/gpu/drm/i915/i915_gem_gtt.h:196:
+   BUILD_BUG_ON(sizeof(struct intel_remapped_info) != 9*sizeof(unsigned 
int));
^

total: 0 errors, 0 warnings, 2 checks, 286 lines checked
08493348be9d drm/i915/selftests: Add mock selftest for remapped vmas
-:76: CHECK:SPACING: spaces preferred around that '-' (ctx:VxV)
#76: FILE: drivers/gpu/drm/i915/selftests/i915_vma.c:436:
+   if (left < PAGE_SIZE || left & (PAGE_SIZE-1)) {
 ^

-:92: CHECK:LINE_SPACING: Please don't use multiple blank lines
#92: FILE: drivers/gpu/drm/i915/selftests/i915_vma.c:452:
+
+

-:128: WARNING:SUSPECT_CODE_INDENT: suspect code indent for conditional 
statements (8, 8)
#128: FILE: drivers/gpu/drm/i915/selftests/i915_vma.c:506:
+   for (t = types; *t; t++) {
for (a = planes; a->width; a++) {

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

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

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

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

total: 0 errors, 5 warnings, 2 checks, 165 lines checked
86cfa7fe1e0b drm/i915/selftests: Add live vma selftest
5f08f33bb992 drm/i915: Shuffle stride checking code around
f7bb2263cd69 drm/i915: Overcome display engine stride limits via GTT remapping
494ec5d13440 drm/i915: Align dumb buffer stride to 4k to allow for gtt remapping
5fc14462b19b drm/i915: Bump fb stride limit to 128KiB for gen4+ and 256KiB for 
gen7+
-:44: CHECK:SPACING: spaces preferred around that '*' (ctx:VxV)
#44: FILE: drivers/gpu/drm/i915/intel_display.c:2530:
+   return 256*1024;
  ^

-:46: CHECK:SPACING: spaces preferred around that '*' (ctx:VxV)
#46: FILE: drivers/gpu/drm/i915/intel_display.c:2532:
+   return 128*1024;
  ^

total: 0 errors, 0 warnings, 2 checks, 19 lines checked
853938b87ba6 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

[Intel-gfx] ✗ Fi.CI.CHECKPATCH: warning for drm/i915: GTT remapping for display (rev7)

2019-01-24 Thread Patchwork 
== Series Details ==

Series: drm/i915: GTT remapping for display (rev7)
URL   : https://patchwork.freedesktop.org/series/55415/
State : warning

== Summary ==

$ dim checkpatch origin/drm-tip
02b1fe191700 drm/i915: Add a new "remapped" gtt_view
-:96: CHECK:LINE_SPACING: Please don't use multiple blank lines
#96: FILE: drivers/gpu/drm/i915/i915_gem.c:5687:
+
+

-:244: CHECK:SPACING: spaces preferred around that '*' (ctx:VxV)
#244: FILE: drivers/gpu/drm/i915/i915_gem_gtt.h:195:
+   BUILD_BUG_ON(sizeof(struct intel_remapped_info) != 9*sizeof(unsigned 
int));
^

total: 0 errors, 0 warnings, 2 checks, 286 lines checked
2264b41f8fa4 drm/i915/selftests: Add mock selftest for remapped vmas
-:75: CHECK:SPACING: spaces preferred around that '-' (ctx:VxV)
#75: FILE: drivers/gpu/drm/i915/selftests/i915_vma.c:436:
+   if (left < PAGE_SIZE || left & (PAGE_SIZE-1)) {
 ^

-:91: CHECK:LINE_SPACING: Please don't use multiple blank lines
#91: FILE: drivers/gpu/drm/i915/selftests/i915_vma.c:452:
+
+

-:127: WARNING:SUSPECT_CODE_INDENT: suspect code indent for conditional 
statements (8, 8)
#127: FILE: drivers/gpu/drm/i915/selftests/i915_vma.c:506:
+   for (t = types; *t; t++) {
for (a = planes; a->width; a++) {

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

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

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

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

total: 0 errors, 5 warnings, 2 checks, 165 lines checked
b4401cf43d11 drm/i915/selftests: Add live vma selftest
a31fbb99b1ce drm/i915: Overcome display engine stride limits via GTT remapping
77a6058e039f 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:2480:
+   return 256*1024;
  ^

total: 0 errors, 0 warnings, 1 checks, 15 lines checked
9044b091f79e drm/i915: Bump gen7+ fb size limits to 16kx16k
a0fc50b1d3bf hack: drm/i915: Always remap gtt
-:9: WARNING:COMMIT_MESSAGE: Missing commit description - Add an appropriate one

-:21: ERROR:MISSING_SIGN_OFF: Missing Signed-off-by: line(s)

total: 1 errors, 1 warnings, 0 checks, 9 lines checked
e35f31943952 hack: align dumb buffer stride to 4k to allow for gtt remapping
-:10: WARNING:COMMIT_MESSAGE: Missing commit description - Add an appropriate 
one

-:26: ERROR:MISSING_SIGN_OFF: Missing Signed-off-by: line(s)

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

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

[Intel-gfx] ✗ Fi.CI.CHECKPATCH: warning for drm/i915: GTT remapping for display (rev2)

2019-01-18 Thread Patchwork 
== Series Details ==

Series: drm/i915: GTT remapping for display (rev2)
URL   : https://patchwork.freedesktop.org/series/55415/
State : warning

== Summary ==

$ dim checkpatch origin/drm-tip
08b5a861466d drm/i915: Add a new "remapped" gtt_view
-:96: CHECK:LINE_SPACING: Please don't use multiple blank lines
#96: FILE: drivers/gpu/drm/i915/i915_gem.c:5685:
+
+

-:244: CHECK:SPACING: spaces preferred around that '*' (ctx:VxV)
#244: FILE: drivers/gpu/drm/i915/i915_gem_gtt.h:195:
+   BUILD_BUG_ON(sizeof(struct intel_remapped_info) != 9*sizeof(unsigned 
int));
^

total: 0 errors, 0 warnings, 2 checks, 286 lines checked
48e2be95ad44 drm/i915/selftests: Add mock selftest for remapped vmas
-:78: CHECK:SPACING: spaces preferred around that '-' (ctx:VxV)
#78: FILE: drivers/gpu/drm/i915/selftests/i915_vma.c:438:
+   if (left < PAGE_SIZE || left & (PAGE_SIZE-1)) {
 ^

-:97: CHECK:LINE_SPACING: Please don't use multiple blank lines
#97: FILE: drivers/gpu/drm/i915/selftests/i915_vma.c:457:
+
+

-:133: WARNING:SUSPECT_CODE_INDENT: suspect code indent for conditional 
statements (8, 8)
#133: FILE: drivers/gpu/drm/i915/selftests/i915_vma.c:511:
+   for (t = types; *t; t++) {
for (a = planes; a->width; a++) {

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

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

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

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

total: 0 errors, 5 warnings, 2 checks, 172 lines checked
31c281758b75 drm/i915/selftests: Add live vma selftest
dd914fe9943f drm/i915: Overcome display engine stride limits via GTT remapping
2883c9d21fd3 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:2484:
+   return 256*1024;
  ^

total: 0 errors, 0 warnings, 1 checks, 19 lines checked
32ab0d5da81a drm/i915: Bump gen7+ fb size limits to 16kx16k
a3ea0c0f5a1f hack: drm/i915: Always remap gtt
-:8: WARNING:COMMIT_MESSAGE: Missing commit description - Add an appropriate one

-:47: ERROR:MISSING_SIGN_OFF: Missing Signed-off-by: line(s)

total: 1 errors, 1 warnings, 0 checks, 35 lines checked
e70492ee3f5e hack: align dumb buffer stride to 4k to allow for gtt remapping
-:10: WARNING:COMMIT_MESSAGE: Missing commit description - Add an appropriate 
one

-:26: ERROR:MISSING_SIGN_OFF: Missing Signed-off-by: line(s)

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

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

[Intel-gfx] ✗ Fi.CI.CHECKPATCH: warning for drm/i915: GTT remapping for display

2019-01-18 Thread Patchwork 
== Series Details ==

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

== Summary ==

$ dim checkpatch origin/drm-tip
6c45cc6e9157 drm/i915: Add a new "remapped" gtt_view
-:96: CHECK:LINE_SPACING: Please don't use multiple blank lines
#96: FILE: drivers/gpu/drm/i915/i915_gem.c:5685:
+
+

-:244: CHECK:SPACING: spaces preferred around that '*' (ctx:VxV)
#244: FILE: drivers/gpu/drm/i915/i915_gem_gtt.h:195:
+   BUILD_BUG_ON(sizeof(struct intel_remapped_info) != 9*sizeof(unsigned 
int));
^

total: 0 errors, 0 warnings, 2 checks, 286 lines checked
e5cab34e27df drm/i915/selftests: Add mock selftest for remapped vmas
-:78: CHECK:SPACING: spaces preferred around that '-' (ctx:VxV)
#78: FILE: drivers/gpu/drm/i915/selftests/i915_vma.c:438:
+   if (left < PAGE_SIZE || left & (PAGE_SIZE-1)) {
 ^

-:97: CHECK:LINE_SPACING: Please don't use multiple blank lines
#97: FILE: drivers/gpu/drm/i915/selftests/i915_vma.c:457:
+
+

-:133: WARNING:SUSPECT_CODE_INDENT: suspect code indent for conditional 
statements (8, 8)
#133: FILE: drivers/gpu/drm/i915/selftests/i915_vma.c:511:
+   for (t = types; *t; t++) {
for (a = planes; a->width; a++) {

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

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

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

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

total: 0 errors, 5 warnings, 2 checks, 172 lines checked
2fe756c3f657 drm/i915/selftests: Add live vma selftest
6d769ecac679 drm/i915: Overcome display engine stride limits via GTT remapping
67f25d52d1f2 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:2484:
+   return 256*1024;
  ^

total: 0 errors, 0 warnings, 1 checks, 19 lines checked
3203cda3c436 drm/i915: Bump gen7+ fb size limits to 16kx16k
792906e7f600 hack: drm/i915: Always remap gtt
-:8: WARNING:COMMIT_MESSAGE: Missing commit description - Add an appropriate one

-:47: ERROR:MISSING_SIGN_OFF: Missing Signed-off-by: line(s)

total: 1 errors, 1 warnings, 0 checks, 35 lines checked
796dff7bd3da hack: align dumb buffer stride to 4k to allow for gtt remapping
-:9: WARNING:COMMIT_MESSAGE: Missing commit description - Add an appropriate one

-:20: ERROR:MISSING_SIGN_OFF: Missing Signed-off-by: line(s)

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

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

[Intel-gfx] ✗ Fi.CI.CHECKPATCH: warning for drm/i915: GTT remapping for display

2019-01-11 Thread Patchwork 
== Series Details ==

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

== Summary ==

$ dim checkpatch origin/drm-tip
ff6533f795e4 drm/i915: Add a new "remapped" gtt_view
-:97: CHECK:LINE_SPACING: Please don't use multiple blank lines
#97: FILE: drivers/gpu/drm/i915/i915_gem.c:6066:
+
+

-:245: CHECK:SPACING: spaces preferred around that '*' (ctx:VxV)
#245: FILE: drivers/gpu/drm/i915/i915_gem_gtt.h:195:
+   BUILD_BUG_ON(sizeof(struct intel_remapped_info) != 9*sizeof(unsigned 
int));
^

total: 0 errors, 0 warnings, 2 checks, 286 lines checked
0b54a3a5a2f8 drm/i915/selftests: Add mock selftest for remapped vmas
-:79: CHECK:SPACING: spaces preferred around that '-' (ctx:VxV)
#79: FILE: drivers/gpu/drm/i915/selftests/i915_vma.c:438:
+   if (left < PAGE_SIZE || left & (PAGE_SIZE-1)) {
 ^

-:98: CHECK:LINE_SPACING: Please don't use multiple blank lines
#98: FILE: drivers/gpu/drm/i915/selftests/i915_vma.c:457:
+
+

-:134: WARNING:SUSPECT_CODE_INDENT: suspect code indent for conditional 
statements (8, 8)
#134: FILE: drivers/gpu/drm/i915/selftests/i915_vma.c:511:
+   for (t = types; *t; t++) {
for (a = planes; a->width; a++) {

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

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

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

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

total: 0 errors, 5 warnings, 2 checks, 172 lines checked
7d5558eb88e7 drm/i915/selftests: Add live vma selftest
dd746017e751 drm/i915: Overcome display engine stride limits via GTT remapping
1541975fafd0 drm/i915: Bump gen4+ fb stride limit to 256KiB
-:41: CHECK:SPACING: spaces preferred around that '*' (ctx:VxV)
#41: FILE: drivers/gpu/drm/i915/intel_display.c:2478:
+   return 256*1024;
  ^

total: 0 errors, 0 warnings, 1 checks, 19 lines checked
8de5d269d35a 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

[Intel-gfx] ✗ Fi.CI.CHECKPATCH: warning for drm/i915: GTT remapping for display (rev5)

2018-10-23 Thread Patchwork 
== Series Details ==

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

== Summary ==

$ dim checkpatch origin/drm-tip
7ecc46df21aa drm/i915: Make sure fb gtt offsets stay within 32bits
-:44: CHECK:MACRO_ARG_REUSE: Macro argument reuse 'B' - possible side-effects?
#44: FILE: drivers/gpu/drm/i915/i915_utils.h:57:
+#define add_overflows(A, B) \
+   add_overflows_t(typeof((A) + (B)), (A), (B))

total: 0 errors, 0 warnings, 1 checks, 50 lines checked
e7d9b84981c9 drm/i915: Decouple SKL stride units from 
intel_fb_stride_alignment()
2a232ab75fe1 drm/i915: Add a new "remapped" gtt_view
-:177: CHECK:SPACING: spaces preferred around that '*' (ctx:VxV)
#177: 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, 231 lines checked
ee777eb6655c drm/i915/selftests: Add mock selftest for remapped vmas
-:114: WARNING:SUSPECT_CODE_INDENT: suspect code indent for conditional 
statements (8, 8)
#114: FILE: drivers/gpu/drm/i915/selftests/i915_vma.c:493:
+   for (t = types; *t; t++) {
for (a = planes; a->width; a++) {

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

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

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

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

total: 0 errors, 5 warnings, 0 checks, 129 lines checked
d2f2a9c684e0 drm/i915/selftests: Add live vma selftest
45bb3b57aa21 drm/i915: Overcome display engine stride limits via GTT remapping
0439f5d5b684 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:2489:
+   return 256*1024;
  ^

total: 0 errors, 0 warnings, 1 checks, 19 lines checked
fa36cffd 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

[Intel-gfx] ✗ Fi.CI.CHECKPATCH: warning for drm/i915: GTT remapping for display

2018-09-25 Thread Patchwork 
== 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, , 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, , 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

[Intel-gfx] ✗ Fi.CI.CHECKPATCH: warning for drm/i915: GTT remapping for display (rev2)

2018-07-20 Thread Patchwork 
== Series Details ==

Series: drm/i915: GTT remapping for display (rev2)
URL   : https://patchwork.freedesktop.org/series/46886/
State : warning

== Summary ==

$ dim checkpatch origin/drm-tip
e205f0cf0cc1 drm/i915: Fix glk/cnl display w/a #1175
c6a91ca3ac1f drm/i915: s/tile_offset/aligned_offset/
4ee0a1a700d7 drm/i915: Add .max_stride() plane hook
-:33: CHECK:SPACING: spaces preferred around that '*' (ctx:VxV)
#33: FILE: drivers/gpu/drm/i915/intel_display.c:3222:
+   return 16*1024;
 ^

-:35: CHECK:SPACING: spaces preferred around that '*' (ctx:VxV)
#35: FILE: drivers/gpu/drm/i915/intel_display.c:3224:
+   return 32*1024;
 ^

-:38: CHECK:SPACING: spaces preferred around that '*' (ctx:VxV)
#38: FILE: drivers/gpu/drm/i915/intel_display.c:3227:
+   return 8*1024;
^

-:40: CHECK:SPACING: spaces preferred around that '*' (ctx:VxV)
#40: FILE: drivers/gpu/drm/i915/intel_display.c:3229:
+   return 16*1024;
 ^

-:43: CHECK:SPACING: spaces preferred around that '*' (ctx:VxV)
#43: FILE: drivers/gpu/drm/i915/intel_display.c:3232:
+   return 4*1024;
^

-:45: CHECK:SPACING: spaces preferred around that '*' (ctx:VxV)
#45: FILE: drivers/gpu/drm/i915/intel_display.c:3234:
+   return 8*1024;
^

total: 0 errors, 0 warnings, 6 checks, 203 lines checked
93251a9665a0 drm/i915: Use pipe A primary plane .max_stride() as the global 
stride limit
4b1480959ec6 drm/i915: Rename the plane_state->main/aux to 
plane_state->color_plane[]
1c5d64f97780 drm/i915: Store the final plane stride in plane_state
ff0cc350264a drm/i915: Store ggtt_view in plane_state
20740285ed71 drm/i915: s/int plane/int color_plane/
7ba7bd1f398a drm/i915: Nuke plane->can_scale/min_downscale
65bc9cb0ca53 drm/i915: Extract per-platform plane->check() functions
e1e9380a3fd9 drm/i915: Move skl plane fb related checks into a better place
7da6e070d957 drm/i915: Move display w/a #1175
5825b1993f3a drm/i915: Move chv rotation checks to plane->check()
e84f5ff122b3 drm/i915: Extract intel_cursor_check_surface()
3ea198dea8eb drm/i915: Add a new "remapped" gtt_view
-:165: CHECK:SPACING: spaces preferred around that '*' (ctx:VxV)
#165: FILE: drivers/gpu/drm/i915/i915_gem_gtt.h:173:
+   BUILD_BUG_ON(sizeof(struct intel_remapped_info) != 10*sizeof(unsigned 
int));
 ^

total: 0 errors, 0 warnings, 1 checks, 215 lines checked
6f75836c2a10 drm/i915: Overcome display engine stride limits via GTT remapping
8d6a79c031b9 drm/i915: Bump gen4+ fb stride limit to 256KiB
-:38: CHECK:SPACING: spaces preferred around that '*' (ctx:VxV)
#38: FILE: drivers/gpu/drm/i915/intel_display.c:2519:
+   return 256*1024;
  ^

total: 0 errors, 0 warnings, 1 checks, 19 lines checked
f27708878ebf drm/i915: Bump gen4+ fb size limits to 32kx32k

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

[Intel-gfx] ✗ Fi.CI.CHECKPATCH: warning for drm/i915: GTT remapping for display

2018-07-19 Thread Patchwork 
== Series Details ==

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

== Summary ==

$ dim checkpatch origin/drm-tip
0655c1625b0a drm/i915: Fix glk/cnl display w/a #1175
c95e166cfd63 drm/i915: s/tile_offset/aligned_offset/
020d82fa9ed9 drm/i915: Add .max_stride() plane hook
-:33: CHECK:SPACING: spaces preferred around that '*' (ctx:VxV)
#33: FILE: drivers/gpu/drm/i915/intel_display.c:3222:
+   return 16*1024;
 ^

-:35: CHECK:SPACING: spaces preferred around that '*' (ctx:VxV)
#35: FILE: drivers/gpu/drm/i915/intel_display.c:3224:
+   return 32*1024;
 ^

-:38: CHECK:SPACING: spaces preferred around that '*' (ctx:VxV)
#38: FILE: drivers/gpu/drm/i915/intel_display.c:3227:
+   return 8*1024;
^

-:40: CHECK:SPACING: spaces preferred around that '*' (ctx:VxV)
#40: FILE: drivers/gpu/drm/i915/intel_display.c:3229:
+   return 16*1024;
 ^

-:43: CHECK:SPACING: spaces preferred around that '*' (ctx:VxV)
#43: FILE: drivers/gpu/drm/i915/intel_display.c:3232:
+   return 4*1024;
^

-:45: CHECK:SPACING: spaces preferred around that '*' (ctx:VxV)
#45: FILE: drivers/gpu/drm/i915/intel_display.c:3234:
+   return 8*1024;
^

total: 0 errors, 0 warnings, 6 checks, 203 lines checked
1081f1cd3907 drm/i915: Use pipe A primary plane .max_stride() as the global 
stride limit
466e9b94ef6b drm/i915: Rename the plane_state->main/aux to 
plane_state->color_plane[]
22a29833142d drm/i915: Store the final plane stride in plane_state
2f03e86953a0 drm/i915: Store ggtt_view in plane_state
efc25030f9e0 drm/i915: s/int plane/int color_plane/
05b220464c98 drm/i915: Nuke plane->can_scale/min_downscale
416335e1c500 drm/i915: Extract per-platform plane->check() functions
eea47ec1a126 drm/i915: Move skl plane fb related checks into a better place
05adce059f5c drm/i915: Move display w/a #1175
6d2816cc9e13 drm/i915: Move chv rotation checks to plane->check()
ad9b29709a6c drm/i915: Extract intel_cursor_check_surface()
f0c079c521ab drm/i915: Add a new "remapped" gtt_view
-:165: CHECK:SPACING: spaces preferred around that '*' (ctx:VxV)
#165: FILE: drivers/gpu/drm/i915/i915_gem_gtt.h:173:
+   BUILD_BUG_ON(sizeof(struct intel_remapped_info) != 10*sizeof(unsigned 
int));
 ^

total: 0 errors, 0 warnings, 1 checks, 215 lines checked
9ff0fb075794 drm/i915: Overcome display engine stride limits via GTT remapping
57b7b6f7429b drm/i915: Bump gen4+ fb stride limit to 256KiB
-:38: CHECK:SPACING: spaces preferred around that '*' (ctx:VxV)
#38: FILE: drivers/gpu/drm/i915/intel_display.c:2519:
+   return 256*1024;
  ^

total: 0 errors, 0 warnings, 1 checks, 19 lines checked
b6b0962ca481 drm/i915: Bump gen4+ fb size limits to 32kx32k

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