✗ Fi.CI.CHECKPATCH: warning for drm/i915/mst: cleanups, renames, clarifications (rev2)

Patchwork patchwork at emeril.freedesktop.org
Fri Nov 8 10:07:16 UTC 2024


== Series Details ==

Series: drm/i915/mst: cleanups, renames, clarifications (rev2)
URL   : https://patchwork.freedesktop.org/series/141068/
State : warning

== Summary ==

Error: dim checkpatch failed
007f6f5fb61b drm/i915/mst: pass primary encoder to primary encoder hooks
-:21: WARNING:LONG_LINE: line length of 104 exceeds 100 columns
#21: FILE: drivers/gpu/drm/i915/display/intel_dp_mst.c:1116:
+		dig_port->base.post_pll_disable(state, &dig_port->base, old_crtc_state, old_conn_state);

total: 0 errors, 1 warnings, 0 checks, 8 lines checked
e1ad605c2348 drm/i915/mst: rename intel_encoder to encoder
51b461cd4e00 drm/i915/mst: introduce to_primary_encoder() and to_primary_dp()
-:153: WARNING:LONG_LINE: line length of 106 exceeds 100 columns
#153: FILE: drivers/gpu/drm/i915/display/intel_dp_mst.c:1126:
+		primary_encoder->post_pll_disable(state, primary_encoder, old_crtc_state, old_conn_state);

total: 0 errors, 1 warnings, 0 checks, 231 lines checked
23fc6c6584f1 drm/i915/mst: use primary_encoder in fake mst encoder creation
853e9b07823f drm/i915/display: make CHICKEN_TRANS() display version aware
-:226: ERROR:COMPLEX_MACRO: Macros with complex values should be enclosed in parentheses
#226: FILE: drivers/gpu/drm/i915/i915_reg.h:2805:
+#define _CHICKEN_TRANS(trans)	_MMIO(_PICK((trans), \
 					    [TRANSCODER_EDP] = _CHICKEN_TRANS_EDP, \
 					    [TRANSCODER_A] = _CHICKEN_TRANS_A, \
 					    [TRANSCODER_B] = _CHICKEN_TRANS_B, \

-:238: WARNING:LONG_LINE: line length of 120 exceeds 100 columns
#238: FILE: drivers/gpu/drm/i915/i915_reg.h:2816:
+#define CHICKEN_TRANS(display, trans)	(DISPLAY_VER(display) >= 14 ? _MTL_CHICKEN_TRANS(trans) : _CHICKEN_TRANS(trans))

-:238: CHECK:MACRO_ARG_REUSE: Macro argument reuse 'trans' - possible side-effects?
#238: FILE: drivers/gpu/drm/i915/i915_reg.h:2816:
+#define CHICKEN_TRANS(display, trans)	(DISPLAY_VER(display) >= 14 ? _MTL_CHICKEN_TRANS(trans) : _CHICKEN_TRANS(trans))

total: 1 errors, 1 warnings, 1 checks, 176 lines checked
412559e20a41 drm/i915/mst: convert to struct intel_display
0531137ecb3e drm/i915/mst: change naming from fake encoders to MST stream encoders
10626c6b1c1f drm/i915/dp: add comments about hooks called from MST stream encoders




More information about the Intel-gfx mailing list