[dpdk-test-report] |WARNING| pw27373 [PATCH v1 07/21] net/mlx5: add reference counter on memory region

[checkpatch at dpdk.org]

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

_coding style issues_


ERROR:POINTER_LOCATION: "foo* bar" should be "foo *bar"
#46: FILE: drivers/net/mlx5/mlx5.h:295:
+struct mlx5_mr* priv_mr_new(struct priv *, struct rte_mempool *);

WARNING:FUNCTION_ARGUMENTS: function definition argument 'struct priv *' should also have an identifier name
#46: FILE: drivers/net/mlx5/mlx5.h:295:
+struct mlx5_mr* priv_mr_new(struct priv *, struct rte_mempool *);

WARNING:FUNCTION_ARGUMENTS: function definition argument 'struct rte_mempool *' should also have an identifier name
#46: FILE: drivers/net/mlx5/mlx5.h:295:
+struct mlx5_mr* priv_mr_new(struct priv *, struct rte_mempool *);

ERROR:POINTER_LOCATION: "foo* bar" should be "foo *bar"
#47: FILE: drivers/net/mlx5/mlx5.h:296:
+struct mlx5_mr* priv_mr_get(struct priv *, struct rte_mempool *);

WARNING:FUNCTION_ARGUMENTS: function definition argument 'struct priv *' should also have an identifier name
#47: FILE: drivers/net/mlx5/mlx5.h:296:
+struct mlx5_mr* priv_mr_get(struct priv *, struct rte_mempool *);

WARNING:FUNCTION_ARGUMENTS: function definition argument 'struct rte_mempool *' should also have an identifier name
#47: FILE: drivers/net/mlx5/mlx5.h:296:
+struct mlx5_mr* priv_mr_get(struct priv *, struct rte_mempool *);

WARNING:FUNCTION_ARGUMENTS: function definition argument 'struct priv *' should also have an identifier name
#48: FILE: drivers/net/mlx5/mlx5.h:297:
+int priv_mr_release(struct priv *, struct mlx5_mr *);

WARNING:FUNCTION_ARGUMENTS: function definition argument 'struct mlx5_mr *' should also have an identifier name
#48: FILE: drivers/net/mlx5/mlx5.h:297:
+int priv_mr_release(struct priv *, struct mlx5_mr *);

WARNING:FUNCTION_ARGUMENTS: function definition argument 'struct priv *' should also have an identifier name
#49: FILE: drivers/net/mlx5/mlx5.h:298:
+int priv_mr_verify(struct priv *);

WARNING:USE_NEGATIVE_ERRNO: return of an errno should typically be negative (ie: return -EBUSY)
#284: FILE: drivers/net/mlx5/mlx5_mr.c:344:
+	return EBUSY;

ERROR:POINTER_LOCATION: "(foo*)" should be "(foo *)"
#302: FILE: drivers/net/mlx5/mlx5_mr.c:362:
+		DEBUG("%p: mr %p still referenced", (void*)priv,

ERROR:POINTER_LOCATION: "(foo*)" should be "(foo *)"
#303: FILE: drivers/net/mlx5/mlx5_mr.c:363:
+		      (void*)mr);

total: 4 errors, 8 warnings, 0 checks, 404 lines checked