From mboxrd@z Thu Jan  1 00:00:00 1970
From: Andrew Lunn <andrew@lunn.ch>
Subject: Re: [PATCH net-next 17/19] net: usb: aqc111: Initialize ethtool_ops
 structure
Date: Sat, 6 Oct 2018 19:08:46 +0200
Message-ID: <20181006170846.GD6990@lunn.ch>
References: <cover.1538734658.git.igor.russkikh@aquantia.com>
 <26309478ca61ae4163acb455d62021cf6801c29a.1538734658.git.igor.russkikh@aquantia.com>
Mime-Version: 1.0
Content-Type: text/plain; charset=us-ascii
Cc: "David S . Miller" <davem@davemloft.net>,
        "linux-usb@vger.kernel.org" <linux-usb@vger.kernel.org>,
        "netdev@vger.kernel.org" <netdev@vger.kernel.org>,
        Dmitry Bezrukov <Dmitry.Bezrukov@aquantia.com>
To: Igor Russkikh <Igor.Russkikh@aquantia.com>
Return-path: <netdev-owner@vger.kernel.org>
Received: from vps0.lunn.ch ([185.16.172.187]:55786 "EHLO vps0.lunn.ch"
        rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP
        id S1726256AbeJGAM4 (ORCPT <rfc822;netdev@vger.kernel.org>);
        Sat, 6 Oct 2018 20:12:56 -0400
Content-Disposition: inline
In-Reply-To: <26309478ca61ae4163acb455d62021cf6801c29a.1538734658.git.igor.russkikh@aquantia.com>
Sender: netdev-owner@vger.kernel.org
List-ID: <netdev.vger.kernel.org>

On Fri, Oct 05, 2018 at 10:25:22AM +0000, Igor Russkikh wrote:
> From: Dmitry Bezrukov <dmitry.bezrukov@aquantia.com>
> 
> Implement get_drvinfo, set/get_msglevel, get_link callbacks
> 
> Signed-off-by: Dmitry Bezrukov <dmitry.bezrukov@aquantia.com>
> Signed-off-by: Igor Russkikh <igor.russkikh@aquantia.com>
> ---
>  drivers/net/usb/aqc111.c | 30 ++++++++++++++++++++++++++++++
>  1 file changed, 30 insertions(+)
> 
> diff --git a/drivers/net/usb/aqc111.c b/drivers/net/usb/aqc111.c
> index 9908b0a04da6..ade2b60b4811 100644
> --- a/drivers/net/usb/aqc111.c
> +++ b/drivers/net/usb/aqc111.c
> @@ -9,6 +9,7 @@
>  
>  #include <linux/module.h>
>  #include <linux/netdevice.h>
> +#include <linux/ethtool.h>
>  #include <linux/mii.h>
>  #include <linux/usb.h>
>  #include <linux/crc32.h>
> @@ -18,6 +19,9 @@
>  
>  #include "aqc111.h"
>  
> +#define DRIVER_VERSION "1.0.0.0"

Hi Igor

This is been discussed a number of times before, and i expect it to be
discussed again. Driver versions of useless. Please drop it.

> +#define DRIVER_NAME "Aquantia AQtion USB to 5GbE"

It might be nice to include the module name in here, making it easy to
link the module to the driver.

> +
>  static int __aqc111_read_cmd(struct usbnet *dev, u8 cmd, u16 value,
>  			     u16 index, u16 size, void *data, int nopm)
>  {
> @@ -170,6 +174,24 @@ static int aq_mdio_write_cmd(struct usbnet *dev, u16 value, u16 index,
>  	return aqc111_write_cmd(dev, AQ_PHY_CMD, value, index, size, data);
>  }
>  

   Andrew