== 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