[dpdk-test-report] |WARNING| pw95904 [PATCH v4] dmadev: introduce DMA device library

checkpatch at dpdk.org checkpatch at dpdk.org
Thu Jul 15 17:45:45 CEST 2021


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

_coding style issues_


WARNING:MACRO_WITH_FLOW_CONTROL: Macros with flow control statements should be avoided
#205: FILE: lib/dmadev/rte_dmadev.c:40:
+#define RTE_DMADEV_VALID_DEV_ID_OR_ERR_RET(dev_id, retval) do { \
+	if (!rte_dmadev_is_valid_dev(dev_id)) { \
+		RTE_DMADEV_LOG(ERR, "Invalid dev_id=%u
", dev_id); \
+		return retval; \
+	} \
+} while (0)

WARNING:TYPO_SPELLING: 'fileds' may be misspelled - perhaps 'fields'?
#1175: FILE: lib/dmadev/rte_dmadev.h:465:
+		 *        these fileds to zero. And also there are no

WARNING:TYPO_SPELLING: 'ATTEMPED' may be misspelled - perhaps 'ATTEMPTED'?
#1374: FILE: lib/dmadev/rte_dmadev.h:664:
+	RTE_DMA_STATUS_NOT_ATTEMPED,

WARNING:TYPO_SPELLING: 'primay' may be misspelled - perhaps 'primary'?
#1891: FILE: lib/dmadev/rte_dmadev_core.h:166:
+	 * It's the primay process's responsibility to deinitialize this field

WARNING:TYPO_SPELLING: 'process'' may be misspelled - perhaps 'process's'?
#1891: FILE: lib/dmadev/rte_dmadev_core.h:166:
+	 * It's the primay process's responsibility to deinitialize this field

total: 0 errors, 5 warnings, 1870 lines checked
Warning in lib/dmadev/rte_dmadev.c:
Using RTE_LOG_REGISTER, prefer RTE_LOG_REGISTER_(DEFAULT|SUFFIX)


More information about the test-report mailing list