[dpdk-test-report] |WARNING| pw78698 [PATCH v3 23/60] common/sfc_efx/base: add event queue module for Riverhead

[checkpatch at dpdk.org]

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

_coding style issues_


ERROR:RETURN_PARENTHESES: return is not a function, parentheses are not required
#319: FILE: drivers/common/sfc_efx/base/rhead_ev.c:81:
+	return (0);

ERROR:RETURN_PARENTHESES: return is not a function, parentheses are not required
#324: FILE: drivers/common/sfc_efx/base/rhead_ev.c:86:
+	return (rc);

ERROR:RETURN_PARENTHESES: return is not a function, parentheses are not required
#354: FILE: drivers/common/sfc_efx/base/rhead_ev.c:116:
+	return (0);

WARNING:TYPO_SPELLING: 'inout' may be misspelled - perhaps 'input'?
#390: FILE: drivers/common/sfc_efx/base/rhead_ev.c:152:
+	__inout		unsigned int *countp,

WARNING:SPACING: space prohibited between function name and open parenthesis '('
#411: FILE: drivers/common/sfc_efx/base/rhead_ev.c:173:
+		offset = (count & eep->ee_mask) * sizeof (efx_qword_t);

WARNING:SPACING: space prohibited between function name and open parenthesis '('
#422: FILE: drivers/common/sfc_efx/base/rhead_ev.c:184:
+			offset += sizeof (efx_qword_t);

WARNING:USE_NEGATIVE_ERRNO: return of an errno should typically be negative (ie: return -ENOTSUP)
#482: FILE: drivers/common/sfc_efx/base/rhead_ev.c:244:
+	return (ENOTSUP);

WARNING:TYPO_SPELLING: 'inout' may be misspelled - perhaps 'input'?
#490: FILE: drivers/common/sfc_efx/base/rhead_ev.c:252:
+	__inout_ecount(EV_NQSTATS)	efsys_stat_t *stat)

WARNING:TYPO_SPELLING: 'inout' may be misspelled - perhaps 'input'?
#571: FILE: drivers/common/sfc_efx/base/rhead_impl.h:153:
+	__inout		unsigned int *countp,

WARNING:TYPO_SPELLING: 'inout' may be misspelled - perhaps 'input'?
#587: FILE: drivers/common/sfc_efx/base/rhead_impl.h:169:
+	__inout_ecount(EV_NQSTATS)	efsys_stat_t *stat);

total: 3 errors, 7 warnings, 450 lines checked