[Intel-gfx] ✗ Fi.CI.CHECKPATCH: warning for Refactor Gen11+ SAGV support (rev11)

[Patchwork]

== Series Details ==

Series: Refactor Gen11+ SAGV support (rev11)
URL   : https://patchwork.freedesktop.org/series/68028/
State : warning

== Summary ==

$ dim checkpatch origin/drm-tip
1294bb4306ba drm/i915: Refactor intel_can_enable_sagv
-:254: CHECK:PARENTHESIS_ALIGNMENT: Alignment should match open parenthesis
#254: FILE: drivers/gpu/drm/i915/intel_pm.c:3847:
+	for_each_new_intel_crtc_in_state(state, crtc,
+					     new_crtc_state, i) {

-:439: CHECK:PARENTHESIS_ALIGNMENT: Alignment should match open parenthesis
#439: FILE: drivers/gpu/drm/i915/intel_pm.c:4421:
+skl_plane_wm_level(struct intel_plane *plane,
+		const struct intel_crtc_state *crtc_state,

-:562: CHECK:PARENTHESIS_ALIGNMENT: Alignment should match open parenthesis
#562: FILE: drivers/gpu/drm/i915/intel_pm.c:4638:
+			wm_uv_level = skl_plane_wm_level(plane, crtc_state,
+						      level, true);

-:653: CHECK:PARENTHESIS_ALIGNMENT: Alignment should match open parenthesis
#653: FILE: drivers/gpu/drm/i915/intel_pm.c:5059:
+		skl_compute_plane_wm(crtc_state, 0, latency,
+		     wm_params, &levels[0],

-:655: CHECK:BRACES: Unbalanced braces around else statement
#655: FILE: drivers/gpu/drm/i915/intel_pm.c:5061:
+	} else

-:657: CHECK:PARENTHESIS_ALIGNMENT: Alignment should match open parenthesis
#657: FILE: drivers/gpu/drm/i915/intel_pm.c:5063:
+		memcpy(&plane_wm->sagv_wm0, &levels[0],
+			sizeof(struct skl_wm_level));

-:740: CHECK:PARENTHESIS_ALIGNMENT: Alignment should match open parenthesis
#740: FILE: drivers/gpu/drm/i915/intel_pm.c:5759:
+			if (!skl_plane_wm_equals(dev_priv,
+				&old_crtc_state->wm.skl.optimal.planes[plane_id],

-:790: CHECK:PARENTHESIS_ALIGNMENT: Alignment should match open parenthesis
#790: FILE: drivers/gpu/drm/i915/intel_pm.c:5976:
+				memcpy(&wm->sagv_wm0, &wm->wm[level],
+					sizeof(struct skl_wm_level));

total: 0 errors, 0 warnings, 8 checks, 697 lines checked
885e46a4e56d drm/i915: Restrict qgv points which don't have enough bandwidth.
-:229: CHECK:BRACES: Blank lines aren't necessary after an open brace '{'
#229: FILE: drivers/gpu/drm/i915/display/intel_bw.c:483:
+	if (!intel_can_enable_sagv(state)) {
+

-:396: CHECK:BRACES: braces {} should be used on all arms of this statement
#396: FILE: drivers/gpu/drm/i915/display/intel_display.c:14964:
+	if (INTEL_GEN(dev_priv) < 11) {
[...]
+	} else
[...]

-:399: CHECK:BRACES: Unbalanced braces around else statement
#399: FILE: drivers/gpu/drm/i915/display/intel_display.c:14967:
+	} else

total: 0 errors, 0 warnings, 3 checks, 405 lines checked