✗ Fi.CI.CHECKPATCH: warning for Ultrajoiner basic functionality series (rev6)

[Patchwork]

== Series Details ==

Series: Ultrajoiner basic functionality series (rev6)
URL   : https://patchwork.freedesktop.org/series/133800/
State : warning

== Summary ==

Error: dim checkpatch failed
23508b7b4b40 drm/i915/display: Modify debugfs for joiner to force n pipes
a064b31b51cd drm/i915/display: Use joined pipes in intel_dp_joiner_needs_dsc
efceeb539426 drm/i915/display: Use joined pipes in intel_mode_valid_max_plane_size
8ab3c7e4bacb drm/i915/display: Use joined pipes in dsc helpers for slices, bpp
d850eebf97fe drm/i915: Add some essential functionality for joiners
0e4c2974746f drm/i915: Split current joiner hw state readout
8f9ba2421a37 drm/i915: Add bigjoiner and uncompressed joiner hw readout sanity checks
ace849677f05 drm/i915: Implement hw state readout and checks for ultrajoiner
a9c4aef266c5 drm/i915/display/vdsc: Add ultrajoiner support with DSC
98f9585af083 drm/i915: Compute config and mode valid changes for ultrajoiner
cecd47d85a7f drm/i915: Add new abstraction layer to handle pipe order for different joiners
-:237: ERROR:COMPLEX_MACRO: Macros with complex values should be enclosed in parentheses
#237: FILE: drivers/gpu/drm/i915/display/intel_display.h:278:
+#define for_each_intel_crtc_in_mask_priority(__dev_priv, intel_crtc, __p, __mask, __priolist) \
+	for_each_pipe(__dev_priv, __p) \
+		for_each_if((__mask) & BIT(__priolist[__p])) \
+			for_each_if(intel_crtc = intel_crtc_for_pipe(__dev_priv, __priolist[__p]))

-:237: CHECK:MACRO_ARG_REUSE: Macro argument reuse '__dev_priv' - possible side-effects?
#237: FILE: drivers/gpu/drm/i915/display/intel_display.h:278:
+#define for_each_intel_crtc_in_mask_priority(__dev_priv, intel_crtc, __p, __mask, __priolist) \
+	for_each_pipe(__dev_priv, __p) \
+		for_each_if((__mask) & BIT(__priolist[__p])) \
+			for_each_if(intel_crtc = intel_crtc_for_pipe(__dev_priv, __priolist[__p]))

-:237: CHECK:MACRO_ARG_REUSE: Macro argument reuse '__p' - possible side-effects?
#237: FILE: drivers/gpu/drm/i915/display/intel_display.h:278:
+#define for_each_intel_crtc_in_mask_priority(__dev_priv, intel_crtc, __p, __mask, __priolist) \
+	for_each_pipe(__dev_priv, __p) \
+		for_each_if((__mask) & BIT(__priolist[__p])) \
+			for_each_if(intel_crtc = intel_crtc_for_pipe(__dev_priv, __priolist[__p]))

-:237: CHECK:MACRO_ARG_REUSE: Macro argument reuse '__priolist' - possible side-effects?
#237: FILE: drivers/gpu/drm/i915/display/intel_display.h:278:
+#define for_each_intel_crtc_in_mask_priority(__dev_priv, intel_crtc, __p, __mask, __priolist) \
+	for_each_pipe(__dev_priv, __p) \
+		for_each_if((__mask) & BIT(__priolist[__p])) \
+			for_each_if(intel_crtc = intel_crtc_for_pipe(__dev_priv, __priolist[__p]))

total: 1 errors, 0 warnings, 3 checks, 257 lines checked
492b5281d865 drm/i915/intel_dp: Add support for forcing ultrajoiner