|WARNING| pw171164 [PATCH v7 15/25] net/ixgbe: reimplement FDIR parser

[checkpatch at dpdk.org]

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

_coding style issues_


ERROR:CODE_INDENT: code indent should use tabs where possible
#3588: FILE: drivers/net/intel/ixgbe/ixgbe_flow_fdir.c:572:
+^I^I^I               FLOW_GRAPH_NODE_EXPECT_SPEC,$

ERROR:CODE_INDENT: code indent should use tabs where possible
#3616: FILE: drivers/net/intel/ixgbe/ixgbe_flow_fdir.c:600:
+^I^I^I               FLOW_GRAPH_NODE_EXPECT_SPEC,$

ERROR:CODE_INDENT: code indent should use tabs where possible
#3624: FILE: drivers/net/intel/ixgbe/ixgbe_flow_fdir.c:608:
+^I^I^I               FLOW_GRAPH_NODE_EXPECT_SPEC,$

WARNING:LONG_LINE_COMMENT: line length of 115 exceeds 100 columns
#3725: FILE: drivers/net/intel/ixgbe/ixgbe_flow_fdir.c:709:
+ * Pattern: START -> [OUTER_ETH] -> (OUTER_IPv4|OUTER_IPv6) -> [UDP] -> (VXLAN|NVGRE) -> INNER_ETH -> [VLAN] -> END

WARNING:LONG_LINE_COMMENT: line length of 104 exceeds 100 columns
#3726: FILE: drivers/net/intel/ixgbe/ixgbe_flow_fdir.c:710:
+ * VxLAN:  START -> [OUTER_ETH] -> (OUTER_IPv4|OUTER_IPv6) -> UDP -> VXLAN -> INNER_ETH -> [VLAN] -> END

WARNING:BRACES: braces {} are not necessary for single statement blocks
#3909: FILE: drivers/net/intel/ixgbe/ixgbe_flow_fdir.c:893:
+		if (eth_mask->hdr.dst_addr.addr_bytes[j] == 0xFF) {
+			rule->mask.mac_addr_byte_mask |= RTE_BIT32(j);
+		}

CHECK:UNNECESSARY_PARENTHESES: Unnecessary parentheses around 'fdir_ctx->rule.mode == RTE_FDIR_MODE_SIGNATURE'
#4224: FILE: drivers/net/intel/ixgbe/ixgbe_flow_fdir.c:1208:
+	if ((fdir_ctx->rule.mode == RTE_FDIR_MODE_SIGNATURE) &&
+	    (fdir_ctx->fwd_action->type == RTE_FLOW_ACTION_TYPE_DROP)) {

CHECK:UNNECESSARY_PARENTHESES: Unnecessary parentheses around 'fdir_ctx->fwd_action->type == RTE_FLOW_ACTION_TYPE_DROP'
#4224: FILE: drivers/net/intel/ixgbe/ixgbe_flow_fdir.c:1208:
+	if ((fdir_ctx->rule.mode == RTE_FDIR_MODE_SIGNATURE) &&
+	    (fdir_ctx->fwd_action->type == RTE_FLOW_ACTION_TYPE_DROP)) {

WARNING:LONG_LINE: line length of 102 exceeds 100 columns
#4299: FILE: drivers/net/intel/ixgbe/ixgbe_flow_fdir.c:1283:
+	if (fdir_ctx->aux_action != NULL && fdir_ctx->aux_action->type == RTE_FLOW_ACTION_TYPE_MARK) {

total: 3 errors, 4 warnings, 2 checks, 4449 lines checked