== Series Details == Series: drm/i915/display: make all i915_drv.h includes explicit URL : https://patchwork.freedesktop.org/series/140764/ State : warning == Summary == Error: dim checkpatch failed 76dd21274c35 drm/i915/gvt: always pass struct intel_display * to register macros -:448: CHECK:MACRO_ARG_REUSE: Macro argument reuse 'display' - possible side-effects? #448: FILE: drivers/gpu/drm/i915/gvt/handlers.c:1015: +#define DSPSURF_TO_PIPE(display, offset) \ + calc_index(offset, DSPSURF(display, PIPE_A), DSPSURF(display, PIPE_B), DSPSURF(display, PIPE_C)) -:449: WARNING:LONG_LINE: line length of 104 exceeds 100 columns #449: FILE: drivers/gpu/drm/i915/gvt/handlers.c:1016: + calc_index(offset, DSPSURF(display, PIPE_A), DSPSURF(display, PIPE_B), DSPSURF(display, PIPE_C)) total: 0 errors, 1 warnings, 1 checks, 452 lines checked be4629dd5a92 drm/i915: extract intel_uncore_trace.[ch] -:117: WARNING:FILE_PATH_CHANGES: added, moved or deleted file(s), does MAINTAINERS need updating? #117: new file mode 100644 -:150: CHECK:PARENTHESIS_ALIGNMENT: Alignment should match open parenthesis #150: FILE: drivers/gpu/drm/i915/intel_uncore_trace.h:16: +TRACE_EVENT_CONDITION(i915_reg_rw, + TP_PROTO(bool write, i915_reg_t reg, u64 val, int len, bool trace), -:156: CHECK:OPEN_ENDED_LINE: Lines should not end with a '(' #156: FILE: drivers/gpu/drm/i915/intel_uncore_trace.h:22: + TP_STRUCT__entry( -:163: CHECK:OPEN_ENDED_LINE: Lines should not end with a '(' #163: FILE: drivers/gpu/drm/i915/intel_uncore_trace.h:29: + TP_fast_assign( -:171: CHECK:PARENTHESIS_ALIGNMENT: Alignment should match open parenthesis #171: FILE: drivers/gpu/drm/i915/intel_uncore_trace.h:37: + TP_printk("%s reg=0x%x, len=%d, val=(0x%x, 0x%x)", + __entry->write ? "write" : "read", -:181: CHECK:SPACING: spaces preferred around that '/' (ctx:VxV) #181: FILE: drivers/gpu/drm/i915/intel_uncore_trace.h:47: +#define TRACE_INCLUDE_PATH ../../drivers/gpu/drm/i915 ^ -:181: CHECK:SPACING: spaces preferred around that '/' (ctx:VxV) #181: FILE: drivers/gpu/drm/i915/intel_uncore_trace.h:47: +#define TRACE_INCLUDE_PATH ../../drivers/gpu/drm/i915 ^ -:181: CHECK:SPACING: spaces preferred around that '/' (ctx:VxV) #181: FILE: drivers/gpu/drm/i915/intel_uncore_trace.h:47: +#define TRACE_INCLUDE_PATH ../../drivers/gpu/drm/i915 ^ -:181: CHECK:SPACING: spaces preferred around that '/' (ctx:VxV) #181: FILE: drivers/gpu/drm/i915/intel_uncore_trace.h:47: +#define TRACE_INCLUDE_PATH ../../drivers/gpu/drm/i915 ^ -:181: CHECK:SPACING: spaces preferred around that '/' (ctx:VxV) #181: FILE: drivers/gpu/drm/i915/intel_uncore_trace.h:47: +#define TRACE_INCLUDE_PATH ../../drivers/gpu/drm/i915 ^ total: 0 errors, 1 warnings, 9 checks, 137 lines checked 63fe30681d3c drm/i915/display: add intel_display_conversion.c to hide stuff better -:32: WARNING:FILE_PATH_CHANGES: added, moved or deleted file(s), does MAINTAINERS need updating? #32: new file mode 100644 -:67: WARNING:LONG_LINE: line length of 102 exceeds 100 columns #67: FILE: drivers/gpu/drm/i915/display/intel_display_conversion.h:22: + const struct drm_i915_private *: __i915_to_display((struct drm_i915_private *)(p)), \ -:67: CHECK:SPACING: spaces preferred around that '*' (ctx:WxO) #67: FILE: drivers/gpu/drm/i915/display/intel_display_conversion.h:22: + const struct drm_i915_private *: __i915_to_display((struct drm_i915_private *)(p)), \ ^ -:67: ERROR:SPACING: spaces required around that ':' (ctx:OxW) #67: FILE: drivers/gpu/drm/i915/display/intel_display_conversion.h:22: + const struct drm_i915_private *: __i915_to_display((struct drm_i915_private *)(p)), \ ^ -:68: CHECK:SPACING: spaces preferred around that '*' (ctx:WxO) #68: FILE: drivers/gpu/drm/i915/display/intel_display_conversion.h:23: + struct drm_i915_private *: __i915_to_display((struct drm_i915_private *)(p)), \ ^ -:68: ERROR:SPACING: spaces required around that ':' (ctx:OxW) #68: FILE: drivers/gpu/drm/i915/display/intel_display_conversion.h:23: + struct drm_i915_private *: __i915_to_display((struct drm_i915_private *)(p)), \ ^ total: 2 errors, 2 warnings, 2 checks, 44 lines checked c4ef00a6c1d9 drm/i915/uncore: add to_intel_uncore() and use it dbdfc9c82c5e drm/i915/display: add struct drm_device to struct intel_display conversion function -:16: WARNING:COMMIT_LOG_LONG_LINE: Prefer a maximum 75 chars per line (possible unwrapped commit description?) #16: [1] https://lore.kernel.org/r/7777ff70e2be0663de4398aa6f75f0c54146cbfc.1709727127.git.jani.nikula@xxxxxxxxx total: 0 errors, 1 warnings, 0 checks, 36 lines checked 792edd21dd3f drm/i915/display: drop i915_drv.h include from intel_display_trace.h
