|WARNING| pw147972 [PATCH 2/2] common/cnxk: add mailbox debug trace

[checkpatch at dpdk.org]

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

_coding style issues_


WARNING:TYPO_SPELLING: 'SND' may be misspelled - perhaps 'SEND'?
#265: FILE: drivers/common/cnxk/roc_dev.c:574:
+		roc_trace_mbox_process("PFVF_SND_UPMSG", mbox_id2name(msg->hdr.id), 1,

WARNING:LONG_LINE: line length of 108 exceeds 100 columns
#279: FILE: drivers/common/cnxk/roc_dev.c:907:
+		roc_trace_mbox_process("PRC_UPMSG", mbox_id2name(msg->id), req_hdr->num_msgs, msg->pcifunc);

WARNING:LONG_LINE: line length of 108 exceeds 100 columns
#292: FILE: drivers/common/cnxk/roc_dev.c:917:
+				plt_err("Error %d handling 0x%x (%s)", err, msg->id, mbox_id2name(msg->id));

WARNING:TYPO_SPELLING: 'SND' may be misspelled - perhaps 'SEND'?
#397: FILE: drivers/common/cnxk/roc_mbox.c:323:
+	mbox_debug_region("SND_MSG", mbox, devid, intr_val, 0);

WARNING:SUSPECT_COMMA_SEMICOLON: Possible comma where semicolon could be used
#509: FILE: drivers/common/cnxk/roc_platform.h:387:
+		RTE_TRACE_POINT_ARGS(const char *func, const char *msg, uint16_t pcifunc, int data,
+				     uint16_t cookie),
+		rte_trace_point_emit_string(func);

WARNING:SUSPECT_COMMA_SEMICOLON: Possible comma where semicolon could be used
#516: FILE: drivers/common/cnxk/roc_platform.h:394:
+		RTE_TRACE_POINT_ARGS(const char *func, const char *msg, uint16_t num_msgs,
+				     uint16_t pcifunc),
+		rte_trace_point_emit_string(func);

WARNING:SUSPECT_COMMA_SEMICOLON: Possible comma where semicolon could be used
#523: FILE: drivers/common/cnxk/roc_platform.h:401:
+		RTE_TRACE_POINT_ARGS(const char *func, int pcifunc, uint64_t intr,
+				     uint64_t mbox_data),
+		rte_trace_point_emit_string(func);

WARNING:SUSPECT_COMMA_SEMICOLON: Possible comma where semicolon could be used
#530: FILE: drivers/common/cnxk/roc_platform.h:408:
+		RTE_TRACE_POINT_ARGS(const char *func, uint16_t from_vf, uint16_t pcifunc),
+		rte_trace_point_emit_string(func);

WARNING:SUSPECT_COMMA_SEMICOLON: Possible comma where semicolon could be used
#535: FILE: drivers/common/cnxk/roc_platform.h:413:
+		RTE_TRACE_POINT_ARGS(const char *ops, uint16_t pcifunc, int data),
+		rte_trace_point_emit_string(ops);

WARNING:SUSPECT_COMMA_SEMICOLON: Possible comma where semicolon could be used
#540: FILE: drivers/common/cnxk/roc_platform.h:418:
+		RTE_TRACE_POINT_ARGS(const char *func, const char *msg, int data),
+		rte_trace_point_emit_string(func);

total: 0 errors, 10 warnings, 441 lines checked
Warning in drivers/common/cnxk/roc_platform.c:
Warning in drivers/common/cnxk/roc_platform.h:
Use plt_ symbols instead of rte_ API in cnxk base driver