|WARNING| pw105249 [PATCH v1 11/25] net/spnic: add queue pairs context initialization

checkpatch at dpdk.org checkpatch at dpdk.org
Sat Dec 18 03:55:14 CET 2021


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

_coding style issues_


CHECK:CAMELCASE: Avoid CamelCase: <PRIu64>
#1091: FILE: drivers/net/spnic/spnic_io.c:508:
+				    "err: %d, out_param: %"PRIu64"",

CHECK:CONCATENATED_STRING: Concatenated strings should use spaces between elements
#1091: FILE: drivers/net/spnic/spnic_io.c:508:
+				    "err: %d, out_param: %"PRIu64"",

CHECK:CONCATENATED_STRING: Concatenated strings should use spaces between elements
#1146: FILE: drivers/net/spnic/spnic_io.c:563:
+				    "err: %d, out_param: %"PRIu64"",

CHECK:CONCATENATED_STRING: Concatenated strings should use spaces between elements
#1187: FILE: drivers/net/spnic/spnic_io.c:604:
+			    "err: %d, out_param: %"PRIu64"", err, out_param);

CHECK:CONCATENATED_STRING: Concatenated strings should use spaces between elements
#1307: FILE: drivers/net/spnic/spnic_io.c:724:
+	PMD_DRV_LOG(INFO, "Update nic feature to %"PRIu64"
",

CHECK:MACRO_ARG_REUSE: Macro argument reuse 'queue' - possible side-effects?
#1402: FILE: drivers/net/spnic/spnic_io.h:75:
+#define	NIC_WQE_ADDR(queue, idx) ((void *)((u64)((queue)->queue_buf_vaddr) + \
+				       ((idx) << (queue)->wqebb_shift)))

WARNING:TYPO_SPELLING: 'coverting' may be misspelled - perhaps 'converting'?
#1426: FILE: drivers/net/spnic/spnic_io.h:99:
+	/* Hardware will do endianness coverting */

total: 0 errors, 1 warnings, 6 checks, 1518 lines checked


More information about the test-report mailing list