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

Patchwork patchwork at emeril.freedesktop.org
Tue Oct 29 11:42:39 UTC 2024


== Series Details ==

Series: drm/i915/display: bunch of struct intel_display conversions (rev5)
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 8696a21bb141e046bd241012bb7cf7febb4c5499
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 738c18d8599dce736da8b6958f96d2eac08a36ab drm-intel
56090af9a525 drm/i915/gmbus: convert to struct intel_display
83e970e049bf drm/i915/cx0: remove unnecessary includes
98228275c98d 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
1fbc80b61dd3 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
c1837d3d78d0 drm/i915/hdcp: further conversion to struct intel_display
34b0ec042179 drm/i915/dp/hdcp: convert to struct intel_display
9de9c1b74254 drm/i915/crt: convert to struct intel_display
-:220: CHECK:MULTIPLE_ASSIGNMENTS: multiple assignments should be avoided
#220: FILE: drivers/gpu/drm/i915/display/intel_crt.c:497:
+		save_adpa = adpa = intel_de_read(display, crt->adpa_reg);

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

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

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

total: 0 errors, 1 warnings, 3 checks, 697 lines checked
c2894f16c8a2 drm/i915/display: convert vlv_wait_port_ready() to struct intel_display
a0c8f58e8304 drm/i915/power: convert assert_chv_phy_status() to struct intel_display
3b910a25ca78 drm/i915/ips: convert to struct intel_display
9b8fc6b1f92c 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
8696a21bb141 drm/i915/de: remove unnecessary generic wrappers




More information about the Intel-xe mailing list