[dpdk-test-report] |WARNING| pw31788 [PATCH 04/11] net/vhostpci: add basic framework

checkpatch at dpdk.org checkpatch at dpdk.org
Thu Nov 30 10:47:58 CET 2017


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

_coding style issues_


CHECK:BRACES: Blank lines aren't necessary after an open brace '{'
#270: FILE: drivers/net/vhostpci/vhostpci_ethdev.c:208:
+{
+

CHECK:BRACES: Blank lines aren't necessary before a close brace '}'
#289: FILE: drivers/net/vhostpci/vhostpci_ethdev.c:227:
+
+}

CHECK:UNNECESSARY_PARENTHESES: Unnecessary parentheses around dev->data->dev_link
#301: FILE: drivers/net/vhostpci/vhostpci_ethdev.c:239:
+	struct rte_eth_link *dst = &(dev->data->dev_link);

CHECK:BRACES: Blank lines aren't necessary after an open brace '{'
#467: FILE: drivers/net/vhostpci/vhostpci_ethdev.c:405:
+	for (i = 0; i < VHOSTPCI_MAX_QUEUE_PAIRS * 2; i++) {
+

CHECK:BRACES: Blank lines aren't necessary before a close brace '}'
#699: FILE: drivers/net/vhostpci/vhostpci_pci.c:92:
+
+}

CHECK:BRACES: Blank lines aren't necessary after an open brace '{'
#825: FILE: drivers/net/vhostpci/vhostpci_pci.c:218:
+	if (offset + length > dev->mem_resource[bar].len) {
+

CHECK:CAMELCASE: Avoid CamelCase: <PRIu64>
#827: FILE: drivers/net/vhostpci/vhostpci_pci.c:220:
+			"invalid cap: overflows bar space: %u > %" PRIu64,

CHECK:BRACES: Blank lines aren't necessary after an open brace '{'
#878: FILE: drivers/net/vhostpci/vhostpci_pci.c:271:
+		if (cap.cap_vndr != PCI_CAP_ID_VNDR) {
+

CHECK:BRACES: Blank lines aren't necessary after an open brace '{'
#931: FILE: drivers/net/vhostpci/vhostpci_pci.c:324:
+{
+

WARNING:UNNECESSARY_ELSE: else is not generally useful after a break or return
#937: FILE: drivers/net/vhostpci/vhostpci_pci.c:330:
+		return 0;
+	} else {

total: 0 errors, 1 warnings, 892 lines checked


More information about the test-report mailing list