[dpdk-test-report] |WARNING| [PATCH v2 01/55] net/sfc: libefx-based PMD stub sufficient to build and init

checkpatch at dpdk.org checkpatch at dpdk.org
Tue Nov 29 17:20:54 CET 2016


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

_coding style issues_


WARNING:USE_NEGATIVE_ERRNO: return of an errno should typically be negative (ie: return -EINVAL)
#809: FILE: drivers/net/sfc/sfc_kvargs.c:53:
+		return EINVAL;

CHECK:CAMELCASE: Avoid CamelCase: <PRIu8>
#975: FILE: drivers/net/sfc/sfc_log.h:40:
+			RTE_FMT("sfc_efx " PCI_PRI_FMT " #%" PRIu8 ": "	\

ERROR:SPACING: space required after that ',' (ctx:VxB)
#976: FILE: drivers/net/sfc/sfc_log.h:41:
+				RTE_FMT_HEAD(__VA_ARGS__,) "
",	\
 				                        ^

ERROR:SPACING: space required after that ',' (ctx:VxB)
#982: FILE: drivers/net/sfc/sfc_log.h:47:
+				RTE_FMT_TAIL(__VA_ARGS__,)));		\
 				                        ^

ERROR:SPACING: space required after that ',' (ctx:VxB)
#1004: FILE: drivers/net/sfc/sfc_log.h:69:
+					RTE_FMT_HEAD(__VA_ARGS__,),	\
 					                        ^

ERROR:SPACING: space required after that ',' (ctx:VxB)
#1006: FILE: drivers/net/sfc/sfc_log.h:71:
+					RTE_FMT_TAIL(__VA_ARGS__,)));	\
 					                        ^

total: 4 errors, 1 warnings, 777 lines checked


More information about the test-report mailing list