✗ 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/144587/
State : warning

== Summary ==

+ KERNEL=/kernel
+ git clone https://gitlab.freedesktop.org/drm/maintainer-tools mt
Cloning into 'mt'...
warning: redirecting to https://gitlab.freedesktop.org/drm/maintainer-tools.git/
+ git -C mt rev-list -n1 origin/master
22f9cda3436b4fe965b5c5f31d2f2c1bcb483189
+ cd /kernel
+ git config --global --add safe.directory /kernel
+ git log -n1
commit 5e5f9d46c04ee198e82f206b2756d60d2bd7593d
Author: Suraj Kandpal <suraj.kandpal at intel.com>
Date:   Wed Feb 12 13:15:42 2025 +0530

    drm/i915/dpll: Replace all other leftover drm_i915_private
    
    Replace all other left over drm_i915_private with intel_display
    in dpll_mgr.c.
    
    --v2
    -Don't use inline to_intel_display() [Jani]
    
    Signed-off-by: Suraj Kandpal <suraj.kandpal at intel.com>
    Reviewed-by: Jani Nikula <jani.nikula at intel.com>
+ /mt/dim checkpatch a5fe717b56653a6f5e5b98ed07d5fe54789465c9 drm-intel
a8e12254dd21 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
df0b5870f73b 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
a3ebd94c04d7 drm/i915/dpll: Use intel_display for dpll dump and compare hw state
cf3059540f07 drm/i915/dpll: Use intel_display possible in shared_dpll_mgr hooks
c3a87e6e398a drm/i915/dpll: Use intel_display for asserting pll
4d361f3548c1 drm/i915/dpll: Use intel_display for update_refclk hook
2d137baeebdf drm/i915/dpll: Accept intel_display as argument for shared_dpll_init
5e5f9d46c04e drm/i915/dpll: Replace all other leftover drm_i915_private