|WARNING| pw130148 [PATCH 3/4] net/cpfl: introduce CPF common library

|WARNING| pw130148 [PATCH 3/4] net/cpfl: introduce CPF common library

[checkpatch]

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

_coding style issues_


CHECK:MACRO_ARG_REUSE: Macro argument reuse 'p' - possible side-effects?
#197: FILE: drivers/net/cpfl/cpfl_actions.h:108:
+#define CPFL_ACT_PREC_CHECK(p)	((p) > 0 && (p) <= CPFL_ACT_PREC_MAX)

CHECK:MACRO_ARG_REUSE: Macro argument reuse 'idx' - possible side-effects?
#269: FILE: drivers/net/cpfl/cpfl_actions.h:180:
+#define CPFL_ACT_MAKE_8B(prec, idx, val) \
+	((CPFL_ACT_PREC_SET(prec)) | CPFL_ACT_OP_8B | \
+	 (((idx) << CPFL_ACT_8B_A_INDEX_S) & CPFL_ACT_8B_A_INDEX_M) | \
+	 (((idx) << CPFL_ACT_8B_B_INDEX_S) & CPFL_ACT_8B_B_INDEX_M) | \
+	 (((val) << CPFL_ACT_8B_A_VAL_S) & CPFL_ACT_8B_A_VAL_M) | \
+	 (((val) << CPFL_ACT_8B_B_VAL_S) & CPFL_ACT_8B_B_VAL_M))

CHECK:MACRO_ARG_REUSE: Macro argument reuse 'val' - possible side-effects?
#269: FILE: drivers/net/cpfl/cpfl_actions.h:180:
+#define CPFL_ACT_MAKE_8B(prec, idx, val) \
+	((CPFL_ACT_PREC_SET(prec)) | CPFL_ACT_OP_8B | \
+	 (((idx) << CPFL_ACT_8B_A_INDEX_S) & CPFL_ACT_8B_A_INDEX_M) | \
+	 (((idx) << CPFL_ACT_8B_B_INDEX_S) & CPFL_ACT_8B_B_INDEX_M) | \
+	 (((val) << CPFL_ACT_8B_A_VAL_S) & CPFL_ACT_8B_A_VAL_M) | \
+	 (((val) << CPFL_ACT_8B_B_VAL_S) & CPFL_ACT_8B_B_VAL_M))

WARNING:TYPO_SPELLING: 'descritpors' may be misspelled - perhaps 'descriptors'?
#1010: FILE: drivers/net/cpfl/cpfl_controlq.c:57:
+ * both the descritpors and the buffers

WARNING:TYPO_SPELLING: 'thru' may be misspelled - perhaps 'through'?
#1433: FILE: drivers/net/cpfl/cpfl_rules.c:37:
+		context |= SHIFT_VAL64(cmn_cfg->cache_wr_thru,

WARNING:TYPO_SPELLING: 'THRU' may be misspelled - perhaps 'THROUGH'?
#1434: FILE: drivers/net/cpfl/cpfl_rules.c:38:
+				       MEV_RULE_CACHE_WR_THRU);

WARNING:TYPO_SPELLING: 'THRU' may be misspelled - perhaps 'THROUGH'?
#1613: FILE: drivers/net/cpfl/cpfl_rules.h:85:
+#define MEV_RULE_CACHE_WR_THRU_S	21

WARNING:TYPO_SPELLING: 'THRU' may be misspelled - perhaps 'THROUGH'?
#1614: FILE: drivers/net/cpfl/cpfl_rules.h:86:
+#define MEV_RULE_CACHE_WR_THRU_M	\

WARNING:TYPO_SPELLING: 'THRU' may be misspelled - perhaps 'THROUGH'?
#1615: FILE: drivers/net/cpfl/cpfl_rules.h:87:
+		MAKE_MASK64(0x1, MEV_RULE_CACHE_WR_THRU_S)

WARNING:TYPO_SPELLING: 'strcuture' may be misspelled - perhaps 'structure'?
#1718: FILE: drivers/net/cpfl/cpfl_rules.h:190:
+ * cpfl_rule_query_addr - LEM/SEM Rule Query Address strcuture

WARNING:TYPO_SPELLING: 'strucure' may be misspelled - perhaps 'structure'?
#1744: FILE: drivers/net/cpfl/cpfl_rules.h:216:
+ *note: some rules may only require part of strucure

WARNING:TYPO_SPELLING: 'thru' may be misspelled - perhaps 'through'?
#1754: FILE: drivers/net/cpfl/cpfl_rules.h:226:
+	uint8_t	cache_wr_thru;

WARNING:TYPO_SPELLING: 'thru' may be misspelled - perhaps 'through'?
#1801: FILE: drivers/net/cpfl/cpfl_rules.h:273:
+			       uint8_t cache_wr_thru,

WARNING:TYPO_SPELLING: 'thru' may be misspelled - perhaps 'through'?
#1816: FILE: drivers/net/cpfl/cpfl_rules.h:288:
+	cfg_cmn->cache_wr_thru = cache_wr_thru;

WARNING:TYPO_SPELLING: 'thru' may be misspelled - perhaps 'through'?
#1816: FILE: drivers/net/cpfl/cpfl_rules.h:288:
+	cfg_cmn->cache_wr_thru = cache_wr_thru;

total: 0 errors, 12 warnings, 3 checks, 1729 lines checked