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