✗ Fi.CI.CHECKPATCH: warning for drm/i915/display: add snapshot capture/print infra (rev4)

Patchwork patchwork at emeril.freedesktop.org
Thu Sep 12 19:36:58 UTC 2024


== Series Details ==

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

== Summary ==

Error: dim checkpatch failed
878773ed0332 drm/i915: dump display parameters captured in error state, not current
-:35: WARNING:MACRO_ARG_UNUSED: Argument 'T' is not used in function-like macro
#35: FILE: drivers/gpu/drm/i915/display/intel_display_params.c:185:
+#define PRINT(T, x, ...) _param_print(p, driver_name, #x, params->x);

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

-:35: WARNING:TRAILING_SEMICOLON: macros should not use a trailing semicolon
#35: 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
3a65ca9dabc6 drm/i915/display: add intel_display_snapshot abstraction
-:33: WARNING:FILE_PATH_CHANGES: added, moved or deleted file(s), does MAINTAINERS need updating?
#33: 
new file mode 100644

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

total: 0 errors, 1 warnings, 1 checks, 217 lines checked
de4f3fb05fa5 drm/i915/display: move device info and params handling to snapshot
dc466f6185b3 drm/i915/display: move dmc snapshotting to new display snapshot




More information about the Intel-gfx mailing list