From mboxrd@z Thu Jan 1 00:00:00 1970 From: Stephen Hemminger Subject: Re: [PATCH net-next v3] Add support of Cavium Liquidio ethernet adapters Date: Thu, 18 Dec 2014 23:01:21 -0800 Message-ID: <20141218230121.6bd27e65@urahara> References: <1418959519-31681-1-git-send-email-rvatsavayi@caviumnetworks.com> Mime-Version: 1.0 Content-Type: text/plain; charset=US-ASCII Content-Transfer-Encoding: 7bit Cc: , , Derek Chickles , Satanand Burla , Felix Manlunas , Raghu Vatsavayi To: Raghu Vatsavayi Return-path: Received: from mail-pa0-f41.google.com ([209.85.220.41]:40925 "EHLO mail-pa0-f41.google.com" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S1751194AbaLSHBd (ORCPT ); Fri, 19 Dec 2014 02:01:33 -0500 Received: by mail-pa0-f41.google.com with SMTP id rd3so637265pab.0 for ; Thu, 18 Dec 2014 23:01:32 -0800 (PST) In-Reply-To: <1418959519-31681-1-git-send-email-rvatsavayi@caviumnetworks.com> Sender: netdev-owner@vger.kernel.org List-ID: On Thu, 18 Dec 2014 19:25:19 -0800 Raghu Vatsavayi wrote: > + > +static u32 lio_get_link(struct net_device *dev) > +{ > + u32 ret; > + > + ret = netif_carrier_ok(dev) ? 1 : 0; > + return ret; > +} > + This is unnecessary, this is what the default ethtool handler already does if you give a NULL handler.