✗ Fi.CI.CHECKPATCH: warning for drm/i915/mtl: Define MOCS and PAT tables for MTL (rev5)

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

 



== Series Details ==

Series: drm/i915/mtl: Define MOCS and PAT tables for MTL (rev5)
URL   : https://patchwork.freedesktop.org/series/115980/
State : warning

== Summary ==

Error: dim checkpatch failed
04d4f946f5f0 drm/i915/mtl: Set has_llc=0
f44a8a034a83 drm/i915/mtl: Define MOCS and PAT tables for MTL
-:63: CHECK:BRACES: Blank lines aren't necessary before a close brace '}'
#63: FILE: drivers/gpu/drm/i915/gt/intel_gtt.c:486:
+
+}

total: 0 errors, 0 warnings, 1 checks, 247 lines checked
5496af3927f8 drm/i915/mtl: Add PTE encode function
-:151: WARNING:AVOID_BUG: Do not crash the kernel unless it is absolutely unavoidable--use WARN_ON_ONCE() plus recovery code (if feasible) instead of BUG() or variants
#151: FILE: drivers/gpu/drm/i915/gt/intel_ggtt.c:229:
+	GEM_BUG_ON(addr & ~GEN12_GGTT_PTE_ADDR_MASK);

total: 0 errors, 1 warnings, 0 checks, 164 lines checked
72e4d658a8e9 drm/i915/mtl: workaround coherency issue for Media
b9a188850588 drm/i915/mtl: end support for set caching ioctl
eaa559214e8c drm/i915: preparation for using PAT index
-:422: CHECK:LINE_SPACING: Please don't use multiple blank lines
#422: FILE: drivers/gpu/drm/i915/selftests/mock_gem_device.c:186:
+
+

total: 0 errors, 0 warnings, 1 checks, 335 lines checked
932e5c4abaa6 drm/i915: use pat_index instead of cache_level
-:22: WARNING:COMMIT_LOG_LONG_LINE: Possible unwrapped commit description (prefer a maximum 75 chars per line)
#22: 
cached, uncached, or writethrough. For these simple cases, using cache_level

-:651: CHECK:PARENTHESIS_ALIGNMENT: Alignment should match open parenthesis
#651: FILE: drivers/gpu/drm/i915/gt/gen8_ppgtt.c:873:
+					      i915_gem_get_pat_index(vm->i915,
+							I915_CACHE_NONE));

-:921: CHECK:PARENTHESIS_ALIGNMENT: Alignment should match open parenthesis
#921: FILE: drivers/gpu/drm/i915/gt/intel_ggtt.c:1303:
+					 i915_gem_get_pat_index(vm->i915,
+							I915_CACHE_NONE),

-:1619: CHECK:PARENTHESIS_ALIGNMENT: Alignment should match open parenthesis
#1619: FILE: drivers/gpu/drm/i915/i915_gem.c:424:
+					i915_gem_object_get_dma_address(obj,
+							offset >> PAGE_SHIFT),

-:1634: CHECK:PARENTHESIS_ALIGNMENT: Alignment should match open parenthesis
#1634: FILE: drivers/gpu/drm/i915/i915_gem.c:606:
+					i915_gem_object_get_dma_address(obj,
+							offset >> PAGE_SHIFT),

-:1652: CHECK:PARENTHESIS_ALIGNMENT: Alignment should match open parenthesis
#1652: FILE: drivers/gpu/drm/i915/i915_gpu_error.c:1121:
+						i915_gem_get_pat_index(gt->i915,
+							I915_CACHE_NONE),

-:1658: CHECK:PARENTHESIS_ALIGNMENT: Alignment should match open parenthesis
#1658: FILE: drivers/gpu/drm/i915/i915_gpu_error.c:1126:
+						i915_gem_get_pat_index(gt->i915,
+							I915_CACHE_NONE),

-:1776: CHECK:PARENTHESIS_ALIGNMENT: Alignment should match open parenthesis
#1776: FILE: drivers/gpu/drm/i915/selftests/i915_gem.c:62:
+				     i915_gem_get_pat_index(i915,
+							I915_CACHE_NONE),

-:1822: CHECK:PARENTHESIS_ALIGNMENT: Alignment should match open parenthesis
#1822: FILE: drivers/gpu/drm/i915/selftests/i915_gem_gtt.c:363:
+					     i915_gem_get_pat_index(vm->i915,
+						     I915_CACHE_NONE),

-:1834: ERROR:CODE_INDENT: code indent should use tabs where possible
#1834: FILE: drivers/gpu/drm/i915/selftests/i915_gem_gtt.c:1384:
+^I^I^I^I^I                    I915_CACHE_NONE),$

-:1834: CHECK:PARENTHESIS_ALIGNMENT: Alignment should match open parenthesis
#1834: FILE: drivers/gpu/drm/i915/selftests/i915_gem_gtt.c:1384:
+				     i915_gem_get_pat_index(i915,
+					                    I915_CACHE_NONE),

-:1868: CHECK:PARENTHESIS_ALIGNMENT: Alignment should match open parenthesis
#1868: FILE: drivers/gpu/drm/i915/selftests/intel_memory_region.c:1075:
+					  i915_gem_get_pat_index(i915,
+							I915_CACHE_NONE),

total: 1 errors, 1 warnings, 10 checks, 1599 lines checked
e50cbe9097b5 drm/i915: Allow user to set cache at BO creation





[Index of Archives]     [AMD Graphics]     [Linux USB Devel]     [Linux Audio Users]     [Yosemite News]     [Linux Kernel]     [Linux SCSI]

  Powered by Linux