✗ CI.checkpatch: warning for drm/i915/display: bunch of struct intel_display conversions (rev4)

Patchwork patchwork at emeril.freedesktop.org
Tue Oct 29 09:51:36 UTC 2024


== Series Details ==

Series: drm/i915/display: bunch of struct intel_display conversions (rev4)
URL   : https://patchwork.freedesktop.org/series/140323/
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 9b8c242a68f7a6d1d75a1f0ae805aabdacb0cd8b
Author: Jani Nikula <jani.nikula at intel.com>
Date:   Mon Oct 28 22:07:30 2024 +0200

    drm/i915/de: remove unnecessary generic wrappers
    
    With many of the intel_de_* callers switched over to struct
    intel_display, we can remove some of the unnecessary generic wrappers.
    
    Reviewed-by: Rodrigo Vivi <rodrigo.vivi at intel.com>
    Signed-off-by: Jani Nikula <jani.nikula at intel.com>
+ /mt/dim checkpatch 79811c7809c46a23b7cfceaf36d4994792c850da drm-intel
41f52d20292e drm/i915/gmbus: convert to struct intel_display
aa321e013a04 drm/i915/cx0: remove unnecessary includes
833f535c3bb9 drm/i915/cx0: convert to struct intel_display
-:441: WARNING:LONG_LINE: line length of 112 exceeds 100 columns
#441: FILE: drivers/gpu/drm/i915/display/intel_cx0_phy.c:2429:
+									  PHY_C20_B_MPLLB_CNTX_CFG(display, i));

-:446: WARNING:LONG_LINE: line length of 112 exceeds 100 columns
#446: FILE: drivers/gpu/drm/i915/display/intel_cx0_phy.c:2433:
+									  PHY_C20_A_MPLLB_CNTX_CFG(display, i));

-:455: WARNING:LONG_LINE: line length of 112 exceeds 100 columns
#455: FILE: drivers/gpu/drm/i915/display/intel_cx0_phy.c:2441:
+									  PHY_C20_B_MPLLA_CNTX_CFG(display, i));

-:460: WARNING:LONG_LINE: line length of 112 exceeds 100 columns
#460: FILE: drivers/gpu/drm/i915/display/intel_cx0_phy.c:2445:
+									  PHY_C20_A_MPLLA_CNTX_CFG(display, i));

-:617: WARNING:LONG_LINE: line length of 101 exceeds 100 columns
#617: FILE: drivers/gpu/drm/i915/display/intel_cx0_phy.c:2792:
+		if (intel_de_wait_for_clear(display, XELPDP_PORT_M2P_MSGBUS_CTL(display, port, lane),

total: 0 errors, 5 warnings, 0 checks, 984 lines checked
a8aaba1fa60c drm/i915/dpio: convert to struct intel_display
-:565: CHECK:LINE_SPACING: Please use a blank line after function/struct/union/enum declarations
#565: FILE: drivers/gpu/drm/i915/display/intel_dpio_phy.h:84:
 }
+static inline void bxt_dpio_phy_init(struct intel_display *display, enum dpio_phy phy)

-:569: CHECK:LINE_SPACING: Please use a blank line after function/struct/union/enum declarations
#569: FILE: drivers/gpu/drm/i915/display/intel_dpio_phy.h:87:
 }
+static inline void bxt_dpio_phy_uninit(struct intel_display *display, enum dpio_phy phy)

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

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

total: 0 errors, 0 warnings, 4 checks, 570 lines checked
e8ec313690b3 drm/i915/hdcp: further conversion to struct intel_display
3e426cb9fa11 drm/i915/dp/hdcp: convert to struct intel_display
0bf1e8c88a35 drm/i915/crt: convert to struct intel_display
-:218: CHECK:MULTIPLE_ASSIGNMENTS: multiple assignments should be avoided
#218: FILE: drivers/gpu/drm/i915/display/intel_crt.c:497:
+		save_adpa = adpa = intel_de_read(display, crt->adpa_reg);

-:272: CHECK:MULTIPLE_ASSIGNMENTS: multiple assignments should be avoided
#272: FILE: drivers/gpu/drm/i915/display/intel_crt.c:557:
+	save_adpa = adpa = intel_de_read(display, crt->adpa_reg);

-:704: CHECK:LINE_SPACING: Please use a blank line after function/struct/union/enum declarations
#704: FILE: drivers/gpu/drm/i915/display/intel_crt.h:26:
 }
+static inline void intel_crt_init(struct intel_display *display)

total: 0 errors, 0 warnings, 3 checks, 704 lines checked
6e582ec5311c drm/i915/display: convert vlv_wait_port_ready() to struct intel_display
747d3fb5b5d9 drm/i915/power: convert assert_chv_phy_status() to struct intel_display
332b5f15d5ec drm/i915/ips: convert to struct intel_display
e54d1779bcfd drm/i915/dsi: convert to struct intel_display
-:1366: WARNING:LONG_LINE: line length of 101 exceeds 100 columns
#1366: FILE: drivers/gpu/drm/i915/display/icl_dsi.c:2041:
+	if (drm_WARN_ON(display->drm, intel_connector->panel.vbt.dsi.cabc_ports & ~intel_dsi->ports))

total: 0 errors, 1 warnings, 0 checks, 1302 lines checked
9b8c242a68f7 drm/i915/de: remove unnecessary generic wrappers




More information about the Intel-xe mailing list