== Series Details == Series: drm_i915_private to intel_display cleanup (rev2) URL : https://patchwork.freedesktop.org/series/144588/ State : warning == Summary == Error: dim checkpatch failed 37f096e9961c drm/i915/display_debug_fs: Use intel_display wherever possible -:54: WARNING:SUSPECT_CODE_INDENT: suspect code indent for conditional statements (8, 63) #54: 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, 507 lines checked a052eb583c24 drm/i915/display_debug_fs: Prefer using display->platform 56335358a845 drm/i915/dpll: Change param to intel_display in for_each_shared_dpll -:141: CHECK:MACRO_ARG_REUSE: Macro argument reuse '__display' - possible side-effects? #141: 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)++) -:141: CHECK:MACRO_ARG_REUSE: Macro argument reuse '__i' - possible side-effects? #141: 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, 218 lines checked f35ee0e7dd2c drm/i915/dpll: Use intel_display for dpll dump and compare hw state b06cd75b3507 drm/i915/dpll: Use intel_display possible in shared_dpll_mgr hooks d6e03bdf8b64 drm/i915/dpll: Use intel_display for asserting pll 5082d7956254 drm/i915/dpll: Use intel_display for update_refclk hook 4df213d2a364 drm/i915/dpll: Accept intel_display as argument for shared_dpll_init 547b40419030 drm/i915/dpll: Replace all other leftover drm_i915_private
