[dpdk-test-report] |WARNING| pw33385 [PATCH v3 4/7] net/mlx5: convert to new Tx offloads API

checkpatch at dpdk.org checkpatch at dpdk.org
Wed Jan 10 10:18:03 CET 2018


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

_coding style issues_


CHECK:CAMELCASE: Avoid CamelCase: <PRIx64>
#163: FILE: drivers/net/mlx5/mlx5_ethdev.c:559:
+		      "requested 0x%" PRIx64 " supported 0x%" PRIx64,

WARNING:USE_NEGATIVE_ERRNO: return of an errno should typically be negative (ie: return -ENOTSUP)
#165: FILE: drivers/net/mlx5/mlx5_ethdev.c:561:
+		return ENOTSUP;

WARNING:FUNCTION_ARGUMENTS: function definition argument 'struct priv *' should also have an identifier name
#260: FILE: drivers/net/mlx5/mlx5_rxtx.h:296:
+uint64_t mlx5_priv_get_tx_port_offloads(struct priv *);

WARNING:FUNCTION_ARGUMENTS: function definition argument 'struct priv *' should also have an identifier name
#270: FILE: drivers/net/mlx5/mlx5_rxtx.h:314:
+int priv_check_raw_vec_tx_support(struct priv *, struct rte_eth_dev *);

WARNING:FUNCTION_ARGUMENTS: function definition argument 'struct rte_eth_dev *' should also have an identifier name
#270: FILE: drivers/net/mlx5/mlx5_rxtx.h:314:
+int priv_check_raw_vec_tx_support(struct priv *, struct rte_eth_dev *);

WARNING:FUNCTION_ARGUMENTS: function definition argument 'struct priv *' should also have an identifier name
#271: FILE: drivers/net/mlx5/mlx5_rxtx.h:315:
+int priv_check_vec_tx_support(struct priv *, struct rte_eth_dev *);

WARNING:FUNCTION_ARGUMENTS: function definition argument 'struct rte_eth_dev *' should also have an identifier name
#271: FILE: drivers/net/mlx5/mlx5_rxtx.h:315:
+int priv_check_vec_tx_support(struct priv *, struct rte_eth_dev *);

total: 0 errors, 6 warnings, 1 checks, 413 lines checked


More information about the test-report mailing list