✗ Fi.CI.CHECKPATCH: warning for drm/i915: prepare for xe driver display integration

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

 



== Series Details ==

Series: drm/i915: prepare for xe driver display integration
URL   : https://patchwork.freedesktop.org/series/123595/
State : warning

== Summary ==

Error: dim checkpatch failed
f4942d5a5a1e drm/i915: define I915 during i915 driver build
b938b39e0624 drm/i915/display: add I915 conditional build to intel_lvds.h
-:30: CHECK:LINE_SPACING: Please use a blank line after function/struct/union/enum declarations
#30: FILE: drivers/gpu/drm/i915/display/intel_lvds.h:28:
+}
+static inline void intel_lvds_init(struct drm_i915_private *dev_priv)

-:33: CHECK:LINE_SPACING: Please use a blank line after function/struct/union/enum declarations
#33: FILE: drivers/gpu/drm/i915/display/intel_lvds.h:31:
+}
+static inline struct intel_encoder *intel_get_lvds_encoder(struct drm_i915_private *dev_priv)

-:37: CHECK:LINE_SPACING: Please use a blank line after function/struct/union/enum declarations
#37: FILE: drivers/gpu/drm/i915/display/intel_lvds.h:35:
+}
+static inline bool intel_is_dual_link_lvds(struct drm_i915_private *dev_priv)

total: 0 errors, 0 warnings, 3 checks, 29 lines checked
a69f63e53be3 drm/i915/display: add I915 conditional build to hsw_ips.h
-:31: CHECK:LINE_SPACING: Please use a blank line after function/struct/union/enum declarations
#31: FILE: drivers/gpu/drm/i915/display/hsw_ips.h:32:
+}
+static inline bool hsw_ips_pre_update(struct intel_atomic_state *state,

-:36: CHECK:LINE_SPACING: Please use a blank line after function/struct/union/enum declarations
#36: FILE: drivers/gpu/drm/i915/display/hsw_ips.h:37:
+}
+static inline void hsw_ips_post_update(struct intel_atomic_state *state,

-:40: CHECK:LINE_SPACING: Please use a blank line after function/struct/union/enum declarations
#40: FILE: drivers/gpu/drm/i915/display/hsw_ips.h:41:
+}
+static inline bool hsw_crtc_supports_ips(struct intel_crtc *crtc)

-:44: CHECK:LINE_SPACING: Please use a blank line after function/struct/union/enum declarations
#44: FILE: drivers/gpu/drm/i915/display/hsw_ips.h:45:
+}
+static inline bool hsw_crtc_state_ips_capable(const struct intel_crtc_state *crtc_state)

-:48: CHECK:LINE_SPACING: Please use a blank line after function/struct/union/enum declarations
#48: FILE: drivers/gpu/drm/i915/display/hsw_ips.h:49:
+}
+static inline int hsw_ips_compute_config(struct intel_atomic_state *state,

-:53: CHECK:LINE_SPACING: Please use a blank line after function/struct/union/enum declarations
#53: FILE: drivers/gpu/drm/i915/display/hsw_ips.h:54:
+}
+static inline void hsw_ips_get_config(struct intel_crtc_state *crtc_state)

-:56: CHECK:LINE_SPACING: Please use a blank line after function/struct/union/enum declarations
#56: FILE: drivers/gpu/drm/i915/display/hsw_ips.h:57:
+}
+static inline void hsw_ips_crtc_debugfs_add(struct intel_crtc *crtc)

total: 0 errors, 0 warnings, 7 checks, 46 lines checked
2e9a86422513 drm/i915/display: add I915 conditional build to i9xx_plane.h
-:33: CHECK:LINE_SPACING: Please use a blank line after function/struct/union/enum declarations
#33: FILE: drivers/gpu/drm/i915/display/i9xx_plane.h:36:
+}
+static inline int i9xx_check_plane_surface(struct intel_plane_state *plane_state)

-:37: CHECK:LINE_SPACING: Please use a blank line after function/struct/union/enum declarations
#37: FILE: drivers/gpu/drm/i915/display/i9xx_plane.h:40:
+}
+static inline struct intel_plane *

-:42: CHECK:LINE_SPACING: Please use a blank line after function/struct/union/enum declarations
#42: FILE: drivers/gpu/drm/i915/display/i9xx_plane.h:45:
+}
+static inline void i9xx_get_initial_plane_config(struct intel_crtc *crtc,

total: 0 errors, 0 warnings, 3 checks, 33 lines checked
4fc83ec3987d drm/i915/display: add I915 conditional build to intel_lpe_audio.h
-:31: CHECK:LINE_SPACING: Please use a blank line after function/struct/union/enum declarations
#31: FILE: drivers/gpu/drm/i915/display/intel_lpe_audio.h:27:
+}
+static inline void intel_lpe_audio_teardown(struct drm_i915_private *dev_priv)

-:34: CHECK:LINE_SPACING: Please use a blank line after function/struct/union/enum declarations
#34: FILE: drivers/gpu/drm/i915/display/intel_lpe_audio.h:30:
+}
+static inline void intel_lpe_audio_irq_handler(struct drm_i915_private *dev_priv)

-:37: CHECK:LINE_SPACING: Please use a blank line after function/struct/union/enum declarations
#37: FILE: drivers/gpu/drm/i915/display/intel_lpe_audio.h:33:
+}
+static inline void intel_lpe_audio_notify(struct drm_i915_private *dev_priv,

total: 0 errors, 0 warnings, 3 checks, 29 lines checked
7682469d640d drm/i915/display: add I915 conditional build to intel_pch_refclk.h
-:31: CHECK:LINE_SPACING: Please use a blank line after function/struct/union/enum declarations
#31: FILE: drivers/gpu/drm/i915/display/intel_pch_refclk.h:26:
+}
+static inline void lpt_disable_iclkip(struct drm_i915_private *dev_priv)

-:34: CHECK:LINE_SPACING: Please use a blank line after function/struct/union/enum declarations
#34: FILE: drivers/gpu/drm/i915/display/intel_pch_refclk.h:29:
+}
+static inline int lpt_get_iclkip(struct drm_i915_private *dev_priv)

-:38: CHECK:LINE_SPACING: Please use a blank line after function/struct/union/enum declarations
#38: FILE: drivers/gpu/drm/i915/display/intel_pch_refclk.h:33:
+}
+static inline int lpt_iclkip(const struct intel_crtc_state *crtc_state)

-:42: CHECK:LINE_SPACING: Please use a blank line after function/struct/union/enum declarations
#42: FILE: drivers/gpu/drm/i915/display/intel_pch_refclk.h:37:
+}
+static inline void intel_init_pch_refclk(struct drm_i915_private *dev_priv)

-:45: CHECK:LINE_SPACING: Please use a blank line after function/struct/union/enum declarations
#45: FILE: drivers/gpu/drm/i915/display/intel_pch_refclk.h:40:
+}
+static inline void lpt_disable_clkout_dp(struct drm_i915_private *dev_priv)

total: 0 errors, 0 warnings, 5 checks, 34 lines checked
64120f26f107 drm/i915/display: add I915 conditional build to intel_pch_display.h
-:33: CHECK:LINE_SPACING: Please use a blank line after function/struct/union/enum declarations
#33: FILE: drivers/gpu/drm/i915/display/intel_pch_display.h:51:
+}
+static inline int intel_crtc_pch_transcoder(struct intel_crtc *crtc)

-:37: CHECK:LINE_SPACING: Please use a blank line after function/struct/union/enum declarations
#37: FILE: drivers/gpu/drm/i915/display/intel_pch_display.h:55:
+}
+static inline void ilk_pch_pre_enable(struct intel_atomic_state *state,

-:41: CHECK:LINE_SPACING: Please use a blank line after function/struct/union/enum declarations
#41: FILE: drivers/gpu/drm/i915/display/intel_pch_display.h:59:
+}
+static inline void ilk_pch_enable(struct intel_atomic_state *state,

-:45: CHECK:LINE_SPACING: Please use a blank line after function/struct/union/enum declarations
#45: FILE: drivers/gpu/drm/i915/display/intel_pch_display.h:63:
+}
+static inline void ilk_pch_disable(struct intel_atomic_state *state,

-:49: CHECK:LINE_SPACING: Please use a blank line after function/struct/union/enum declarations
#49: FILE: drivers/gpu/drm/i915/display/intel_pch_display.h:67:
+}
+static inline void ilk_pch_post_disable(struct intel_atomic_state *state,

-:53: CHECK:LINE_SPACING: Please use a blank line after function/struct/union/enum declarations
#53: FILE: drivers/gpu/drm/i915/display/intel_pch_display.h:71:
+}
+static inline void ilk_pch_get_config(struct intel_crtc_state *crtc_state)

-:56: CHECK:LINE_SPACING: Please use a blank line after function/struct/union/enum declarations
#56: FILE: drivers/gpu/drm/i915/display/intel_pch_display.h:74:
+}
+static inline void lpt_pch_enable(struct intel_atomic_state *state,

-:60: CHECK:LINE_SPACING: Please use a blank line after function/struct/union/enum declarations
#60: FILE: drivers/gpu/drm/i915/display/intel_pch_display.h:78:
+}
+static inline void lpt_pch_disable(struct intel_atomic_state *state,

-:64: CHECK:LINE_SPACING: Please use a blank line after function/struct/union/enum declarations
#64: FILE: drivers/gpu/drm/i915/display/intel_pch_display.h:82:
+}
+static inline void lpt_pch_get_config(struct intel_crtc_state *crtc_state)

-:67: CHECK:LINE_SPACING: Please use a blank line after function/struct/union/enum declarations
#67: FILE: drivers/gpu/drm/i915/display/intel_pch_display.h:85:
+}
+static inline void intel_pch_transcoder_get_m1_n1(struct intel_crtc *crtc,

-:71: CHECK:LINE_SPACING: Please use a blank line after function/struct/union/enum declarations
#71: FILE: drivers/gpu/drm/i915/display/intel_pch_display.h:89:
+}
+static inline void intel_pch_transcoder_get_m2_n2(struct intel_crtc *crtc,

-:75: CHECK:LINE_SPACING: Please use a blank line after function/struct/union/enum declarations
#75: FILE: drivers/gpu/drm/i915/display/intel_pch_display.h:93:
+}
+static inline void intel_pch_sanitize(struct drm_i915_private *i915)

total: 0 errors, 0 warnings, 12 checks, 64 lines checked
ef9a2c8cbd58 drm/i915/display: add I915 conditional build to intel_sprite.h
932758835aca drm/i915/display: add I915 conditional build to intel_overlay.h
-:31: CHECK:LINE_SPACING: Please use a blank line after function/struct/union/enum declarations
#31: FILE: drivers/gpu/drm/i915/display/intel_overlay.h:33:
+}
+static inline void intel_overlay_cleanup(struct drm_i915_private *dev_priv)

-:34: CHECK:LINE_SPACING: Please use a blank line after function/struct/union/enum declarations
#34: FILE: drivers/gpu/drm/i915/display/intel_overlay.h:36:
+}
+static inline int intel_overlay_switch_off(struct intel_overlay *overlay)

-:38: CHECK:LINE_SPACING: Please use a blank line after function/struct/union/enum declarations
#38: FILE: drivers/gpu/drm/i915/display/intel_overlay.h:40:
+}
+static inline int intel_overlay_put_image_ioctl(struct drm_device *dev, void *data,

-:39: CHECK:PARENTHESIS_ALIGNMENT: Alignment should match open parenthesis
#39: FILE: drivers/gpu/drm/i915/display/intel_overlay.h:41:
+static inline int intel_overlay_put_image_ioctl(struct drm_device *dev, void *data,
+				  struct drm_file *file_priv)

-:43: CHECK:LINE_SPACING: Please use a blank line after function/struct/union/enum declarations
#43: FILE: drivers/gpu/drm/i915/display/intel_overlay.h:45:
+}
+static inline int intel_overlay_attrs_ioctl(struct drm_device *dev, void *data,

-:48: CHECK:LINE_SPACING: Please use a blank line after function/struct/union/enum declarations
#48: FILE: drivers/gpu/drm/i915/display/intel_overlay.h:50:
+}
+static inline void intel_overlay_reset(struct drm_i915_private *dev_priv)

-:51: CHECK:LINE_SPACING: Please use a blank line after function/struct/union/enum declarations
#51: FILE: drivers/gpu/drm/i915/display/intel_overlay.h:53:
+}
+static inline struct intel_overlay_error_state *

-:56: CHECK:LINE_SPACING: Please use a blank line after function/struct/union/enum declarations
#56: FILE: drivers/gpu/drm/i915/display/intel_overlay.h:58:
+}
+static inline void intel_overlay_print_error_state(struct drm_i915_error_state_buf *e,

total: 0 errors, 0 warnings, 8 checks, 46 lines checked
36381483b96f drm/i915/display: add I915 conditional build to g4x_dp.h
-:31: CHECK:LINE_SPACING: Please use a blank line after function/struct/union/enum declarations
#31: FILE: drivers/gpu/drm/i915/display/g4x_dp.h:35:
+}
+static inline int vlv_active_pipe(struct intel_dp *intel_dp)

-:35: CHECK:LINE_SPACING: Please use a blank line after function/struct/union/enum declarations
#35: FILE: drivers/gpu/drm/i915/display/g4x_dp.h:39:
+}
+static inline void g4x_dp_set_clock(struct intel_encoder *encoder,

-:39: CHECK:LINE_SPACING: Please use a blank line after function/struct/union/enum declarations
#39: FILE: drivers/gpu/drm/i915/display/g4x_dp.h:43:
+}
+static inline bool g4x_dp_port_enabled(struct drm_i915_private *dev_priv,

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

total: 0 errors, 0 warnings, 4 checks, 37 lines checked
5f3e7c3412a1 drm/i915/display: add I915 conditional build to intel_dpio_phy.h
-:32: CHECK:LINE_SPACING: Please use a blank line after function/struct/union/enum declarations
#32: FILE: drivers/gpu/drm/i915/display/intel_dpio_phy.h:79:
+}
+static inline void bxt_ddi_phy_set_signal_levels(struct intel_encoder *encoder,

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

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

-:42: CHECK:LINE_SPACING: Please use a blank line after function/struct/union/enum declarations
#42: FILE: drivers/gpu/drm/i915/display/intel_dpio_phy.h:89:
+}
+static inline bool bxt_ddi_phy_is_enabled(struct drm_i915_private *dev_priv,

-:47: CHECK:LINE_SPACING: Please use a blank line after function/struct/union/enum declarations
#47: FILE: drivers/gpu/drm/i915/display/intel_dpio_phy.h:94:
+}
+static inline bool bxt_ddi_phy_verify_state(struct drm_i915_private *dev_priv,

-:52: CHECK:LINE_SPACING: Please use a blank line after function/struct/union/enum declarations
#52: FILE: drivers/gpu/drm/i915/display/intel_dpio_phy.h:99:
+}
+static inline u8 bxt_ddi_phy_calc_lane_lat_optim_mask(u8 lane_count)

-:56: CHECK:LINE_SPACING: Please use a blank line after function/struct/union/enum declarations
#56: FILE: drivers/gpu/drm/i915/display/intel_dpio_phy.h:103:
+}
+static inline void bxt_ddi_phy_set_lane_optim_mask(struct intel_encoder *encoder,

-:60: CHECK:LINE_SPACING: Please use a blank line after function/struct/union/enum declarations
#60: FILE: drivers/gpu/drm/i915/display/intel_dpio_phy.h:107:
+}
+static inline u8 bxt_ddi_phy_get_lane_lat_optim_mask(struct intel_encoder *encoder)

-:64: CHECK:LINE_SPACING: Please use a blank line after function/struct/union/enum declarations
#64: FILE: drivers/gpu/drm/i915/display/intel_dpio_phy.h:111:
+}
+static inline enum dpio_channel vlv_dig_port_to_channel(struct intel_digital_port *dig_port)

-:68: CHECK:LINE_SPACING: Please use a blank line after function/struct/union/enum declarations
#68: FILE: drivers/gpu/drm/i915/display/intel_dpio_phy.h:115:
+}
+static inline enum dpio_phy vlv_dig_port_to_phy(struct intel_digital_port *dig_port)

-:72: CHECK:LINE_SPACING: Please use a blank line after function/struct/union/enum declarations
#72: FILE: drivers/gpu/drm/i915/display/intel_dpio_phy.h:119:
+}
+static inline enum dpio_channel vlv_pipe_to_channel(enum pipe pipe)

-:76: CHECK:LINE_SPACING: Please use a blank line after function/struct/union/enum declarations
#76: FILE: drivers/gpu/drm/i915/display/intel_dpio_phy.h:123:
+}
+static inline void chv_set_phy_signal_level(struct intel_encoder *encoder,

-:82: CHECK:LINE_SPACING: Please use a blank line after function/struct/union/enum declarations
#82: FILE: drivers/gpu/drm/i915/display/intel_dpio_phy.h:129:
+}
+static inline void chv_data_lane_soft_reset(struct intel_encoder *encoder,

-:87: CHECK:LINE_SPACING: Please use a blank line after function/struct/union/enum declarations
#87: FILE: drivers/gpu/drm/i915/display/intel_dpio_phy.h:134:
+}
+static inline void chv_phy_pre_pll_enable(struct intel_encoder *encoder,

-:91: CHECK:LINE_SPACING: Please use a blank line after function/struct/union/enum declarations
#91: FILE: drivers/gpu/drm/i915/display/intel_dpio_phy.h:138:
+}
+static inline void chv_phy_pre_encoder_enable(struct intel_encoder *encoder,

-:95: CHECK:LINE_SPACING: Please use a blank line after function/struct/union/enum declarations
#95: FILE: drivers/gpu/drm/i915/display/intel_dpio_phy.h:142:
+}
+static inline void chv_phy_release_cl2_override(struct intel_encoder *encoder)

-:98: CHECK:LINE_SPACING: Please use a blank line after function/struct/union/enum declarations
#98: FILE: drivers/gpu/drm/i915/display/intel_dpio_phy.h:145:
+}
+static inline void chv_phy_post_pll_disable(struct intel_encoder *encoder,

-:109: CHECK:LINE_SPACING: Please use a blank line after function/struct/union/enum declarations
#109: FILE: drivers/gpu/drm/i915/display/intel_dpio_phy.h:156:
+}
+static inline void vlv_phy_pre_pll_enable(struct intel_encoder *encoder,

-:113: CHECK:LINE_SPACING: Please use a blank line after function/struct/union/enum declarations
#113: FILE: drivers/gpu/drm/i915/display/intel_dpio_phy.h:160:
+}
+static inline void vlv_phy_pre_encoder_enable(struct intel_encoder *encoder,

-:117: CHECK:LINE_SPACING: Please use a blank line after function/struct/union/enum declarations
#117: FILE: drivers/gpu/drm/i915/display/intel_dpio_phy.h:164:
+}
+static inline void vlv_phy_reset_lanes(struct intel_encoder *encoder,

total: 0 errors, 0 warnings, 20 checks, 107 lines checked
cc3672160057 drm/i915/display: add I915 conditional build to intel_crt.h
-:29: CHECK:LINE_SPACING: Please use a blank line after function/struct/union/enum declarations
#29: FILE: drivers/gpu/drm/i915/display/intel_crt.h:26:
+}
+static inline void intel_crt_init(struct drm_i915_private *dev_priv)

-:32: CHECK:LINE_SPACING: Please use a blank line after function/struct/union/enum declarations
#32: FILE: drivers/gpu/drm/i915/display/intel_crt.h:29:
+}
+static inline void intel_crt_reset(struct drm_encoder *encoder)

total: 0 errors, 0 warnings, 2 checks, 23 lines checked
01cfb3f97e2e drm/i915/display: add I915 conditional build to vlv_dsi.h
-:26: CHECK:LINE_SPACING: Please use a blank line after function/struct/union/enum declarations
#26: FILE: drivers/gpu/drm/i915/display/vlv_dsi.h:23:
+}
+static inline enum mipi_dsi_pixel_format pixel_format_from_register_bits(u32 fmt)

-:30: CHECK:LINE_SPACING: Please use a blank line after function/struct/union/enum declarations
#30: FILE: drivers/gpu/drm/i915/display/vlv_dsi.h:27:
+}
+static inline void vlv_dsi_init(struct drm_i915_private *dev_priv)

total: 0 errors, 0 warnings, 2 checks, 21 lines checked
32958b459b1f drm/i915/display: add I915 conditional build to i9xx_wm.h
-:28: CHECK:LINE_SPACING: Please use a blank line after function/struct/union/enum declarations
#28: FILE: drivers/gpu/drm/i915/display/i9xx_wm.h:25:
+}
+static inline void ilk_wm_sanitize(struct drm_i915_private *i915)

-:31: CHECK:LINE_SPACING: Please use a blank line after function/struct/union/enum declarations
#31: FILE: drivers/gpu/drm/i915/display/i9xx_wm.h:28:
+}
+static inline bool intel_set_memory_cxsr(struct drm_i915_private *i915, bool enable)

-:35: CHECK:LINE_SPACING: Please use a blank line after function/struct/union/enum declarations
#35: FILE: drivers/gpu/drm/i915/display/i9xx_wm.h:32:
+}
+static inline void i9xx_wm_init(struct drm_i915_private *i915)

total: 0 errors, 0 warnings, 3 checks, 26 lines checked
28f5fd631911 drm/i915/display: add I915 conditional build to g4x_hdmi.h
-:28: CHECK:LINE_SPACING: Please use a blank line after function/struct/union/enum declarations
#28: FILE: drivers/gpu/drm/i915/display/g4x_hdmi.h:28:
+}
+static inline int g4x_hdmi_connector_atomic_check(struct drm_connector *connector,

total: 0 errors, 0 warnings, 1 checks, 21 lines checked
0e6bc53de51f drm/i915/display: add I915 conditional build to intel_dvo.h
9f1b9f41ce43 drm/i915/display: add I915 conditional build to intel_sdvo.h
-:29: CHECK:LINE_SPACING: Please use a blank line after function/struct/union/enum declarations
#29: FILE: drivers/gpu/drm/i915/display/intel_sdvo.h:28:
+}
+static inline bool intel_sdvo_init(struct drm_i915_private *dev_priv,

total: 0 errors, 0 warnings, 1 checks, 22 lines checked
f5116722d649 drm/i915/display: add I915 conditional build to intel_tv.h
b1d4bd296c0f drm/i915/display: add I915 conditional build to vlv_dsi_pll.h
-:25: CHECK:LINE_SPACING: Please use a blank line after function/struct/union/enum declarations
#25: FILE: drivers/gpu/drm/i915/display/vlv_dsi_pll.h:42:
+}
+static inline void assert_dsi_pll_disabled(struct drm_i915_private *i915)

total: 0 errors, 0 warnings, 1 checks, 16 lines checked





[Index of Archives]     [AMD Graphics]     [Linux USB Devel]     [Linux Audio Users]     [Yosemite News]     [Linux Kernel]     [Linux SCSI]

  Powered by Linux