|WARNING| pw169680 [PATCH v3 10/19] net/ixgbe: reimplement FDIR parser

[checkpatch at dpdk.org]

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

_coding style issues_


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

ERROR:CODE_INDENT: code indent should use tabs where possible
#3019: FILE: drivers/net/intel/ixgbe/ixgbe_flow_fdir.c:604:
+^I^I^I               FLOW_GRAPH_NODE_EXPECT_MASK |$

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

ERROR:CODE_INDENT: code indent should use tabs where possible
#3028: FILE: drivers/net/intel/ixgbe/ixgbe_flow_fdir.c:613:
+^I^I^I               FLOW_GRAPH_NODE_EXPECT_MASK |$

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

WARNING:LONG_LINE_COMMENT: line length of 115 exceeds 100 columns
#3131: FILE: drivers/net/intel/ixgbe/ixgbe_flow_fdir.c:716:
+ * 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
#3132: FILE: drivers/net/intel/ixgbe/ixgbe_flow_fdir.c:717:
+ * VxLAN:  START -> [OUTER_ETH] -> (OUTER_IPv4|OUTER_IPv6) -> UDP -> VXLAN -> INNER_ETH -> [VLAN] -> END

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

CHECK:UNNECESSARY_PARENTHESES: Unnecessary parentheses around 'fdir_ctx->rule.mode == RTE_FDIR_MODE_SIGNATURE'
#3599: FILE: drivers/net/intel/ixgbe/ixgbe_flow_fdir.c:1184:
+	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'
#3599: FILE: drivers/net/intel/ixgbe/ixgbe_flow_fdir.c:1184:
+	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
#3669: FILE: drivers/net/intel/ixgbe/ixgbe_flow_fdir.c:1254:
+	if (fdir_ctx->aux_action != NULL && fdir_ctx->aux_action->type == RTE_FLOW_ACTION_TYPE_MARK) {

total: 5 errors, 4 warnings, 2 checks, 3910 lines checked