== Series Details == Series: drm/i915: i915_display_info cleanup URL : https://patchwork.freedesktop.org/series/70222/ State : warning == Summary == $ dim checkpatch origin/drm-tip 6356e0862ab6 drm/i915: Use drm_rect to simplify plane {crtc, src}_{x, y, w, h} printing -:41: WARNING:LONG_LINE: line over 100 characters #41: FILE: drivers/gpu/drm/i915/i915_debugfs.c:2624: + seq_printf(m, "\t--Plane id %d: type=%s, dst=" DRM_RECT_FMT ", src=" DRM_RECT_FP_FMT ", format=%s, rotation=%s\n", total: 0 errors, 1 warnings, 0 checks, 38 lines checked 2a80e4622b63 drm/i915: Switch to intel_ types in debugfs display_info b62404023212 drm/i915: Reorganize plane/fb dump in debugfs -:62: WARNING:LONG_LINE: line over 100 characters #62: FILE: drivers/gpu/drm/i915/i915_debugfs.c:2601: + seq_printf(m, "\t--Plane id %d: type=%s, fb=%d,%s,%dx%d, src=" DRM_RECT_FP_FMT ", dst=" DRM_RECT_FMT ", rotation=%s\n", total: 0 errors, 1 warnings, 0 checks, 80 lines checked b0243cfbf813 drm/i915: Refactor debugfs display info code 7daef64f127f drm/i915: Dump the mode for the crtc just the once cb15b4882e11 drm/i915: Use drm_modeset_lock_all() in debugfs display info 1932dba700b8 drm/i915: Use the canonical [CRTC:%d:%s]/etc. format in i915_display_info -:50: WARNING:LONG_LINE: line over 100 characters #50: FILE: drivers/gpu/drm/i915/i915_debugfs.c:2588: + seq_printf(m, "\t[PLANE:%d:%s]: type=%s, fb=%d,%s,%dx%d, src=" DRM_RECT_FP_FMT ", dst=" DRM_RECT_FMT ", rotation=%s\n", total: 0 errors, 1 warnings, 0 checks, 41 lines checked c8b8d3658b34 drm/i915: Dump both the uapi and hw states for crtcs and planes -:87: WARNING:LONG_LINE: line over 100 characters #87: FILE: drivers/gpu/drm/i915/i915_debugfs.c:2601: + seq_printf(m, "\t\thw: fb=%d,%s,%dx%d, visible=%s, src=" DRM_RECT_FP_FMT ", dst=" DRM_RECT_FMT ", rotation=%s\n", total: 0 errors, 1 warnings, 0 checks, 142 lines checked 108411fb7af8 drm/i915: Stop using connector->encoder and encoder->crtc links in i915_display_info -:24: ERROR:COMPLEX_MACRO: Macros with complex values should be enclosed in parentheses #24: FILE: drivers/gpu/drm/i915/display/intel_display.h:383: +#define for_each_intel_encoder_mask(dev, intel_encoder, encoder_mask) \ + list_for_each_entry(intel_encoder, \ + &(dev)->mode_config.encoder_list, \ + base.head) \ + for_each_if((encoder_mask) & \ + drm_encoder_mask(&intel_encoder->base)) -:24: CHECK:MACRO_ARG_REUSE: Macro argument reuse 'intel_encoder' - possible side-effects? #24: FILE: drivers/gpu/drm/i915/display/intel_display.h:383: +#define for_each_intel_encoder_mask(dev, intel_encoder, encoder_mask) \ + list_for_each_entry(intel_encoder, \ + &(dev)->mode_config.encoder_list, \ + base.head) \ + for_each_if((encoder_mask) & \ + drm_encoder_mask(&intel_encoder->base)) total: 1 errors, 0 warnings, 1 checks, 113 lines checked _______________________________________________ Intel-gfx mailing list Intel-gfx@xxxxxxxxxxxxxxxxxxxxx https://lists.freedesktop.org/mailman/listinfo/intel-gfx