[dpdk-test-report] |WARNING| pw85502 [PATCH v1 2/5] net/iavf_be: control queue enabling

checkpatch at dpdk.org checkpatch at dpdk.org
Sat Dec 19 09:08:00 CET 2020


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

_coding style issues_


WARNING:TYPO_SPELLING: 'implemention' may be misspelled - perhaps 'implementation'?
#60: 
2. Emu device callback functions implemention.

ERROR:MULTISTATEMENT_MACRO_USE_DO_WHILE: Macros with multiple statements should be enclosed in a do - while loop
#88: FILE: drivers/net/iavf_be/iavf_be.h:14:
+#define IAVFBE_WRITE_32(addr, val)  \
+	*(volatile uint32_t *)(addr) = rte_cpu_to_le_32(val);

WARNING:TRAILING_SEMICOLON: macros should not use a trailing semicolon
#88: FILE: drivers/net/iavf_be/iavf_be.h:14:
+#define IAVFBE_WRITE_32(addr, val)  \
+	*(volatile uint32_t *)(addr) = rte_cpu_to_le_32(val);

CHECK:UNNECESSARY_PARENTHESES: Unnecessary parentheses around adapter->mem_table
#208: FILE: drivers/net/iavf_be/iavf_be_ethdev.c:148:
+	struct rte_iavf_emu_mem **mem = &(adapter->mem_table);

CHECK:UNNECESSARY_PARENTHESES: Unnecessary parentheses around adapter->mem_table
#312: FILE: drivers/net/iavf_be/iavf_be_ethdev.c:252:
+	struct rte_iavf_emu_mem **mem = &(adapter->mem_table);

CHECK:BRACES: Blank lines aren't necessary before a close brace '}'
#388: FILE: drivers/net/iavf_be/iavf_be_ethdev.c:328:
+
+}

CHECK:BRACES: Blank lines aren't necessary before a close brace '}'
#413: FILE: drivers/net/iavf_be/iavf_be_ethdev.c:353:
+
+}

WARNING:BLOCK_COMMENT_STYLE: Block comments use a trailing */ on a separate line
#500: FILE: drivers/net/iavf_be/iavf_be_ethdev.c:481:
+	 * done when emu device is ready.*/

CHECK:OPEN_ENDED_LINE: Lines should not end with a '('
#607: FILE: drivers/net/iavf_be/iavf_be_vchnl.c:74:
+		buf_va = (void *)(uintptr_t)rte_iavf_emu_get_dma_vaddr(

CHECK:OPEN_ENDED_LINE: Lines should not end with a '('
#673: FILE: drivers/net/iavf_be/iavf_be_vchnl.c:140:
+			buf_va = (void *)(uintptr_t)rte_iavf_emu_get_dma_vaddr(

CHECK:OPEN_ENDED_LINE: Lines should not end with a '('
#732: FILE: drivers/net/iavf_be/iavf_be_vchnl.c:199:
+	asq->ring = (void *)(uintptr_t)rte_iavf_emu_get_dma_vaddr(

CHECK:OPEN_ENDED_LINE: Lines should not end with a '('
#741: FILE: drivers/net/iavf_be/iavf_be_vchnl.c:208:
+	arq->ring = (void *)(uintptr_t)rte_iavf_emu_get_dma_vaddr(

WARNING:TYPO_SPELLING: 'initilized' may be misspelled - perhaps 'initialized'?
#766: FILE: drivers/net/iavf_be/iavf_be_vchnl.c:233:
+		/* Check if control queue is initilized */

WARNING:MISSING_SPACE: break quoted strings at a space character
#784: FILE: drivers/net/iavf_be/iavf_be_vchnl.c:251:
+			IAVF_BE_LOG(DEBUG, "Failed to read msg"
+				    "from AdminQ");

total: 1 errors, 5 warnings, 8 checks, 713 lines checked
Warning in drivers/net/iavf_be/iavf_be_ethdev.c:
Using rte_atomicNN_xxx


More information about the test-report mailing list