✗ Fi.CI.CHECKPATCH: warning for drm_i915_private to intel_display cleanup (rev3)

[Patchwork]

== Series Details ==

Series: drm_i915_private to intel_display cleanup (rev3)
URL   : https://patchwork.freedesktop.org/series/144588/
State : warning

== Summary ==

Error: dim checkpatch failed
26d1f67a12d2 drm/i915: Use intel_display wherever possible
-:59: WARNING:SUSPECT_CODE_INDENT: suspect code indent for conditional statements (8, 63)
#59: FILE: drivers/gpu/drm/i915/display/intel_display_debugfs.c:95:
+	if (DISPLAY_VER(display) >= 9)
 		/* no global SR status; inspect per-plane WM */;

total: 0 errors, 1 warnings, 0 checks, 512 lines checked
d231a45cadf8 drm/i915/dpll: Change param to intel_display in for_each_shared_dpll
-:145: CHECK:MACRO_ARG_REUSE: Macro argument reuse '__display' - possible side-effects?
#145: FILE: drivers/gpu/drm/i915/display/intel_dpll_mgr.h:33:
+#define for_each_shared_dpll(__display, __pll, __i) \
+	for ((__i) = 0; (__i) < (__display)->dpll.num_shared_dpll && \
+		     ((__pll) = &(__display)->dpll.shared_dplls[(__i)]) ; (__i)++)

-:145: CHECK:MACRO_ARG_REUSE: Macro argument reuse '__i' - possible side-effects?
#145: FILE: drivers/gpu/drm/i915/display/intel_dpll_mgr.h:33:
+#define for_each_shared_dpll(__display, __pll, __i) \
+	for ((__i) = 0; (__i) < (__display)->dpll.num_shared_dpll && \
+		     ((__pll) = &(__display)->dpll.shared_dplls[(__i)]) ; (__i)++)

total: 0 errors, 0 warnings, 2 checks, 210 lines checked
60dd3d110de9 drm/i915/dpll: Use intel_display for dpll dump and compare hw state
53896c57e730 drm/i915/dpll: Use intel_display possible in shared_dpll_mgr hooks
b29ca0b14e45 drm/i915/dpll: Use intel_display for asserting pll
3b78188ab98a drm/i915/dpll: Use intel_display for update_refclk hook
8dce1119aebc drm/i915/dpll: Accept intel_display as argument for shared_dpll_init
c52f87551853 drm/i915/dpll: Replace all other leftover drm_i915_private