[dpdk-test-report] |WARNING| pw20746 [PATCH 12/16] net/avp: packet transmit functions

checkpatch at dpdk.org checkpatch at dpdk.org
Sat Feb 25 02:24:59 CET 2017


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

_coding style issues_


WARNING:UNSPECIFIED_INT: Prefer 'unsigned int' to bare use of 'unsigned'
#119: FILE: drivers/net/avp/avp_ethdev.c:1625:
+			unsigned count)

WARNING:UNSPECIFIED_INT: Prefer 'unsigned int' to bare use of 'unsigned'
#130: FILE: drivers/net/avp/avp_ethdev.c:1636:
+	unsigned i;

CHECK:SPACING: spaces preferred around that '+' (ctx:VxV)
#143: FILE: drivers/net/avp/avp_ethdev.c:1649:
+			pkt_buf = avp_dev_translate_buffer(avp, buffers[i+1]);
 			                                                 ^

WARNING:UNSPECIFIED_INT: Prefer 'unsigned int' to bare use of 'unsigned'
#216: FILE: drivers/net/avp/avp_ethdev.c:1722:
+	unsigned count, avail, n;

WARNING:UNSPECIFIED_INT: Prefer 'unsigned int' to bare use of 'unsigned'
#217: FILE: drivers/net/avp/avp_ethdev.c:1723:
+	unsigned orig_nb_pkts;

WARNING:UNSPECIFIED_INT: Prefer 'unsigned int' to bare use of 'unsigned'
#219: FILE: drivers/net/avp/avp_ethdev.c:1725:
+	unsigned required;

WARNING:UNSPECIFIED_INT: Prefer 'unsigned int' to bare use of 'unsigned'
#220: FILE: drivers/net/avp/avp_ethdev.c:1726:
+	unsigned segments;

WARNING:UNSPECIFIED_INT: Prefer 'unsigned int' to bare use of 'unsigned'
#221: FILE: drivers/net/avp/avp_ethdev.c:1727:
+	unsigned tx_bytes;

WARNING:UNSPECIFIED_INT: Prefer 'unsigned int' to bare use of 'unsigned'
#222: FILE: drivers/net/avp/avp_ethdev.c:1728:
+	unsigned i;

WARNING:UNSPECIFIED_INT: Prefer 'unsigned int' to bare use of 'unsigned'
#256: FILE: drivers/net/avp/avp_ethdev.c:1762:
+		if (likely(i < (unsigned)nb_pkts - 1)) {

CHECK:SPACING: spaces preferred around that '+' (ctx:VxV)
#258: FILE: drivers/net/avp/avp_ethdev.c:1764:
+			rte_prefetch0(tx_pkts[i+1]);
 			                       ^

WARNING:UNSPECIFIED_INT: Prefer 'unsigned int' to bare use of 'unsigned'
#337: FILE: drivers/net/avp/avp_ethdev.c:1843:
+	unsigned count, avail, n;

WARNING:UNSPECIFIED_INT: Prefer 'unsigned int' to bare use of 'unsigned'
#339: FILE: drivers/net/avp/avp_ethdev.c:1845:
+	unsigned pkt_len;

WARNING:UNSPECIFIED_INT: Prefer 'unsigned int' to bare use of 'unsigned'
#340: FILE: drivers/net/avp/avp_ethdev.c:1846:
+	unsigned tx_bytes;

WARNING:UNSPECIFIED_INT: Prefer 'unsigned int' to bare use of 'unsigned'
#342: FILE: drivers/net/avp/avp_ethdev.c:1848:
+	unsigned i;

CHECK:BRACES: Blank lines aren't necessary after an open brace '{'
#386: FILE: drivers/net/avp/avp_ethdev.c:1892:
+	for (i = 0; i < count; i++) {
+

CHECK:SPACING: spaces preferred around that '-' (ctx:VxV)
#388: FILE: drivers/net/avp/avp_ethdev.c:1894:
+		if (i < count-1) {
 		             ^

CHECK:SPACING: spaces preferred around that '+' (ctx:VxV)
#389: FILE: drivers/net/avp/avp_ethdev.c:1895:
+			pkt_buf = avp_dev_translate_buffer(avp, avp_bufs[i+1]);
 			                                                  ^

total: 0 errors, 13 warnings, 5 checks, 383 lines checked


More information about the test-report mailing list