[Intel-gfx] ✗ Fi.CI.CHECKPATCH: warning for drm/i915/tc: Add a workaround for an IOM/TCSS firmware hang issue (rev2)

Patchwork patchwork at emeril.freedesktop.org
Thu May 4 00:21:02 UTC 2023


== Series Details ==

Series: drm/i915/tc: Add a workaround for an IOM/TCSS firmware hang issue (rev2)
URL   : https://patchwork.freedesktop.org/series/117004/
State : warning

== Summary ==

Error: dim checkpatch failed
c0e23e703186 drm/i915: Fix PIPEDMC disabling for a bigjoiner configuration
6288bedac45a drm/i915: Make the CRTC state consistent during sanitize-disabling
9d06ba0378f5 drm/i915: Update connector atomic state before crtc sanitize-disabling
911d807fe5cb drm/i915: Separate intel_crtc_disable_noatomic_begin/complete()
27a7a66bf8ad drm/i915: Factor out set_encoder_for_connector()
30ef074e0f77 drm/i915: Add support for disabling any CRTCs during HW readout/sanitization
16c5654f45a5 drm/i915/dp: Add link training debug and error printing helpers
-:269: CHECK:PARENTHESIS_ALIGNMENT: Alignment should match open parenthesis
#269: FILE: drivers/gpu/drm/i915/display/intel_dp_link_training.c:725:
+		lt_dbg(intel_dp, DP_PHY_DPRX,
+			    "Using LINK_RATE_SET value %02x\n",

total: 0 errors, 0 warnings, 1 checks, 778 lines checked
04bb39f61ef4 drm/i915/dp: Convert link training error to debug message on disconnected sink
78e39d21dd0a drm/i915/dp: Prevent link training fallback on disconnected port
501ae18f29b3 drm/i915/dp: Factor out intel_dp_get_active_pipes()
d43245ee5705 drm/i915: Factor out a helper for handling atomic modeset locks/state
Traceback (most recent call last):
  File "scripts/spdxcheck.py", line 6, in <module>
    from ply import lex, yacc
ModuleNotFoundError: No module named 'ply'
Traceback (most recent call last):
  File "scripts/spdxcheck.py", line 6, in <module>
    from ply import lex, yacc
ModuleNotFoundError: No module named 'ply'
-:13: WARNING:COMMIT_LOG_LONG_LINE: Possible unwrapped commit description (prefer a maximum 75 chars per line)
#13: 
https://lore.kernel.org/all/20210715184954.7794-2-ville.syrjala@linux.intel.com/

-:85: WARNING:FILE_PATH_CHANGES: added, moved or deleted file(s), does MAINTAINERS need updating?
#85: 
new file mode 100644

-:173: CHECK:MACRO_ARG_REUSE: Macro argument reuse 'ctx' - possible side-effects?
#173: FILE: drivers/gpu/drm/i915/display/intel_modeset_lock.h:28:
+#define intel_modeset_lock_ctx_retry(ctx, state, flags, ret) \
+	for (_intel_modeset_lock_begin((ctx), (state), (flags), &(ret)); \
+	     _intel_modeset_lock_loop(&(ret)); \
+	     _intel_modeset_lock_end((ctx), (state), &(ret)))

-:173: CHECK:MACRO_ARG_REUSE: Macro argument reuse 'state' - possible side-effects?
#173: FILE: drivers/gpu/drm/i915/display/intel_modeset_lock.h:28:
+#define intel_modeset_lock_ctx_retry(ctx, state, flags, ret) \
+	for (_intel_modeset_lock_begin((ctx), (state), (flags), &(ret)); \
+	     _intel_modeset_lock_loop(&(ret)); \
+	     _intel_modeset_lock_end((ctx), (state), &(ret)))

-:173: CHECK:MACRO_ARG_REUSE: Macro argument reuse 'ret' - possible side-effects?
#173: FILE: drivers/gpu/drm/i915/display/intel_modeset_lock.h:28:
+#define intel_modeset_lock_ctx_retry(ctx, state, flags, ret) \
+	for (_intel_modeset_lock_begin((ctx), (state), (flags), &(ret)); \
+	     _intel_modeset_lock_loop(&(ret)); \
+	     _intel_modeset_lock_end((ctx), (state), &(ret)))

total: 0 errors, 2 warnings, 3 checks, 125 lines checked
edd95b07db89 drm/i915/tc: Reset TypeC PHYs left enabled in DP-alt mode after the sink disconnects
-:48: WARNING:COMMIT_LOG_USE_LINK: Unknown link reference 'Closes:', use 'Link:' instead
#48: 
Closes: https://gitlab.freedesktop.org/drm/intel/-/issues/5860

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




More information about the Intel-gfx mailing list