|WARNING| pw151617 [PATCH v13 05/28] net/rnp: add device init and uninit

checkpatch at dpdk.org checkpatch at dpdk.org
Wed Feb 19 09:03:08 CET 2025


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

_coding style issues_


WARNING:TYPO_SPELLING: 'alloced' may be misspelled - perhaps 'allocated'?
#583: FILE: drivers/net/rnp/base/rnp_hw.h:74:
+	bool msg_alloced;

CHECK:MACRO_ARG_REUSE: Macro argument reuse 'f' - possible side-effects?
#589: FILE: drivers/net/rnp/base/rnp_hw.h:80:
+#define rnp_call_hwif_impl(port, f, arg...) \
+	(((f) != NULL) ? ((f) (port, arg)) : (-ENODEV))

CHECK:MACRO_ARG_REUSE: Macro argument reuse 'x' - possible side-effects?
#1073: FILE: drivers/net/rnp/base/rnp_osdep.h:39:
+#define ffs(x) (rte_fls_u32((x) & (-x)))

CHECK:CAMELCASE: Avoid CamelCase: <PRIu64>
#1127: FILE: drivers/net/rnp/base/rnp_osdep.h:103:
+		snprintf(z_name, sizeof(z_name), "rnp_dma_%" PRIu64,

CHECK:CONCATENATED_STRING: Concatenated strings should use spaces between elements
#1141: FILE: drivers/net/rnp/base/rnp_osdep.h:117:
+			"%"PRIu64, mz->name, mem->pa);

CHECK:CONCATENATED_STRING: Concatenated strings should use spaces between elements
#1151: FILE: drivers/net/rnp/base/rnp_osdep.h:127:
+			"%"PRIu64, ((const struct rte_memzone *)mem->mz)->name,

WARNING:TYPO_SPELLING: 'onw' may be misspelled - perhaps 'own'?
#1231: FILE: drivers/net/rnp/rnp.h:66:
+rnp_pf_onw_ports(uint32_t id)

WARNING:TYPO_SPELLING: 'onw' may be misspelled - perhaps 'own'?
#1241: FILE: drivers/net/rnp/rnp.h:76:
+	return rnp_pf_onw_ports(verbit) == 1 ? 0 : 1;

total: 0 errors, 3 warnings, 5 checks, 1421 lines checked
Warning in drivers/net/rnp/base/rnp_hw.h:
Do not use variadic argument pack in macros


More information about the test-report mailing list