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

Patchwork patchwork at emeril.freedesktop.org
Fri Apr 17 15:21:38 UTC 2020


== Series Details ==

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

== Summary ==

$ dim checkpatch origin/drm-tip
04a7a12971b4 video/hdmi: Add Unpack only function for DRM infoframe
89d92042f6da drm/i915/dp: Read out DP SDPs
69460e7ad9d1 drm: Add logging function for DP VSC SDP
808aab8c443c drm/i915: Include HDMI DRM infoframe in the crtc state dump
8ec365f17f48 drm/i915: Include DP HDR Metadata Infoframe SDP in the crtc state dump
578da91e2148 drm/i915: Include DP VSC SDP in the crtc state dump
0754e9ee06df drm/i915: Program DP SDPs with computed configs
22a5e5bca724 drm/i915: Add state readout for DP HDR Metadata Infoframe SDP
b47e402f86e9 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:13728:
+#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
65c3865381d1 drm/i915: Fix enabled infoframe states of lspcon
4a022e8fc205 drm/i915: Program DP SDPs on pipe updates
b1e535972161 drm/i915: Stop sending DP SDPs on ddi disable
ca6476d66567 drm/i915/dp: Add compute routine for DP PSR VSC SDP
683a7302b245 drm/i915/psr: Use new DP VSC SDP compute routine on PSR



More information about the Intel-gfx mailing list