* [dpdk-test-report] |WARNING| pw95774 [PATCH v3] dmadev: introduce DMA device library
[not found] <1626179263-14645-1-git-send-email-fengchengwen@huawei.com>
@ 2021-07-13 12:33 ` checkpatch
2021-07-13 12:59 ` [dpdk-test-report] |SUCCESS| pw95774 [dpdk-dev] " 0-day Robot
1 sibling, 0 replies; 2+ messages in thread
From: checkpatch @ 2021-07-13 12:33 UTC (permalink / raw)
To: test-report; +Cc: Chengwen Feng
Test-Label: checkpatch
Test-Status: WARNING
http://dpdk.org/patch/95774
_coding style issues_
WARNING:MACRO_WITH_FLOW_CONTROL: Macros with flow control statements should be avoided
#188: 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:MACRO_WITH_FLOW_CONTROL: Macros with flow control statements should be avoided
#195: FILE: lib/dmadev/rte_dmadev.c:47:
+#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:MACRO_WITH_FLOW_CONTROL: Macros with flow control statements should be avoided
#203: FILE: lib/dmadev/rte_dmadev.c:55:
+#define RTE_DMADEV_PTR_OR_ERR_RET(ptr, retval) do { \
+ if ((ptr) == NULL) \
+ return retval; \
+} while (0)
total: 0 errors, 3 warnings, 1832 lines checked
Warning in lib/dmadev/rte_dmadev.c:
Using RTE_LOG_REGISTER, prefer RTE_LOG_REGISTER_(DEFAULT|SUFFIX)
^ permalink raw reply [flat|nested] 2+ messages in thread