✗ CI.checkpatch: warning for drm/i915/display: conversions to struct intel_display

[Patchwork]

== Series Details ==

Series: drm/i915/display: conversions to struct intel_display
URL   : https://patchwork.freedesktop.org/series/144749/
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 e5f913a0f834843e837392c7da955ee7fe12604d
Author: Jani Nikula <jani.nikula at intel.com>
Date:   Wed Feb 12 18:36:43 2025 +0200

    drm/i915/display: convert i915_pipestat_enable_mask() to struct intel_display
    
    Going forward, struct intel_display is the main display device data
    pointer. Convert i915_pipestat_enable_mask() to struct intel_display,
    allowing further conversions elsewhere.
    
    Signed-off-by: Jani Nikula <jani.nikula at intel.com>
+ /mt/dim checkpatch 1e935d0d289627796230e9e1ca1451647fe9a2ad drm-intel
309e6045bdfc drm/i915/dp: convert g4x_dp.[ch] to struct intel display
-:349: CHECK:LINE_SPACING: Please use a blank line after function/struct/union/enum declarations
#349: FILE: drivers/gpu/drm/i915/display/g4x_dp.h:32:
 }
+static inline bool g4x_dp_port_enabled(struct intel_display *display,

-:356: CHECK:LINE_SPACING: Please use a blank line after function/struct/union/enum declarations
#356: FILE: drivers/gpu/drm/i915/display/g4x_dp.h:38:
 }
+static inline bool g4x_dp_init(struct intel_display *display,

total: 0 errors, 0 warnings, 2 checks, 431 lines checked
9fb9a895a185 drm/i915/hdmi: convert g4x_hdmi.[ch] to struct intel_display
6dbf2bf737db drm/i915/ips: convert hsw_ips.c to struct intel_display
7b731afcb780 drm/i915/display: convert assert_transcoder*() to struct intel_display
9b541f288b4c drm/i915/display: convert assert_port_valid() to struct intel_display
98bfb0c0314a drm/i915/hpd: drop dev_priv parameter from intel_hpd_pin_default()
b727af307b1a drm/i915/display: convert intel_set_{cpu, pch}_fifo_underrun_reporting() to intel_display
e60d6897f59d drm/i915/sdvo: convert intel_sdvo.[ch] to struct intel_display
-:960: WARNING:LONG_LINE: line length of 101 exceeds 100 columns
#960: FILE: drivers/gpu/drm/i915/display/intel_sdvo.c:3170:
+			drm_property_create_range(display->drm, 0, "right_margin", 0, data_value[0]);

-:1157: CHECK:LINE_SPACING: Please use a blank line after function/struct/union/enum declarations
#1157: FILE: drivers/gpu/drm/i915/display/intel_sdvo.h:28:
 }
+static inline bool intel_sdvo_init(struct intel_display *display,

total: 0 errors, 1 warnings, 1 checks, 1036 lines checked
dd9f14206d5b drm/i915/display: convert intel_cpu_transcoder_mode_valid() to intel_display
d64e5e5d69be drm/i915/display: convert intel_mode_valid_max_plane_size() to intel_display
a3ee93c2794a drm/i915/dsi: convert platform checks to display->platform.<platform> style
b2d07622452e drm/i915/combo-phy: convert intel_combo_phy.[ch] to struct intel_display
-:61: CHECK:MACRO_ARG_REUSE: Macro argument reuse '__phy' - possible side-effects?
#61: FILE: drivers/gpu/drm/i915/display/intel_combo_phy.c:13:
+#define for_each_combo_phy(__display, __phy) \
 	for ((__phy) = PHY_A; (__phy) < I915_MAX_PHYS; (__phy)++)	\
+		for_each_if(intel_phy_is_combo(__display, __phy))

-:67: CHECK:MACRO_ARG_REUSE: Macro argument reuse '__phy' - possible side-effects?
#67: FILE: drivers/gpu/drm/i915/display/intel_combo_phy.c:17:
+#define for_each_combo_phy_reverse(__display, __phy) \
 	for ((__phy) = I915_MAX_PHYS; (__phy)-- > PHY_A;) \
+		for_each_if(intel_phy_is_combo(__display, __phy))

total: 0 errors, 0 warnings, 2 checks, 531 lines checked
f25b223c0326 drm/i915/display: convert intel_fifo_underrun.[ch] to struct intel_display
e5f913a0f834 drm/i915/display: convert i915_pipestat_enable_mask() to struct intel_display