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

Patchwork patchwork at emeril.freedesktop.org
Thu May 7 14:16:04 UTC 2020


== Series Details ==

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

== Summary ==

$ dim checkpatch origin/drm-tip
96f3abd228ee video/hdmi: Add Unpack only function for DRM infoframe
2ad5c4085132 drm/i915/dp: Read out DP SDPs
e32e44f11938 drm: Add logging function for DP VSC SDP
ee8f99248537 drm/i915: Include HDMI DRM infoframe in the crtc state dump
20cdec4dde7d drm/i915: Include DP HDR Metadata Infoframe SDP in the crtc state dump
ffb06944568a drm/i915: Include DP VSC SDP in the crtc state dump
0c6956c780de drm/i915: Program DP SDPs with computed configs
e1c3006e8068 drm/i915: Add state readout for DP HDR Metadata Infoframe SDP
84bf9208397c 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
a71792fd1d5a drm/i915: Fix enabled infoframe states of lspcon
0251102af3b9 drm/i915: Program DP SDPs on pipe updates
5cf81085a0c1 drm/i915: Stop sending DP SDPs on ddi disable
74c2a914a012 drm/i915/dp: Add compute routine for DP PSR VSC SDP
21e992a96510 drm/i915/psr: Use new DP VSC SDP compute routine on PSR
-:107: WARNING:TABSTOP: Statements should start on a tabstop
#107: FILE: drivers/gpu/drm/i915/display/intel_psr.c:727:
+	 if (!psr_global_enabled(dev_priv))

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



More information about the Intel-gfx mailing list