✗ Fi.CI.CHECKPATCH: warning for drm/i915: Bigjoiner modeset sequence redesign and MST support (rev3)

[Patchwork]

== Series Details ==

Series: drm/i915: Bigjoiner modeset sequence redesign and MST support (rev3)
URL   : https://patchwork.freedesktop.org/series/131797/
State : warning

== Summary ==

Error: dim checkpatch failed
64805a6e8087 drm/i915: Update pipes in reverse order for bigjoiner
-:86: CHECK:MACRO_ARG_REUSE: Macro argument reuse '__state' - possible side-effects?
#86: FILE: drivers/gpu/drm/i915/display/intel_display.h:347:
+#define for_each_new_intel_crtc_in_state_reverse(__state, crtc, new_crtc_state, __i) \
+	for ((__i) = (__state)->base.dev->mode_config.num_crtc - 1; \
+	     (__i) >= 0  && \
+	     ((crtc) = to_intel_crtc((__state)->base.crtcs[__i].ptr), \
+	      (new_crtc_state) = to_intel_crtc_state((__state)->base.crtcs[__i].new_state), 1); \
+	     (__i)--) \
+		for_each_if(crtc)

-:86: CHECK:MACRO_ARG_REUSE: Macro argument reuse 'crtc' - possible side-effects?
#86: FILE: drivers/gpu/drm/i915/display/intel_display.h:347:
+#define for_each_new_intel_crtc_in_state_reverse(__state, crtc, new_crtc_state, __i) \
+	for ((__i) = (__state)->base.dev->mode_config.num_crtc - 1; \
+	     (__i) >= 0  && \
+	     ((crtc) = to_intel_crtc((__state)->base.crtcs[__i].ptr), \
+	      (new_crtc_state) = to_intel_crtc_state((__state)->base.crtcs[__i].new_state), 1); \
+	     (__i)--) \
+		for_each_if(crtc)

-:86: CHECK:MACRO_ARG_REUSE: Macro argument reuse '__i' - possible side-effects?
#86: FILE: drivers/gpu/drm/i915/display/intel_display.h:347:
+#define for_each_new_intel_crtc_in_state_reverse(__state, crtc, new_crtc_state, __i) \
+	for ((__i) = (__state)->base.dev->mode_config.num_crtc - 1; \
+	     (__i) >= 0  && \
+	     ((crtc) = to_intel_crtc((__state)->base.crtcs[__i].ptr), \
+	      (new_crtc_state) = to_intel_crtc_state((__state)->base.crtcs[__i].new_state), 1); \
+	     (__i)--) \
+		for_each_if(crtc)

total: 0 errors, 0 warnings, 3 checks, 40 lines checked
109ce10fb958 drm/i915/psr: Disable PSR when bigjoiner is used
092a582e7793 drm/i915: Disable port sync when bigjoiner is used
ab6f55f1c1f9 drm/i915: Disable live M/N updates when using bigjoiner
b8d2a8a7126c drm/i915/vrr: Disable VRR when using bigjoiner
8cf2e8910507 drm/i915: Fix intel_modeset_pipe_config_late() for bigjoiner
31c625608506 drm/i915: s/intel_dp_can_bigjoiner()/intel_dp_has_bigjoiner()/
b75e532ae3f2 drm/i915: Extract intel_dp_joiner_needs_dsc()
43196b2562b6 drm/i915/mst: Check intel_dp_joiner_needs_dsc()
4ed822db08f4 drm/i915: Pass connector to intel_dp_need_bigjoiner()
07fd4e245f9e drm/i915: Introduce intel_crtc_joined_pipe_mask()
77c9494e9ab0 drm/i915: Extract intel_ddi_post_disable_hdmi_or_sst()
0a8a9cd81700 drm/i915: Utilize intel_crtc_joined_pipe_mask() more
fe7a5e821875 drm/i915: Handle joined pipes inside hsw_crtc_disable()
cf1c9bf0f176 drm/i915: Handle joined pipes inside hsw_crtc_enable()
-:11: WARNING:COMMIT_LOG_LONG_LINE: Prefer a maximum 75 chars per line (possible unwrapped commit description?)
#11: 
That way we can also remove a bunch of checks like intel_crtc_is_bigjoiner_slave.

-:320: ERROR:COMPLEX_MACRO: Macros with complex values should be enclosed in parentheses
#320: FILE: drivers/gpu/drm/i915/display/intel_display.h:283:
+#define for_each_intel_crtc_in_pipe_mask_reverse(dev, intel_crtc, pipe_mask)	\
+	list_for_each_entry_reverse((intel_crtc),				\
+				    &(dev)->mode_config.crtc_list,		\
+				    base.head)					\
+		for_each_if((pipe_mask) & BIT((intel_crtc)->pipe))

-:320: CHECK:MACRO_ARG_REUSE: Macro argument reuse 'intel_crtc' - possible side-effects?
#320: FILE: drivers/gpu/drm/i915/display/intel_display.h:283:
+#define for_each_intel_crtc_in_pipe_mask_reverse(dev, intel_crtc, pipe_mask)	\
+	list_for_each_entry_reverse((intel_crtc),				\
+				    &(dev)->mode_config.crtc_list,		\
+				    base.head)					\
+		for_each_if((pipe_mask) & BIT((intel_crtc)->pipe))

total: 1 errors, 1 warnings, 1 checks, 275 lines checked
aedefafbd992 drm/i915/mst: Add bigjoiner handling to MST modeset sequence
68f275de39a5 drm/i915: Allow bigjoiner for MST