|WARNING| pw131460 [PATCH v4 4/9] net/cpfl: add FXP low level implementation

[checkpatch at dpdk.org]

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

_coding style issues_


CHECK:MACRO_ARG_REUSE: Macro argument reuse 'p' - possible side-effects?
#203: 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?
#275: 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?
#275: 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: 'thru' may be misspelled - perhaps 'through'?
#1794: 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'?
#1795: FILE: drivers/net/cpfl/cpfl_rules.c:38:
+				       MEV_RULE_CACHE_WR_THRU);

WARNING:TYPO_SPELLING: 'THRU' may be misspelled - perhaps 'THROUGH'?
#1974: 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'?
#1975: FILE: drivers/net/cpfl/cpfl_rules.h:86:
+#define MEV_RULE_CACHE_WR_THRU_M	\

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

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

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

WARNING:TYPO_SPELLING: 'thru' may be misspelled - perhaps 'through'?
#2177: 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'?
#2177: FILE: drivers/net/cpfl/cpfl_rules.h:288:
+	cfg_cmn->cache_wr_thru = cache_wr_thru;

total: 0 errors, 9 warnings, 3 checks, 2214 lines checked