[dpdk-test-report] |WARNING| pw90379 [PATCH v3 06/52] common/cnxk: add mailbox base infra

checkpatch at dpdk.org checkpatch at dpdk.org
Thu Apr 1 14:41:39 CEST 2021


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

_coding style issues_


ERROR:MULTISTATEMENT_MACRO_USE_DO_WHILE: Macros with multiple statements should be enclosed in a do - while loop
#591: FILE: drivers/common/cnxk/roc_mbox.c:461:
+#define M(_name, _id, _1, _2, _3)                                              \
+	case _id:                                                              \
+		return #_name;

WARNING:MACRO_WITH_FLOW_CONTROL: Macros with flow control statements should be avoided
#591: FILE: drivers/common/cnxk/roc_mbox.c:461:
+#define M(_name, _id, _1, _2, _3)                                              \
+	case _id:                                                              \
+		return #_name;

WARNING:TRAILING_SEMICOLON: macros should not use a trailing semicolon
#591: FILE: drivers/common/cnxk/roc_mbox.c:461:
+#define M(_name, _id, _1, _2, _3)                                              \
+	case _id:                                                              \
+		return #_name;

ERROR:MULTISTATEMENT_MACRO_USE_DO_WHILE: Macros with multiple statements should be enclosed in a do - while loop
#606: FILE: drivers/common/cnxk/roc_mbox.c:476:
+#define M(_1, _id, _2, _req_type, _3)                                          \
+	case _id:                                                              \
+		return sizeof(struct _req_type);

WARNING:MACRO_WITH_FLOW_CONTROL: Macros with flow control statements should be avoided
#606: FILE: drivers/common/cnxk/roc_mbox.c:476:
+#define M(_1, _id, _2, _req_type, _3)                                          \
+	case _id:                                                              \
+		return sizeof(struct _req_type);

WARNING:TRAILING_SEMICOLON: macros should not use a trailing semicolon
#606: FILE: drivers/common/cnxk/roc_mbox.c:476:
+#define M(_1, _id, _2, _req_type, _3)                                          \
+	case _id:                                                              \
+		return sizeof(struct _req_type);

total: 2 errors, 4 warnings, 771 lines checked


More information about the test-report mailing list