|WARNING| pw143879 [PATCH 21/33] net/cnxk: support Rx burst scalar for cn20k

|WARNING| pw143879 [PATCH 21/33] net/cnxk: support Rx burst scalar for cn20k

[checkpatch]

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

_coding style issues_


CHECK:MACRO_ARG_REUSE: Macro argument reuse 'b' - possible side-effects?
#290: FILE: drivers/net/cnxk/cn20k_rx.h:37:
+#define CQE_PTR_OFF(b, i, o, f)                                                                    \
+	(((f) & NIX_RX_VWQE_F) ? (uint64_t *)(((uintptr_t)((uint64_t *)(b))[i]) + (o)) :           \
+				 (uint64_t *)(((uintptr_t)(b)) + CQE_SZ(i) + (o)))

CHECK:MACRO_ARG_REUSE: Macro argument reuse 'i' - possible side-effects?
#290: FILE: drivers/net/cnxk/cn20k_rx.h:37:
+#define CQE_PTR_OFF(b, i, o, f)                                                                    \
+	(((f) & NIX_RX_VWQE_F) ? (uint64_t *)(((uintptr_t)((uint64_t *)(b))[i]) + (o)) :           \
+				 (uint64_t *)(((uintptr_t)(b)) + CQE_SZ(i) + (o)))

CHECK:MACRO_ARG_REUSE: Macro argument reuse 'o' - possible side-effects?
#290: FILE: drivers/net/cnxk/cn20k_rx.h:37:
+#define CQE_PTR_OFF(b, i, o, f)                                                                    \
+	(((f) & NIX_RX_VWQE_F) ? (uint64_t *)(((uintptr_t)((uint64_t *)(b))[i]) + (o)) :           \
+				 (uint64_t *)(((uintptr_t)(b)) + CQE_SZ(i) + (o)))

CHECK:MACRO_ARG_REUSE: Macro argument reuse 'b' - possible side-effects?
#293: FILE: drivers/net/cnxk/cn20k_rx.h:40:
+#define CQE_PTR_DIFF(b, i, o, f)                                                                   \
+	(((f) & NIX_RX_VWQE_F) ? (uint64_t *)(((uintptr_t)((uint64_t *)(b))[i]) - (o)) :           \
+				 (uint64_t *)(((uintptr_t)(b)) + CQE_SZ(i) - (o)))

CHECK:MACRO_ARG_REUSE: Macro argument reuse 'i' - possible side-effects?
#293: FILE: drivers/net/cnxk/cn20k_rx.h:40:
+#define CQE_PTR_DIFF(b, i, o, f)                                                                   \
+	(((f) & NIX_RX_VWQE_F) ? (uint64_t *)(((uintptr_t)((uint64_t *)(b))[i]) - (o)) :           \
+				 (uint64_t *)(((uintptr_t)(b)) + CQE_SZ(i) - (o)))

CHECK:MACRO_ARG_REUSE: Macro argument reuse 'o' - possible side-effects?
#293: FILE: drivers/net/cnxk/cn20k_rx.h:40:
+#define CQE_PTR_DIFF(b, i, o, f)                                                                   \
+	(((f) & NIX_RX_VWQE_F) ? (uint64_t *)(((uintptr_t)((uint64_t *)(b))[i]) - (o)) :           \
+				 (uint64_t *)(((uintptr_t)(b)) + CQE_SZ(i) - (o)))

WARNING:BRACES: braces {} are not necessary for single statement blocks
#308: FILE: drivers/net/cnxk/cn20k_rx.h:55:
+	if (m->nb_segs == 1 && m->next) {
+		rte_panic("mbuf->next[%p] valid when mbuf->nb_segs is %d", m->next, m->nb_segs);
+	}

total: 0 errors, 1 warnings, 6 checks, 756 lines checked
Warning in drivers/net/cnxk/cn20k_rx.h:
Using rte_panic/rte_exit