== 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
