|WARNING| pw171171 [PATCH v7 22/25] net/i40e: reimplement FDIR parser

[checkpatch at dpdk.org]

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

_coding style issues_


WARNING:LONG_LINE_COMMENT: line length of 101 exceeds 100 columns
#3594: FILE: drivers/net/intel/i40e/i40e_flow_fdir.c:48:
+ * Pattern: START -> ETH -> [VLAN] -> (IPv4 | IPv6) -> [TCP | UDP | SCTP | ESP | L2TPv3 | GTP] -> END

WARNING:LONG_LINE: line length of 125 exceeds 100 columns
#4030: FILE: drivers/net/intel/i40e/i40e_flow_fdir.c:484:
+		memcpy(&fdir_filter->input.flow.ipv6_flow.src_ip, &ipv6_spec->hdr.src_addr, sizeof(ipv6_spec->hdr.src_addr));

WARNING:LONG_LINE: line length of 125 exceeds 100 columns
#4034: FILE: drivers/net/intel/i40e/i40e_flow_fdir.c:488:
+		memcpy(&fdir_filter->input.flow.ipv6_flow.dst_ip, &ipv6_spec->hdr.dst_addr, sizeof(ipv6_spec->hdr.dst_addr));

WARNING:BRACES: braces {} are not necessary for any arm of this statement
#4114: FILE: drivers/net/intel/i40e/i40e_flow_fdir.c:568:
+	if (fdir_filter->input.flow_ext.oip_type == I40E_FDIR_IPTYPE_IPV4) {
[...]
+	} else if (fdir_filter->input.flow_ext.oip_type == I40E_FDIR_IPTYPE_IPV6) {
[...]

WARNING:BRACES: braces {} are not necessary for any arm of this statement
#4200: FILE: drivers/net/intel/i40e/i40e_flow_fdir.c:654:
+	if (fdir_filter->input.flow_ext.oip_type == I40E_FDIR_IPTYPE_IPV4) {
[...]
+	} else if (fdir_filter->input.flow_ext.oip_type == I40E_FDIR_IPTYPE_IPV6) {
[...]

WARNING:BRACES: braces {} are not necessary for any arm of this statement
#4290: FILE: drivers/net/intel/i40e/i40e_flow_fdir.c:744:
+	if (fdir_filter->input.flow_ext.oip_type == I40E_FDIR_IPTYPE_IPV4) {
[...]
+	} else if (fdir_filter->input.flow_ext.oip_type == I40E_FDIR_IPTYPE_IPV6) {
[...]

WARNING:BRACES: braces {} are not necessary for single statement blocks
#4299: FILE: drivers/net/intel/i40e/i40e_flow_fdir.c:753:
+	if (!CI_FIELD_IS_ZERO(&sctp_mask->hdr.tag)) {
+		fdir_filter->input.flow_ext.input_set |= I40E_INSET_SCTP_VT;
+	}

WARNING:BRACES: braces {} are not necessary for any arm of this statement
#4576: FILE: drivers/net/intel/i40e/i40e_flow_fdir.c:1030:
+		if (is_udp)
[...]
+		else {
[...]

CHECK:BRACES: Unbalanced braces around else statement
#4578: FILE: drivers/net/intel/i40e/i40e_flow_fdir.c:1032:
+		else {

WARNING:BRACES: braces {} are not necessary for any arm of this statement
#4582: FILE: drivers/net/intel/i40e/i40e_flow_fdir.c:1036:
+		if (is_udp)
[...]
+		else {
[...]

CHECK:BRACES: Unbalanced braces around else statement
#4584: FILE: drivers/net/intel/i40e/i40e_flow_fdir.c:1038:
+		else {

CHECK:BRACES: Blank lines aren't necessary before a close brace '}'
#4606: FILE: drivers/net/intel/i40e/i40e_flow_fdir.c:1060:
+
+}

WARNING:BRACES: braces {} are not necessary for any arm of this statement
#4622: FILE: drivers/net/intel/i40e/i40e_flow_fdir.c:1076:
+	if (fdir_filter->input.flow_ext.oip_type == I40E_FDIR_IPTYPE_IPV4) {
[...]
+	} else if (fdir_filter->input.flow_ext.oip_type == I40E_FDIR_IPTYPE_IPV6) {
[...]

WARNING:BRACES: braces {} are not necessary for single statement blocks
#5140: FILE: drivers/net/intel/i40e/i40e_flow_fdir.c:1594:
+	if (ret) {
+		return ret;
+	}

WARNING:BRACES: braces {} are not necessary for single statement blocks
#5145: FILE: drivers/net/intel/i40e/i40e_flow_fdir.c:1599:
+	if (ret) {
+		return ret;
+	}

CHECK:BOOL_COMPARISON: Using comparison to false is error prone
#5396: FILE: drivers/net/intel/i40e/i40e_flow_fdir.c:1850:
+	needs_wait = fdir_flow->installed == false &&

total: 0 errors, 12 warnings, 4 checks, 5541 lines checked