|WARNING| pw155853 [PATCH v5 2/2] net/zxdh: add support flow director ops

[checkpatch at dpdk.org]

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

_coding style issues_


WARNING:TYPO_SPELLING: 'stdio' may be misspelled - perhaps 'studio'?
#333: FILE: drivers/net/zxdh/zxdh_flow.c:6:
+#include <stdio.h>

CHECK:MACRO_ARG_REUSE: Macro argument reuse 'buf_size' - possible side-effects?
#375: FILE: drivers/net/zxdh/zxdh_flow.c:48:
+#define MKDUMPSTR(buf, buf_size, cur_len, ...) \
+do { \
+	if ((cur_len) >= (buf_size)) \
+		break; \
+	(cur_len) += snprintf((buf) + (cur_len), (buf_size) - (cur_len), __VA_ARGS__); \
+} while (0)

CHECK:MACRO_ARG_REUSE: Macro argument reuse 'cur_len' - possible side-effects?
#375: FILE: drivers/net/zxdh/zxdh_flow.c:48:
+#define MKDUMPSTR(buf, buf_size, cur_len, ...) \
+do { \
+	if ((cur_len) >= (buf_size)) \
+		break; \
+	(cur_len) += snprintf((buf) + (cur_len), (buf_size) - (cur_len), __VA_ARGS__); \
+} while (0)

CHECK:CAMELCASE: Avoid CamelCase: <IPv6_BYTES_FMT>
#453: FILE: drivers/net/zxdh/zxdh_flow.c:126:
+		"L3	  dstip=0x%08x 0x%08x 0x%08x 0x%08x("IPv6_BYTES_FMT")
",

CHECK:CONCATENATED_STRING: Concatenated strings should use spaces between elements
#453: FILE: drivers/net/zxdh/zxdh_flow.c:126:
+		"L3	  dstip=0x%08x 0x%08x 0x%08x 0x%08x("IPv6_BYTES_FMT")
",

CHECK:CAMELCASE: Avoid CamelCase: <IPv6_BYTES>
#457: FILE: drivers/net/zxdh/zxdh_flow.c:130:
+		IPv6_BYTES(key->dst_ip));

CHECK:CONCATENATED_STRING: Concatenated strings should use spaces between elements
#459: FILE: drivers/net/zxdh/zxdh_flow.c:132:
+		"	  srcip=0x%08x 0x%08x 0x%08x 0x%08x("IPv6_BYTES_FMT")
",

CHECK:CAMELCASE: Avoid CamelCase: <IPv4_BYTES>
#2382: FILE: drivers/net/zxdh/zxdh_flow.h:32:
+#ifndef IPv4_BYTES

CHECK:CAMELCASE: Avoid CamelCase: <IPv4_BYTES_FMT>
#2383: FILE: drivers/net/zxdh/zxdh_flow.h:33:
+#define IPv4_BYTES_FMT "%" PRIu8 ".%" PRIu8 ".%" PRIu8 ".%" PRIu8

CHECK:CAMELCASE: Avoid CamelCase: <PRIu8>
#2383: FILE: drivers/net/zxdh/zxdh_flow.h:33:
+#define IPv4_BYTES_FMT "%" PRIu8 ".%" PRIu8 ".%" PRIu8 ".%" PRIu8

ERROR:COMPLEX_MACRO: Macros with complex values should be enclosed in parentheses
#2384: FILE: drivers/net/zxdh/zxdh_flow.h:34:
+#define IPv4_BYTES(addr) \
+		(uint8_t)(((addr) >> 24) & 0xFF),\
+		(uint8_t)(((addr) >> 16) & 0xFF),\
+		(uint8_t)(((addr) >> 8) & 0xFF),\
+		(uint8_t)((addr) & 0xFF)

CHECK:MACRO_ARG_REUSE: Macro argument reuse 'addr' - possible side-effects?
#2384: FILE: drivers/net/zxdh/zxdh_flow.h:34:
+#define IPv4_BYTES(addr) \
+		(uint8_t)(((addr) >> 24) & 0xFF),\
+		(uint8_t)(((addr) >> 16) & 0xFF),\
+		(uint8_t)(((addr) >> 8) & 0xFF),\
+		(uint8_t)((addr) & 0xFF)

ERROR:COMPLEX_MACRO: Macros with complex values should be enclosed in parentheses
#2394: FILE: drivers/net/zxdh/zxdh_flow.h:44:
+#define IPv6_BYTES(addr) \
+	addr[0], addr[1], addr[2], addr[3], addr[4], addr[5], addr[6], addr[7], \
+	addr[8], addr[9], addr[10], addr[11], addr[12], addr[13], addr[14], addr[15]

CHECK:MACRO_ARG_REUSE: Macro argument reuse 'addr' - possible side-effects?
#2394: FILE: drivers/net/zxdh/zxdh_flow.h:44:
+#define IPv6_BYTES(addr) \
+	addr[0], addr[1], addr[2], addr[3], addr[4], addr[5], addr[6], addr[7], \
+	addr[8], addr[9], addr[10], addr[11], addr[12], addr[13], addr[14], addr[15]

total: 2 errors, 1 warnings, 11 checks, 3025 lines checked