✗ Fi.CI.CHECKPATCH: warning for series starting with [v2,1/2] drm/i915/perf: Add helper macros for comparing with whitelisted registers

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

 



== Series Details ==

Series: series starting with [v2,1/2] drm/i915/perf: Add helper macros for comparing with whitelisted registers
URL   : https://patchwork.freedesktop.org/series/68582/
State : warning

== Summary ==

$ dim checkpatch origin/drm-tip
d9a93528ebbd drm/i915/perf: Add helper macros for comparing with whitelisted registers
-:21: CHECK:MACRO_ARG_REUSE: Macro argument reuse 'addr' - possible side-effects?
#21: FILE: drivers/gpu/drm/i915/i915_perf.c:3518:
+#define ADDR_IN_RANGE(addr, start, end) \
+	((addr) >= (start) && \
+	 (addr) <= (end))

-:25: CHECK:MACRO_ARG_REUSE: Macro argument reuse 'addr' - possible side-effects?
#25: FILE: drivers/gpu/drm/i915/i915_perf.c:3522:
+#define REG_IN_RANGE(addr, start, end) \
+	((addr) >= i915_mmio_reg_offset(start) && \
+	 (addr) <= i915_mmio_reg_offset(end))

total: 0 errors, 0 warnings, 2 checks, 98 lines checked
c4fe94c04553 drm/i915/tgl: Add perf support on TGL
-:132: ERROR:TRAILING_WHITESPACE: trailing whitespace
#132: FILE: drivers/gpu/drm/i915/i915_perf.c:758:
+^I^I^I  (IS_GEN(stream->perf->i915, 12) ? $

-:133: ERROR:TRAILING_WHITESPACE: trailing whitespace
#133: FILE: drivers/gpu/drm/i915/i915_perf.c:759:
+^I^I^I   OAREPORT_REASON_MASK_EXTENDED : $

-:457: CHECK:COMPARISON_TO_NULL: Comparison to NULL could be written "oa_config"
#457: FILE: drivers/gpu/drm/i915/i915_perf.c:2473:
+					    oa_config != NULL);

-:836: WARNING:FILE_PATH_CHANGES: added, moved or deleted file(s), does MAINTAINERS need updating?
#836: 
new file mode 100644

total: 2 errors, 1 warnings, 1 checks, 869 lines checked

_______________________________________________
Intel-gfx mailing list
Intel-gfx@xxxxxxxxxxxxxxxxxxxxx
https://lists.freedesktop.org/mailman/listinfo/intel-gfx




[Index of Archives]     [AMD Graphics]     [Linux USB Devel]     [Linux Audio Users]     [Yosemite News]     [Linux Kernel]     [Linux SCSI]

  Powered by Linux