|WARNING| pw118626 [PATCH v4 08/18] net/mlx5: add HW steering counter action

checkpatch at dpdk.org checkpatch at dpdk.org
Wed Oct 19 18:30:01 CEST 2022


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

_coding style issues_


WARNING:REPEATED_WORD: Possible repeated word: 'the'
#1445: FILE: drivers/net/mlx5/mlx5_hws_cnt.c:39:
+	 * In the end, user fetch the the counter from minimal to the maximum.

CHECK:CAMELCASE: Avoid CamelCase: <PRIu64>
#1596: FILE: drivers/net/mlx5/mlx5_hws_cnt.c:190:
+		DRV_LOG(ERR, "counter number %"PRIu64" is out of 32bit range",

CHECK:CONCATENATED_STRING: Concatenated strings should use spaces between elements
#1596: FILE: drivers/net/mlx5/mlx5_hws_cnt.c:190:
+		DRV_LOG(ERR, "counter number %"PRIu64" is out of 32bit range",

WARNING:TYPO_SPELLING: 'alloced' may be misspelled - perhaps 'allocated'?
#1708: FILE: drivers/net/mlx5/mlx5_hws_cnt.c:302:
+	uint32_t idx, alloced = 0;

WARNING:TYPO_SPELLING: 'alloced' may be misspelled - perhaps 'allocated'?
#1734: FILE: drivers/net/mlx5/mlx5_hws_cnt.c:328:
+	alloced = cpool->dcs_mng.dcs[0].batch_sz;

WARNING:TYPO_SPELLING: 'alloced' may be misspelled - perhaps 'allocated'?
#1745: FILE: drivers/net/mlx5/mlx5_hws_cnt.c:339:
+			cpool->dcs_mng.dcs[idx].iidx = alloced;

WARNING:TYPO_SPELLING: 'alloced' may be misspelled - perhaps 'allocated'?
#1746: FILE: drivers/net/mlx5/mlx5_hws_cnt.c:340:
+			alloced += cpool->dcs_mng.dcs[idx].batch_sz;

CHECK:CAMELCASE: Avoid CamelCase: <PRIu32>
#1753: FILE: drivers/net/mlx5/mlx5_hws_cnt.c:347:
+		"Cannot alloc device counter, allocated[%" PRIu32 "] request[%" PRIu32 "]",

WARNING:TYPO_SPELLING: 'alloced' may be misspelled - perhaps 'allocated'?
#1754: FILE: drivers/net/mlx5/mlx5_hws_cnt.c:348:
+		alloced, cnt_num);

WARNING:TYPO_SPELLING: 'master' may be misspelled - perhaps 'primary'?
#1791: FILE: drivers/net/mlx5/mlx5_hws_cnt.c:385:
+	if (priv->sh->config.dv_esw_en && priv->master)

total: 0 errors, 7 warnings, 3 checks, 2197 lines checked


More information about the test-report mailing list