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 8EAFCA32A3 for ; Fri, 25 Oct 2019 14:34:30 +0200 (CEST) Received: from [92.243.14.124] (localhost [127.0.0.1]) by dpdk.org (Postfix) with ESMTP id EDD8E1C190; Fri, 25 Oct 2019 14:34:29 +0200 (CEST) Received: from huawei.com (szxga07-in.huawei.com [45.249.212.35]) by dpdk.org (Postfix) with ESMTP id 2B8AD1C18F for ; Fri, 25 Oct 2019 14:34:28 +0200 (CEST) Received: from DGGEMS408-HUB.china.huawei.com (unknown [172.30.72.60]) by Forcepoint Email with ESMTP id 4D89C485754EE165150A; Fri, 25 Oct 2019 20:34:25 +0800 (CST) Received: from [127.0.0.1] (10.57.115.182) by DGGEMS408-HUB.china.huawei.com (10.3.19.208) with Microsoft SMTP Server id 14.3.439.0; Fri, 25 Oct 2019 20:34:24 +0800 To: Ferruh Yigit , "Wei Hu (Xavier)" , References: <20191009141653.39364-1-xavier.huwei@tom.com> <20191009141653.39364-4-xavier.huwei@tom.com> From: "Wei Hu (Xavier)" Message-ID: Date: Fri, 25 Oct 2019 20:34:24 +0800 User-Agent: Mozilla/5.0 (Windows NT 6.1; WOW64; rv:45.0) Gecko/20100101 Thunderbird/45.7.1 MIME-Version: 1.0 In-Reply-To: Content-Type: text/plain; charset="utf-8" Content-Transfer-Encoding: 7bit X-Originating-IP: [10.57.115.182] X-CFilter-Loop: Reflected Subject: Re: [dpdk-dev] [PATCH 3/4] net/hns3: Renew command and desc structure X-BeenThere: dev@dpdk.org X-Mailman-Version: 2.1.15 Precedence: list List-Id: DPDK patches and discussions List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , Errors-To: dev-bounces@dpdk.org Sender: "dev" Hi, Ferruh Yigit On 2019/10/15 16:45, Ferruh Yigit wrote: > On 10/9/2019 3:16 PM, Wei Hu (Xavier) wrote: >> From: humin >> >> This patch adds commands and modifies descriptor structures for >> accessing manage table and mac table. >> >> Signed-off-by: humin > Can you please provide a "Name Surname " format? > >> Signed-off-by: Wei Hu (Xavier) >> --- >> drivers/net/hns3/hns3_cmd.h | 9 ++++++--- >> 1 file changed, 6 insertions(+), 3 deletions(-) >> >> diff --git a/drivers/net/hns3/hns3_cmd.h b/drivers/net/hns3/hns3_cmd.h >> index be0ecbe86..13a3b87d5 100644 >> --- a/drivers/net/hns3/hns3_cmd.h >> +++ b/drivers/net/hns3/hns3_cmd.h >> @@ -217,6 +217,8 @@ enum hns3_opcode_type { >> /* PPP module intr commands */ >> HNS3_PPP_CMD0_INT_CMD = 0x2100, >> HNS3_PPP_CMD1_INT_CMD = 0x2101, >> + HNS3_PPP_MAC_VLAN_IDX_RD = 0x2104, >> + HNS3_MAC_ETHERTYPE_IDX_RD = 0x2105, >> }; >> >> #define HNS3_CMD_FLAG_IN BIT(0) >> @@ -642,7 +644,7 @@ struct hns3_mac_mgr_tbl_entry_cmd { >> uint16_t vlan_tag; >> uint32_t mac_addr_hi32; >> uint16_t mac_addr_lo16; >> - uint16_t rsv1; >> + uint16_t index; >> uint16_t ethter_type; >> uint16_t egress_port; >> uint16_t egress_queue; >> @@ -707,12 +709,13 @@ struct hns3_mac_vlan_tbl_entry_cmd { >> uint16_t vlan_tag; >> uint32_t mac_addr_hi32; >> uint16_t mac_addr_lo16; >> - uint16_t rsv1; >> + uint16_t port; >> uint8_t entry_type; >> uint8_t mc_mac_en; >> uint16_t egress_port; >> uint16_t egress_queue; >> - uint8_t rsv2[6]; >> + uint8_t rsv2[2]; >> + uint32_t index; >> }; >> > Aren't these new fields and defines used at all? If not why added? In fact ,this patch used for some query function in future. We will remove this patch from this series. Thanks for your comments. Regards Xavier > >