From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: Received: from dpdk.org (dpdk.org [92.243.14.124]) by inbox.dpdk.org (Postfix) with ESMTP id 414C6A04B5 for ; Thu, 10 Sep 2020 09:43:12 +0200 (CEST) Received: from [92.243.14.124] (localhost [127.0.0.1]) by dpdk.org (Postfix) with ESMTP id 3B5C51B9B7; Thu, 10 Sep 2020 09:43:12 +0200 (CEST) Received: by dpdk.org (Postfix, from userid 1017) id AD22E1BEB3; Thu, 10 Sep 2020 09:43:10 +0200 (CEST) In-Reply-To: <1599723455-353059-4-git-send-email-simei.su@intel.com> References: <1599723455-353059-4-git-send-email-simei.su@intel.com> To: test-report@dpdk.org Cc: Simei Su Message-Id: <20200910074310.AD22E1BEB3@dpdk.org> Date: Thu, 10 Sep 2020 09:43:10 +0200 (CEST) From: checkpatch@dpdk.org Subject: [dpdk-test-report] |WARNING| pw77127 [PATCH v1 3/3] net/ice: support ACL filter in DCF X-BeenThere: test-report@dpdk.org X-Mailman-Version: 2.1.15 Precedence: list List-Id: automatic DPDK test reports List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , Errors-To: test-report-bounces@dpdk.org Sender: "test-report" Test-Label: checkpatch Test-Status: WARNING http://dpdk.org/patch/77127 _coding style issues_ CHECK:UNNECESSARY_PARENTHESES: Unnecessary parentheses around 'ipv4_mask->hdr.src_addr == UINT32_MAX' #891: FILE: drivers/net/ice/ice_acl_filter.c:810: + if ((ipv4_mask->hdr.src_addr == UINT32_MAX) || + (ipv4_mask->hdr.dst_addr == UINT32_MAX)) { CHECK:UNNECESSARY_PARENTHESES: Unnecessary parentheses around 'ipv4_mask->hdr.dst_addr == UINT32_MAX' #891: FILE: drivers/net/ice/ice_acl_filter.c:810: + if ((ipv4_mask->hdr.src_addr == UINT32_MAX) || + (ipv4_mask->hdr.dst_addr == UINT32_MAX)) { CHECK:UNNECESSARY_PARENTHESES: Unnecessary parentheses around 'ipv4_spec->hdr.src_addr != 0' #900: FILE: drivers/net/ice/ice_acl_filter.c:819: + if ((ipv4_spec->hdr.src_addr != 0) && + (ipv4_mask->hdr.src_addr != 0)) { CHECK:UNNECESSARY_PARENTHESES: Unnecessary parentheses around 'ipv4_mask->hdr.src_addr != 0' #900: FILE: drivers/net/ice/ice_acl_filter.c:819: + if ((ipv4_spec->hdr.src_addr != 0) && + (ipv4_mask->hdr.src_addr != 0)) { CHECK:UNNECESSARY_PARENTHESES: Unnecessary parentheses around 'ipv4_spec->hdr.dst_addr != 0' #910: FILE: drivers/net/ice/ice_acl_filter.c:829: + if ((ipv4_spec->hdr.dst_addr != 0) && + (ipv4_mask->hdr.dst_addr != 0)) { CHECK:UNNECESSARY_PARENTHESES: Unnecessary parentheses around 'ipv4_mask->hdr.dst_addr != 0' #910: FILE: drivers/net/ice/ice_acl_filter.c:829: + if ((ipv4_spec->hdr.dst_addr != 0) && + (ipv4_mask->hdr.dst_addr != 0)) { CHECK:UNNECESSARY_PARENTHESES: Unnecessary parentheses around 'tcp_mask->hdr.src_port == UINT16_MAX' #943: FILE: drivers/net/ice/ice_acl_filter.c:862: + if ((tcp_mask->hdr.src_port == UINT16_MAX) || + (tcp_mask->hdr.dst_port == UINT16_MAX)) { CHECK:UNNECESSARY_PARENTHESES: Unnecessary parentheses around 'tcp_mask->hdr.dst_port == UINT16_MAX' #943: FILE: drivers/net/ice/ice_acl_filter.c:862: + if ((tcp_mask->hdr.src_port == UINT16_MAX) || + (tcp_mask->hdr.dst_port == UINT16_MAX)) { CHECK:UNNECESSARY_PARENTHESES: Unnecessary parentheses around 'tcp_mask->hdr.src_port == UINT16_MAX' #952: FILE: drivers/net/ice/ice_acl_filter.c:871: + if ((tcp_mask->hdr.src_port == UINT16_MAX) || + (tcp_mask->hdr.dst_port == UINT16_MAX)) { CHECK:UNNECESSARY_PARENTHESES: Unnecessary parentheses around 'tcp_mask->hdr.dst_port == UINT16_MAX' #952: FILE: drivers/net/ice/ice_acl_filter.c:871: + if ((tcp_mask->hdr.src_port == UINT16_MAX) || + (tcp_mask->hdr.dst_port == UINT16_MAX)) { CHECK:UNNECESSARY_PARENTHESES: Unnecessary parentheses around 'l3 == RTE_FLOW_ITEM_TYPE_IPV4' #961: FILE: drivers/net/ice/ice_acl_filter.c:880: + if ((l3 == RTE_FLOW_ITEM_TYPE_IPV4) && + (tcp_spec->hdr.src_port != 0)) { CHECK:UNNECESSARY_PARENTHESES: Unnecessary parentheses around 'tcp_spec->hdr.src_port != 0' #961: FILE: drivers/net/ice/ice_acl_filter.c:880: + if ((l3 == RTE_FLOW_ITEM_TYPE_IPV4) && + (tcp_spec->hdr.src_port != 0)) { CHECK:UNNECESSARY_PARENTHESES: Unnecessary parentheses around 'l3 == RTE_FLOW_ITEM_TYPE_IPV4' #970: FILE: drivers/net/ice/ice_acl_filter.c:889: + if ((l3 == RTE_FLOW_ITEM_TYPE_IPV4) && + (tcp_spec->hdr.dst_port != 0)) { CHECK:UNNECESSARY_PARENTHESES: Unnecessary parentheses around 'tcp_spec->hdr.dst_port != 0' #970: FILE: drivers/net/ice/ice_acl_filter.c:889: + if ((l3 == RTE_FLOW_ITEM_TYPE_IPV4) && + (tcp_spec->hdr.dst_port != 0)) { CHECK:UNNECESSARY_PARENTHESES: Unnecessary parentheses around 'udp_mask->hdr.src_port == UINT16_MAX' #999: FILE: drivers/net/ice/ice_acl_filter.c:918: + if ((udp_mask->hdr.src_port == UINT16_MAX) || + (udp_mask->hdr.dst_port == UINT16_MAX)) { CHECK:UNNECESSARY_PARENTHESES: Unnecessary parentheses around 'udp_mask->hdr.dst_port == UINT16_MAX' #999: FILE: drivers/net/ice/ice_acl_filter.c:918: + if ((udp_mask->hdr.src_port == UINT16_MAX) || + (udp_mask->hdr.dst_port == UINT16_MAX)) { CHECK:UNNECESSARY_PARENTHESES: Unnecessary parentheses around 'l3 == RTE_FLOW_ITEM_TYPE_IPV4' #1009: FILE: drivers/net/ice/ice_acl_filter.c:928: + if ((l3 == RTE_FLOW_ITEM_TYPE_IPV4) && + (udp_spec->hdr.src_port != 0)) { CHECK:UNNECESSARY_PARENTHESES: Unnecessary parentheses around 'udp_spec->hdr.src_port != 0' #1009: FILE: drivers/net/ice/ice_acl_filter.c:928: + if ((l3 == RTE_FLOW_ITEM_TYPE_IPV4) && + (udp_spec->hdr.src_port != 0)) { CHECK:UNNECESSARY_PARENTHESES: Unnecessary parentheses around 'l3 == RTE_FLOW_ITEM_TYPE_IPV4' #1018: FILE: drivers/net/ice/ice_acl_filter.c:937: + if ((l3 == RTE_FLOW_ITEM_TYPE_IPV4) && + (udp_spec->hdr.dst_port != 0)) { CHECK:UNNECESSARY_PARENTHESES: Unnecessary parentheses around 'udp_spec->hdr.dst_port != 0' #1018: FILE: drivers/net/ice/ice_acl_filter.c:937: + if ((l3 == RTE_FLOW_ITEM_TYPE_IPV4) && + (udp_spec->hdr.dst_port != 0)) { total: 0 errors, 0 warnings, 20 checks, 1119 lines checked