✗ CI.checkpatch: warning for drm/i915/display: convert I915_STATE_WARN() to struct intel_display (rev2)

[Patchwork]

== Series Details ==

Series: drm/i915/display: convert I915_STATE_WARN() to struct intel_display (rev2)
URL   : https://patchwork.freedesktop.org/series/140450/
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
30ab6715fc09baee6cc14cb3c89ad8858688d474
+ cd /kernel
+ git config --global --add safe.directory /kernel
+ git log -n1
commit abab92412612f8e0ee324de50a0f4150c6bda99e
Author: Jani Nikula <jani.nikula at intel.com>
Date:   Thu Oct 24 19:25:10 2024 +0300

    drm/i915/display: convert I915_STATE_WARN() to struct intel_display
    
    Convert I915_STATE_WARN() to struct intel_display *, and rename to
    INTEL_DISPLAY_STATE_WARN(). Do some minor opportunistic struct
    drm_i915_private to struct intel_display conversions while at it.
    
    v2: crtc_state may be NULL in intel_connector_verify_state()
    
    Reviewed-by: Ville Syrjälä <ville.syrjala at linux.intel.com>
    Signed-off-by: Jani Nikula <jani.nikula at intel.com>
+ /mt/dim checkpatch 810b65a3bab4c67517ef050df06dcf998d66b664 drm-intel
abab92412612 drm/i915/display: convert I915_STATE_WARN() to struct intel_display
-:156: WARNING:LONG_LINE: line length of 109 exceeds 100 columns
#156: FILE: drivers/gpu/drm/i915/display/intel_cx0_phy.c:3408:
+			INTEL_DISPLAY_STATE_WARN(display, mpll_hw_state->mpllb[i] != mpll_sw_state->mpllb[i],

-:167: WARNING:LONG_LINE: line length of 109 exceeds 100 columns
#167: FILE: drivers/gpu/drm/i915/display/intel_cx0_phy.c:3415:
+			INTEL_DISPLAY_STATE_WARN(display, mpll_hw_state->mplla[i] != mpll_sw_state->mplla[i],

-:264: CHECK:MACRO_ARG_REUSE: Macro argument reuse '__display' - possible side-effects?
#264: FILE: drivers/gpu/drm/i915/display/intel_display.h:594:
+#define INTEL_DISPLAY_STATE_WARN(__display, condition, format...) ({	\
 	int __ret_warn_on = !!(condition);				\
 	if (unlikely(__ret_warn_on))					\
+		if (!drm_WARN((__display)->drm, (__display)->params.verbose_state_checks, format)) \
+			drm_err((__display)->drm, format);		\
 	unlikely(__ret_warn_on);					\
 })

-:349: WARNING:LONG_LINE: line length of 154 exceeds 100 columns
#349: FILE: drivers/gpu/drm/i915/display/intel_display_power.c:1212:
+				 (intel_de_read(display, UTIL_PIN_CTL) & (UTIL_PIN_ENABLE | UTIL_PIN_MODE_MASK)) == (UTIL_PIN_ENABLE | UTIL_PIN_MODE_PWM),

-:549: WARNING:LONG_LINE: line length of 103 exceeds 100 columns
#549: FILE: drivers/gpu/drm/i915/display/intel_dpll_mgr.c:4709:
+					 pll->info->name, pipe_name(crtc->pipe), pll->state.pipe_mask);

-:576: WARNING:LONG_LINE: line length of 111 exceeds 100 columns
#576: FILE: drivers/gpu/drm/i915/display/intel_fdi.c:41:
+					  TRANS_DDI_FUNC_CTL(display, cpu_transcoder)) & TRANS_DDI_FUNC_ENABLE;

-:914: WARNING:LONG_LINE: line length of 109 exceeds 100 columns
#914: FILE: drivers/gpu/drm/i915/display/intel_pch_display.c:89:
+				 intel_crt_port_enabled(dev_priv, PCH_ADPA, &port_pipe) && port_pipe == pipe,

-:923: WARNING:LONG_LINE: line length of 110 exceeds 100 columns
#923: FILE: drivers/gpu/drm/i915/display/intel_pch_display.c:94:
+				 intel_lvds_port_enabled(dev_priv, PCH_LVDS, &port_pipe) && port_pipe == pipe,

-:1033: WARNING:LONG_LINE: line length of 104 exceeds 100 columns
#1033: FILE: drivers/gpu/drm/i915/display/intel_snps_phy.c:2024:
+				 "[CRTC:%d:%s] mismatch in MPLLB: %s (expected 0x%08x, found 0x%08x)", \

total: 0 errors, 8 warnings, 1 checks, 947 lines checked