✗ CI.checkpatch: warning for drm/i915/display: add snapshot capture/print infra

[Patchwork]

== Series Details ==

Series: drm/i915/display: add snapshot capture/print infra
URL   : https://patchwork.freedesktop.org/series/138150/
State : warning

== Summary ==

+ KERNEL=/kernel
+ git clone https://gitlab.freedesktop.org/drm/maintainer-tools mt
Cloning into 'mt'...
warning: redirecting to https://gitlab.freedesktop.org/drm/maintainer-tools.git/
+ git -C mt rev-list -n1 origin/master
c62d7e164862503a3662a095da1c6c9014248cb2
+ cd /kernel
+ git config --global --add safe.directory /kernel
+ git log -n1
commit 5470bda738786beac28c1e7997f40e21d89989dd
Author: Jani Nikula <jani.nikula at intel.com>
Date:   Tue Sep 3 17:03:00 2024 +0300

    drm/i915/display: move dmc snapshotting to new display snapshot
    
    Convert dmc error state printing to new snapshot capture/print division.
    
    Signed-off-by: Jani Nikula <jani.nikula at intel.com>
+ /mt/dim checkpatch 7378077b0be32160022a73b7050b963d52364618 drm-intel
62bed4abc65d drm/i915: dump display parameters captured in error state, not current
-:34: WARNING:MACRO_ARG_UNUSED: Argument 'T' is not used in function-like macro
#34: FILE: drivers/gpu/drm/i915/display/intel_display_params.c:185:
+#define PRINT(T, x, ...) _param_print(p, driver_name, #x, params->x);

-:34: CHECK:MACRO_ARG_PRECEDENCE: Macro argument 'x' may be better as '(x)' to avoid precedence issues
#34: FILE: drivers/gpu/drm/i915/display/intel_display_params.c:185:
+#define PRINT(T, x, ...) _param_print(p, driver_name, #x, params->x);

-:34: WARNING:TRAILING_SEMICOLON: macros should not use a trailing semicolon
#34: FILE: drivers/gpu/drm/i915/display/intel_display_params.c:185:
+#define PRINT(T, x, ...) _param_print(p, driver_name, #x, params->x);

total: 0 errors, 2 warnings, 1 checks, 52 lines checked
cdc9a4f738be drm/i915/display: add intel_display_snapshot abstraction
-:28: WARNING:FILE_PATH_CHANGES: added, moved or deleted file(s), does MAINTAINERS need updating?
#28: 
new file mode 100644

-:176: CHECK:LINE_SPACING: Please use a blank line after function/struct/union/enum declarations
#176: FILE: drivers/gpu/drm/i915/display/intel_overlay.h:54:
 }
+static inline struct intel_overlay_snapshot *

-:183: CHECK:LINE_SPACING: Please use a blank line after function/struct/union/enum declarations
#183: FILE: drivers/gpu/drm/i915/display/intel_overlay.h:59:
 }
+static inline void intel_overlay_snapshot_print(const struct intel_overlay_snapshot *error,

total: 0 errors, 1 warnings, 2 checks, 207 lines checked
1ac87318f16d drm/i915/display: move device info and params handling to snapshot
5470bda73878 drm/i915/display: move dmc snapshotting to new display snapshot