[dpdk-test-report] |WARNING| pw30627 [PATCH RFC 4/9] net/avf: enable basic Rx Tx func

checkpatch at dpdk.org checkpatch at dpdk.org
Fri Oct 20 10:36:41 CEST 2017


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

_coding style issues_


CHECK:BRACES: Unbalanced braces around else statement
#386: FILE: drivers/net/avf/avf_rxtx.c:926:
+			} else

WARNING:DEEP_INDENTATION: Too many leading tabs - consider code refactoring
#657: FILE: drivers/net/avf/avf_rxtx.c:1197:
+						if (nb_tx == 0)

CHECK:CAMELCASE: Avoid CamelCase: <PRIx64>
#701: FILE: drivers/net/avf/avf_rxtx.c:1241:
+				"type_cmd_tso_mss: %#"PRIx64";
",

CHECK:CONCATENATED_STRING: Concatenated strings should use spaces between elements
#701: FILE: drivers/net/avf/avf_rxtx.c:1241:
+				"type_cmd_tso_mss: %#"PRIx64";
",

CHECK:CONCATENATED_STRING: Concatenated strings should use spaces between elements
#724: FILE: drivers/net/avf/avf_rxtx.c:1264:
+				"buf_dma_addr: %#"PRIx64";
"

WARNING:UNSPECIFIED_INT: Prefer 'unsigned int' to bare use of 'unsigned'
#771: FILE: drivers/net/avf/avf_rxtx.c:1311:
+		   (unsigned)txq->port_id, (unsigned)txq->queue_id,

WARNING:UNSPECIFIED_INT: Prefer 'unsigned int' to bare use of 'unsigned'
#771: FILE: drivers/net/avf/avf_rxtx.c:1311:
+		   (unsigned)txq->port_id, (unsigned)txq->queue_id,

WARNING:UNSPECIFIED_INT: Prefer 'unsigned int' to bare use of 'unsigned'
#772: FILE: drivers/net/avf/avf_rxtx.c:1312:
+		   (unsigned)tx_id, (unsigned)nb_tx);

WARNING:UNSPECIFIED_INT: Prefer 'unsigned int' to bare use of 'unsigned'
#772: FILE: drivers/net/avf/avf_rxtx.c:1312:
+		   (unsigned)tx_id, (unsigned)nb_tx);

total: 0 errors, 5 warnings, 4 checks, 867 lines checked


More information about the test-report mailing list