[dpdk-test-report] |WARNING| pw65000 [PATCH v1 02/38] mlx5: prepare common library

checkpatch at dpdk.org checkpatch at dpdk.org
Tue Jan 21 10:13:17 CET 2020


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

_coding style issues_


ERROR:COMPLEX_MACRO: Macros with complex values should be enclosed in parentheses
#768: FILE: drivers/common/mlx5/mlx5_common.h:18:
+#define PMD_DRV_LOG_OPAREN (

ERROR:SPACING: space prohibited before that close parenthesis ')'
#769: FILE: drivers/common/mlx5/mlx5_common.h:19:
+#define PMD_DRV_LOG_CPAREN )

ERROR:COMPLEX_MACRO: Macros with complex values should be enclosed in parentheses
#769: FILE: drivers/common/mlx5/mlx5_common.h:19:
+#define PMD_DRV_LOG_CPAREN )

ERROR:SPACING: space prohibited before that ',' (ctx:WxE)
#770: FILE: drivers/common/mlx5/mlx5_common.h:20:
+#define PMD_DRV_LOG_COMMA ,
                           ^

ERROR:COMPLEX_MACRO: Macros with complex values should be enclosed in parentheses
#770: FILE: drivers/common/mlx5/mlx5_common.h:20:
+#define PMD_DRV_LOG_COMMA ,

ERROR:SPACING: space required after that ',' (ctx:VxB)
#788: FILE: drivers/common/mlx5/mlx5_common.h:38:
+			RTE_FMT_HEAD(__VA_ARGS__,), \
 			                        ^

ERROR:SPACING: space required after that ',' (ctx:VxB)
#789: FILE: drivers/common/mlx5/mlx5_common.h:39:
+		RTE_FMT_TAIL(__VA_ARGS__,)))
 		                        ^

ERROR:SPACING: need consistent spacing around '*' (ctx:WxV)
#1823: FILE: drivers/common/mlx5/mlx5_devx_cmds.c:954:
+			void *tx_domain __rte_unused, FILE *file __rte_unused)
 			                                   ^

WARNING:USE_NEGATIVE_ERRNO: return of an errno should typically be negative (ie: return -ENOTSUP)
#2276: FILE: drivers/common/mlx5/mlx5_glue.c:194:
+	return ENOTSUP;

WARNING:USE_NEGATIVE_ERRNO: return of an errno should typically be negative (ie: return -ENOTSUP)
#2335: FILE: drivers/common/mlx5/mlx5_glue.c:253:
+	return ENOTSUP;

WARNING:USE_NEGATIVE_ERRNO: return of an errno should typically be negative (ie: return -ENOTSUP)
#2350: FILE: drivers/common/mlx5/mlx5_glue.c:268:
+	return ENOTSUP;

WARNING:USE_NEGATIVE_ERRNO: return of an errno should typically be negative (ie: return -ENOTSUP)
#2363: FILE: drivers/common/mlx5/mlx5_glue.c:281:
+	return ENOTSUP;

WARNING:USE_NEGATIVE_ERRNO: return of an errno should typically be negative (ie: return -ENOTSUP)
#2388: FILE: drivers/common/mlx5/mlx5_glue.c:306:
+	return ENOTSUP;

WARNING:USE_NEGATIVE_ERRNO: return of an errno should typically be negative (ie: return -ENOTSUP)
#2403: FILE: drivers/common/mlx5/mlx5_glue.c:321:
+	return ENOTSUP;

WARNING:USE_NEGATIVE_ERRNO: return of an errno should typically be negative (ie: return -ENOTSUP)
#2420: FILE: drivers/common/mlx5/mlx5_glue.c:338:
+	return ENOTSUP;

total: 8 errors, 7 warnings, 5942 lines checked


More information about the test-report mailing list