From mboxrd@z Thu Jan 1 00:00:00 1970 From: Shreyansh Jain Subject: Re: [PATCH v4 05/15] net/dpaa2: use rte_eth_linkstatus_set Date: Tue, 16 Jan 2018 15:14:39 +0530 Message-ID: References: <20180108174514.14688-1-stephen@networkplumber.org> <20180111170658.2809-1-ferruh.yigit@intel.com> <20180111170658.2809-5-ferruh.yigit@intel.com> Mime-Version: 1.0 Content-Type: text/plain; charset="utf-8"; format=flowed Content-Transfer-Encoding: 7bit Cc: Bruce Richardson , Konstantin Ananyev , To: Ferruh Yigit , Hemant Agrawal , Stephen Hemminger Return-path: Received: from NAM02-BL2-obe.outbound.protection.outlook.com (mail-bl2nam02on0050.outbound.protection.outlook.com [104.47.38.50]) by dpdk.org (Postfix) with ESMTP id 3B36A44CF for ; Tue, 16 Jan 2018 10:30:30 +0100 (CET) In-Reply-To: <20180111170658.2809-5-ferruh.yigit@intel.com> Content-Language: en-US List-Id: DPDK patches and discussions List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , Errors-To: dev-bounces@dpdk.org Sender: "dev" On Thursday 11 January 2018 10:36 PM, Ferruh Yigit wrote: > From: Stephen Hemminger > > Use new helper function to update the link status. > As a good side effect this fixes a but because this driver was not ^^^^^^ needs rephrasing > returning correct status (should be -1 in link_status changed). > > Signed-off-by: Stephen Hemminger > --- > drivers/net/dpaa2/dpaa2_ethdev.c | 65 +++++----------------------------------- > 1 file changed, 7 insertions(+), 58 deletions(-) Other than the change in commit message highlighted above: Acked-by: Shreyansh Jain Thanks.