[Intel-gfx] ✗ Fi.CI.CHECKPATCH: warning for In order to readout DP SDPs, refactors the handling of DP SDPs (rev14)

Patchwork patchwork at emeril.freedesktop.org
Thu May 14 06:24:43 UTC 2020


== Series Details ==

Series: In order to readout DP SDPs, refactors the handling of DP SDPs (rev14)
URL   : https://patchwork.freedesktop.org/series/72853/
State : warning

== Summary ==

$ dim checkpatch origin/drm-tip
e1053fdca03e video/hdmi: Add Unpack only function for DRM infoframe
0a10f4cd6cbe drm/i915/dp: Read out DP SDPs
bf5fbd357375 drm: Add logging function for DP VSC SDP
b53f926b3fa5 drm/i915: Include HDMI DRM infoframe in the crtc state dump
af8b261a6986 drm/i915: Include DP HDR Metadata Infoframe SDP in the crtc state dump
2375c1f7c8e0 drm/i915: Include DP VSC SDP in the crtc state dump
c7d93732395b drm/i915: Program DP SDPs with computed configs
1723115921c6 drm/i915: Add state readout for DP HDR Metadata Infoframe SDP
99bcd01ba088 drm/i915: Add state readout for DP VSC SDP
-:83: CHECK:MACRO_ARG_REUSE: Macro argument reuse 'name' - possible side-effects?
#83: FILE: drivers/gpu/drm/i915/display/intel_display.c:13750:
+#define PIPE_CONF_CHECK_DP_VSC_SDP(name) do { \
+	if (!current_config->has_psr && !pipe_config->has_psr && \
+	    !intel_compare_dp_vsc_sdp(&current_config->infoframes.name, \
+				      &pipe_config->infoframes.name)) { \
+		pipe_config_dp_vsc_sdp_mismatch(dev_priv, fastset, __stringify(name), \
+						&current_config->infoframes.name, \
+						&pipe_config->infoframes.name); \
+		ret = false; \
+	} \
+} while (0)

total: 0 errors, 0 warnings, 1 checks, 75 lines checked
e373569ff8b1 drm/i915: Fix enabled infoframe states of lspcon
5ac865ae0f3c drm/i915: Program DP SDPs on pipe updates
f8d2eda49214 drm/i915: Stop sending DP SDPs on ddi disable
6d1aa4db1cc0 drm/i915/dp: Add compute routine for DP PSR VSC SDP
b391c6a63abc drm/i915/psr: Use new DP VSC SDP compute routine on PSR



More information about the Intel-gfx mailing list