[Intel-gfx] ✗ Fi.CI.CHECKPATCH: warning for drm/i915/tgl: Render/media decompression support (rev10)

[Patchwork]

== Series Details ==

Series: drm/i915/tgl: Render/media decompression support (rev10)
URL   : https://patchwork.freedesktop.org/series/71125/
State : warning

== Summary ==

$ dim checkpatch origin/drm-tip
0e244fde713c drm/framebuffer: Format modifier for Intel Gen-12 render compression
b8b5d1fe8a14 drm/i915: Use intel_tile_height() instead of re-implementing
cfb29f3dfc82 drm/i915: Move CCS stride alignment W/A inside intel_fb_stride_alignment
302b5a034ca3 drm/i915: Extract framebufer CCS offset checks into a function
9c5b00d5c04f drm/i915: Add helpers to select correct ccs/aux planes
5b81b67242e0 drm/i915/tgl: Gen-12 render decompression
31001f05269d drm/i915/tgl: Make sure FBs have a correct CCS plane stride
bc4f1cd5e087 drm/i915: Skip rotated offset adjustment for unsupported modifiers
38863af06562 drm/i915: Make sure Y slave planes get all the required state
a3117f225e10 drm/i915: Make sure CCS YUV semiplanar format checks work
-:55: ERROR:CODE_INDENT: code indent should use tabs where possible
#55: FILE: drivers/gpu/drm/i915/display/intel_display.c:3855:
+^I^I^I^I^I        fb->modifier)) {$

-:55: CHECK:PARENTHESIS_ALIGNMENT: Alignment should match open parenthesis
#55: FILE: drivers/gpu/drm/i915/display/intel_display.c:3855:
+	if (intel_format_info_is_yuv_semiplanar(fb->format,
+					        fb->modifier)) {

-:65: CHECK:PREFER_KERNEL_TYPES: Prefer kernel type 'u64' over 'uint64_t'
#65: FILE: drivers/gpu/drm/i915/display/intel_display.c:5785:
+		  uint64_t modifier, bool need_scaler)

total: 1 errors, 0 warnings, 2 checks, 142 lines checked
428f9f32c6f5 drm/framebuffer: Format modifier for Intel Gen-12 media compression
78444dfcb223 drm/fb: Extend format_info member arrays to handle four planes
e9bbc97aba9f drm/i915/tgl: Gen-12 display can decompress surfaces compressed by the media engine
-:13: WARNING:COMMIT_LOG_LONG_LINE: Possible unwrapped commit description (prefer a maximum 75 chars per line)
#13: 
compressed buffers. Unlike render decompression, plane 6 and  plane 7 do not

-:124: WARNING:MISSING_BREAK: Possible switch case/default not preceded by break or fallthrough comment
#124: FILE: drivers/gpu/drm/i915/display/intel_display.c:2644:
+	case I915_FORMAT_MOD_Y_TILED_GEN12_MC_CCS:

total: 0 errors, 2 warnings, 0 checks, 478 lines checked
8420af3f7552 drm/framebuffer: Format modifier for Intel Gen 12 render compression with Clear Color
64e5f7fd003b drm/i915/tgl: Add Clear Color support for TGL Render Decompression
-:304: CHECK:MACRO_ARG_REUSE: Macro argument reuse 'pipe' - possible side-effects?
#304: FILE: drivers/gpu/drm/i915/i915_reg.h:6889:
+#define PLANE_CC_VAL(pipe, plane)	\
+	_MMIO_PLANE(plane, _PLANE_CC_VAL_1(pipe), _PLANE_CC_VAL_2(pipe))

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