[dpdk-test-report] |WARNING| pw78252 [PATCH 03/60] common/sfc_efx/base: add event queue operation to do polling

checkpatch at dpdk.org checkpatch at dpdk.org
Tue Sep 22 10:52:17 CEST 2020


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

_coding style issues_


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

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

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

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

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

WARNING:PREFER_FALLTHROUGH: Prefer 'fallthrough;' over fallthrough comment
#422: FILE: drivers/common/sfc_efx/base/efx_ev.c:1473:
+				/* else fallthrough */

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

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

WARNING:FUNCTION_ARGUMENTS: function definition argument 'efx_evq_t *' should also have an identifier name
#479: FILE: drivers/common/sfc_efx/base/efx_impl.h:88:
+	void		(*eevo_qpoll)(efx_evq_t *, unsigned int *,

WARNING:FUNCTION_ARGUMENTS: function definition argument 'unsigned int *' should also have an identifier name
#479: FILE: drivers/common/sfc_efx/base/efx_impl.h:88:
+	void		(*eevo_qpoll)(efx_evq_t *, unsigned int *,

WARNING:FUNCTION_ARGUMENTS: function definition argument 'const efx_ev_callbacks_t *' should also have an identifier name
#479: FILE: drivers/common/sfc_efx/base/efx_impl.h:88:
+	void		(*eevo_qpoll)(efx_evq_t *, unsigned int *,

WARNING:FUNCTION_ARGUMENTS: function definition argument 'void *' should also have an identifier name
#479: FILE: drivers/common/sfc_efx/base/efx_impl.h:88:
+	void		(*eevo_qpoll)(efx_evq_t *, unsigned int *,

total: 0 errors, 12 warnings, 370 lines checked


More information about the test-report mailing list