[dpdk-test-report] |WARNING| pw38566 [PATCH v2 1/8] net/sfc/base: cope with clang warning on negative shift

checkpatch at dpdk.org checkpatch at dpdk.org
Thu Apr 19 18:42:28 CEST 2018


Test-Label: checkpatch
Test-Status: WARNING
http://dpdk.org/patch/38566

_coding style issues_


CHECK:MACRO_ARG_REUSE: Macro argument reuse '_val' - possible side-effects?
#42: FILE: drivers/net/sfc/base/efx_types.h:338:
+#define	EFX_SSUB(_val, _sub) \
+	((_val) > (_sub) ? ((_val) - (_sub)) : 0)

CHECK:MACRO_ARG_REUSE: Macro argument reuse '_sub' - possible side-effects?
#42: FILE: drivers/net/sfc/base/efx_types.h:338:
+#define	EFX_SSUB(_val, _sub) \
+	((_val) > (_sub) ? ((_val) - (_sub)) : 0)

ERROR:SPACING: space required after that ',' (ctx:VxV)
#109: FILE: drivers/net/sfc/base/efx_types.h:1306:
+		((uint32_t)1 << EFX_SSUB((_bit),(_base))) :		\
 		                               ^

total: 1 errors, 0 warnings, 2 checks, 89 lines checked


More information about the test-report mailing list