[dpdk-test-report] |WARNING| pw26404 [PATCH v2 36/40] net/dpaa: add support for packet type parsing

checkpatch at dpdk.org checkpatch at dpdk.org
Wed Jul 5 02:11:00 CEST 2017


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

_coding style issues_


ERROR:SPACING: space required after that close brace '}'
#334: FILE: drivers/net/dpaa/dpaa_rxtx.h:148:
+		}__attribute__((__packed__));

ERROR:COMPLEX_MACRO: Macros with complex values should be enclosed in parentheses
#362: FILE: drivers/net/dpaa/dpaa_rxtx.h:176:
+#define first_ip_option        first_ip_err_code & 0x01

ERROR:COMPLEX_MACRO: Macros with complex values should be enclosed in parentheses
#363: FILE: drivers/net/dpaa/dpaa_rxtx.h:177:
+#define first_unknown_ip_proto first_ip_err_code & 0x02

ERROR:COMPLEX_MACRO: Macros with complex values should be enclosed in parentheses
#364: FILE: drivers/net/dpaa/dpaa_rxtx.h:178:
+#define first_fragmented       first_ip_err_code & 0x04

ERROR:COMPLEX_MACRO: Macros with complex values should be enclosed in parentheses
#365: FILE: drivers/net/dpaa/dpaa_rxtx.h:179:
+#define first_ip_type          first_ip_err_code & 0x18

ERROR:SPACING: space required after that close brace '}'
#367: FILE: drivers/net/dpaa/dpaa_rxtx.h:181:
+		}__attribute__((__packed__));

CHECK:MACRO_ARG_PRECEDENCE: Macro argument '_buf' may be better as '(_buf)' to avoid precedence issues
#411: FILE: drivers/net/dpaa/dpaa_rxtx.h:225:
+#define GET_RX_PRS(_buf) \
+	(struct dpaa_eth_parse_results_t *)((uint8_t *)_buf + DEFAULT_RX_ICEOF)

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


More information about the test-report mailing list