[dpdk-test-report] |WARNING| pw95646 [PATCH v2] dmadev: introduce DMA device library

checkpatch at dpdk.org checkpatch at dpdk.org
Sun Jul 11 11:31:08 CEST 2021


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

_coding style issues_


WARNING:TYPO_SPELLING: 'multilpe' may be misspelled - perhaps 'multiple'?
#738: FILE: lib/dmadev/rte_dmadev.h:38:
+ * The DMA controller could have multilpe HW-DMA-channels (aka. HW-DMA-queues),

WARNING:MACRO_WITH_FLOW_CONTROL: Macros with flow control statements should be avoided
#850: FILE: lib/dmadev/rte_dmadev.h:150:
+#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:MACRO_WITH_FLOW_CONTROL: Macros with flow control statements should be avoided
#857: FILE: lib/dmadev/rte_dmadev.h:157:
+#define RTE_DMADEV_VALID_DEV_ID_OR_RET(dev_id) do { \
+	if (!rte_dmadev_is_valid_dev(dev_id)) { \
+		RTE_DMADEV_LOG(ERR, "Invalid dev_id=%u
", dev_id); \
+		return; \
+	} \
+} while (0)

WARNING:TYPO_SPELLING: 'slave' may be misspelled - perhaps 'secondary'?
#910: FILE: lib/dmadev/rte_dmadev.h:210:
+/**< DMA device support slave mode & mem-to-dev transfer.

WARNING:TYPO_SPELLING: 'slave' may be misspelled - perhaps 'secondary'?
#915: FILE: lib/dmadev/rte_dmadev.h:215:
+/**< DMA device support slave mode & dev-to-mem transfer.

WARNING:TYPO_SPELLING: 'slave' may be misspelled - perhaps 'secondary'?
#920: FILE: lib/dmadev/rte_dmadev.h:220:
+/**< DMA device support slave mode & dev-to-dev transfer.

WARNING:TYPO_SPELLING: 'slave' may be misspelled - perhaps 'secondary'?
#1143: FILE: lib/dmadev/rte_dmadev.h:443:
+/**< DMA transfer direction - slave mode & from memory to device.

WARNING:TYPO_SPELLING: 'slave' may be misspelled - perhaps 'secondary'?
#1145: FILE: lib/dmadev/rte_dmadev.h:445:
+ * this case, the ARM SoCs works in slave mode, it could initiate a DMA move

WARNING:TYPO_SPELLING: 'slave' may be misspelled - perhaps 'secondary'?
#1151: FILE: lib/dmadev/rte_dmadev.h:451:
+/**< DMA transfer direction - slave mode & from device to memory.

WARNING:TYPO_SPELLING: 'slave' may be misspelled - perhaps 'secondary'?
#1153: FILE: lib/dmadev/rte_dmadev.h:453:
+ * this case, the ARM SoCs works in slave mode, it could initiate a DMA move

WARNING:TYPO_SPELLING: 'slave' may be misspelled - perhaps 'secondary'?
#1159: FILE: lib/dmadev/rte_dmadev.h:459:
+/**< DMA transfer direction - slave mode & from device to device.

WARNING:TYPO_SPELLING: 'slave' may be misspelled - perhaps 'secondary'?
#1161: FILE: lib/dmadev/rte_dmadev.h:461:
+ * this case, the ARM SoCs works in slave mode, it could initiate a DMA move

WARNING:TYPO_SPELLING: 'slave' may be misspelled - perhaps 'secondary'?
#1172: FILE: lib/dmadev/rte_dmadev.h:472:
+ * enum rte_dma_slave_port_type - slave mode type defines

WARNING:TYPO_SPELLING: 'slave' may be misspelled - perhaps 'secondary'?
#1172: FILE: lib/dmadev/rte_dmadev.h:472:
+ * enum rte_dma_slave_port_type - slave mode type defines

WARNING:TYPO_SPELLING: 'slave' may be misspelled - perhaps 'secondary'?
#1174: FILE: lib/dmadev/rte_dmadev.h:474:
+enum rte_dma_slave_port_type {

WARNING:TYPO_SPELLING: 'slave' may be misspelled - perhaps 'secondary'?
#1175: FILE: lib/dmadev/rte_dmadev.h:475:
+	/** The slave port is PCIE. */

WARNING:TYPO_SPELLING: 'SLAVE' may be misspelled - perhaps 'SECONDARY'?
#1176: FILE: lib/dmadev/rte_dmadev.h:476:
+	RTE_DMA_SLAVE_PORT_PCIE = 1,

WARNING:TYPO_SPELLING: 'slave' may be misspelled - perhaps 'secondary'?
#1180: FILE: lib/dmadev/rte_dmadev.h:480:
+ * A structure used to descript slave port parameters.

WARNING:TYPO_SPELLING: 'slave' may be misspelled - perhaps 'secondary'?
#1182: FILE: lib/dmadev/rte_dmadev.h:482:
+struct rte_dma_slave_port_parameters {

WARNING:TYPO_SPELLING: 'slave' may be misspelled - perhaps 'secondary'?
#1183: FILE: lib/dmadev/rte_dmadev.h:483:
+	enum rte_dma_slave_port_type port_type;

WARNING:TYPO_SPELLING: 'slave' may be misspelled - perhaps 'secondary'?
#1212: FILE: lib/dmadev/rte_dmadev.h:512:
+	struct rte_dma_slave_port_parameters port;

WARNING:TYPO_SPELLING: 'slave' may be misspelled - perhaps 'secondary'?
#1216: FILE: lib/dmadev/rte_dmadev.h:516:
+	struct rte_dma_slave_port_parameters peer_port;

total: 0 errors, 22 warnings, 1893 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