From patchwork Fri Aug 30 08:45:29 2019 Content-Type: text/plain; charset="utf-8" MIME-Version: 1.0 Content-Transfer-Encoding: 7bit X-Patchwork-Submitter: Yong Wang X-Patchwork-Id: 58302 X-Patchwork-Delegate: xiaolong.ye@intel.com Return-Path: X-Original-To: patchwork@dpdk.org Delivered-To: patchwork@dpdk.org Received: from [92.243.14.124] (localhost [127.0.0.1]) by dpdk.org (Postfix) with ESMTP id B4C081E8E2; Fri, 30 Aug 2019 11:32:46 +0200 (CEST) Received: from mxhk.zte.com.cn (mxhk.zte.com.cn [63.217.80.70]) by dpdk.org (Postfix) with ESMTP id F05851E8DC for ; Fri, 30 Aug 2019 11:32:44 +0200 (CEST) Received: from mse-fl1.zte.com.cn (unknown [10.30.14.238]) by Forcepoint Email with ESMTPS id 9289D9B529F53AA0754E; Fri, 30 Aug 2019 17:32:43 +0800 (CST) Received: from notes_smtp.zte.com.cn (notessmtp.zte.com.cn [10.30.1.239]) by mse-fl1.zte.com.cn with ESMTP id x7U9VEjI088853; Fri, 30 Aug 2019 17:31:14 +0800 (GMT-8) (envelope-from wang.yong19@zte.com.cn) Received: from 99-99-1-100.lightspeed.wepbfl.sbcglobal.net ([10.43.166.165]) by szsmtp06.zte.com.cn (Lotus Domino Release 8.5.3FP6) with ESMTP id 2019083017315095-3297877 ; Fri, 30 Aug 2019 17:31:50 +0800 From: Yong Wang To: wenzhuo.lu@intel.com Cc: dev@dpdk.org, Yong Wang Date: Fri, 30 Aug 2019 16:45:29 +0800 Message-Id: <1567154729-7183-1-git-send-email-wang.yong19@zte.com.cn> X-Mailer: git-send-email 1.8.3.1 X-MIMETrack: Itemize by SMTP Server on SZSMTP06/server/zte_ltd(Release 8.5.3FP6|November 21, 2013) at 2019-08-30 17:31:51, Serialize by Router on notes_smtp/zte_ltd(Release 9.0.1FP7|August 17, 2016) at 2019-08-30 17:31:19, Serialize complete at 2019-08-30 17:31:19 X-MAIL: mse-fl1.zte.com.cn x7U9VEjI088853 Subject: [dpdk-dev] [PATCH] net/ixgbe: remove redundant assignment 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" Since "link.link_duplex" has been assigned to ETH_LINK_FULL_DUPLEX just before switch statement, the assignment in switch-case statement is redundant. Remove it. Fixes: 82113036e4e5 ("ethdev: redesign link speed config") Signed-off-by: Yong Wang Reviewed-by: Xiaolong Ye --- drivers/net/ixgbe/ixgbe_ethdev.c | 1 - 1 file changed, 1 deletion(-) diff --git a/drivers/net/ixgbe/ixgbe_ethdev.c b/drivers/net/ixgbe/ixgbe_ethdev.c index 03fc1f7..f328d7c 100644 --- a/drivers/net/ixgbe/ixgbe_ethdev.c +++ b/drivers/net/ixgbe/ixgbe_ethdev.c @@ -4138,7 +4138,6 @@ static int ixgbevf_dev_xstats_get_names(__rte_unused struct rte_eth_dev *dev, link.link_speed = ETH_SPEED_NUM_10M; else link.link_speed = ETH_SPEED_NUM_100M; - link.link_duplex = ETH_LINK_FULL_DUPLEX; break; case IXGBE_LINK_SPEED_100_FULL: