✗ Fi.CI.CHECKPATCH: warning for bits: Fixed-type GENMASK_U*() and BIT_U*() (rev2)

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

 



== Series Details ==

Series: bits: Fixed-type GENMASK_U*() and BIT_U*() (rev2)
URL   : https://patchwork.freedesktop.org/series/146088/
State : warning

== Summary ==

Error: dim checkpatch failed
66702fd9fb3b bits: introduce fixed-type GENMASK_U*()
-:89: CHECK:MACRO_ARG_REUSE: Macro argument reuse 't' - possible side-effects?
#89: FILE: include/linux/bits.h:48:
+#define GENMASK_TYPE(t, h, l)					\
+	((t)(GENMASK_INPUT_CHECK(h, l) +			\
+	     (type_max(t) << (l) &				\
+	      type_max(t) >> (BITS_PER_TYPE(t) - 1 - (h)))))

-:89: CHECK:MACRO_ARG_REUSE: Macro argument reuse 'h' - possible side-effects?
#89: FILE: include/linux/bits.h:48:
+#define GENMASK_TYPE(t, h, l)					\
+	((t)(GENMASK_INPUT_CHECK(h, l) +			\
+	     (type_max(t) << (l) &				\
+	      type_max(t) >> (BITS_PER_TYPE(t) - 1 - (h)))))

-:89: CHECK:MACRO_ARG_REUSE: Macro argument reuse 'l' - possible side-effects?
#89: FILE: include/linux/bits.h:48:
+#define GENMASK_TYPE(t, h, l)					\
+	((t)(GENMASK_INPUT_CHECK(h, l) +			\
+	     (type_max(t) << (l) &				\
+	      type_max(t) >> (BITS_PER_TYPE(t) - 1 - (h)))))

total: 0 errors, 0 warnings, 3 checks, 66 lines checked
78a9e8e9feff bits: introduce fixed-type BIT_U*()
-:42: CHECK:MACRO_ARG_REUSE: Macro argument reuse 'nr' - possible side-effects?
#42: FILE: include/linux/bits.h:69:
+#define BIT_TYPE(type, nr) ((type)(BIT_INPUT_CHECK(type, nr) + BIT_ULL(nr)))

total: 0 errors, 0 warnings, 1 checks, 32 lines checked
2c7dada269ed drm/i915: Convert REG_GENMASK*() to fixed-width GENMASK_U*()
300efb11156e test_bits: add tests for GENMASK_U*()
-:26: ERROR:SPACING: spaces required around that ':' (ctx:VxW)
#26: FILE: lib/test_bits.c:10:
+#define assert_type(t, x) _Generic(x, t: x, default: 0)
                                        ^

-:26: ERROR:SPACING: spaces required around that ':' (ctx:VxW)
#26: FILE: lib/test_bits.c:10:
+#define assert_type(t, x) _Generic(x, t: x, default: 0)
                                                    ^

-:26: CHECK:MACRO_ARG_REUSE: Macro argument reuse 'x' - possible side-effects?
#26: FILE: lib/test_bits.c:10:
+#define assert_type(t, x) _Generic(x, t: x, default: 0)

total: 2 errors, 0 warnings, 1 checks, 37 lines checked
56e1dfba3900 test_bits: add tests for BIT_U*()





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

  Powered by Linux