Return-Path: Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S1751867AbdG1Lw7 convert rfc822-to-8bit (ORCPT ); Fri, 28 Jul 2017 07:52:59 -0400 Received: from lhrrgout.huawei.com ([194.213.3.17]:32400 "EHLO lhrrgout.huawei.com" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S1751786AbdG1Lw5 (ORCPT ); Fri, 28 Jul 2017 07:52:57 -0400 From: Salil Mehta To: Leon Romanovsky CC: "davem@davemloft.net" , "Zhuangyuzeng (Yisen)" , huangdaode , "lipeng (Y)" , "mehta.salil.lnk@gmail.com" , "netdev@vger.kernel.org" , "linux-kernel@vger.kernel.org" , "linux-rdma@vger.kernel.org" , Linuxarm Subject: RE: [PATCH V4 net-next 2/8] net: hns3: Add support of the HNAE3 framework Thread-Topic: [PATCH V4 net-next 2/8] net: hns3: Add support of the HNAE3 framework Thread-Index: AQHTAzd5ptG4YuKSvUW4+mhxmQksjaJhQ40AgAaX/fCAALQYgIAAl5QA Date: Fri, 28 Jul 2017 11:52:38 +0000 Message-ID: References: <20170722220942.78852-1-salil.mehta@huawei.com> <20170722220942.78852-3-salil.mehta@huawei.com> <20170723131530.GJ3259@mtr-leonro.local> <20170728044137.GF13672@mtr-leonro.local> In-Reply-To: <20170728044137.GF13672@mtr-leonro.local> Accept-Language: en-US Content-Language: en-US X-MS-Has-Attach: X-MS-TNEF-Correlator: x-originating-ip: [10.203.181.161] Content-Type: text/plain; charset="us-ascii" Content-Transfer-Encoding: 8BIT MIME-Version: 1.0 X-CFilter-Loop: Reflected X-Mirapoint-Virus-RAPID-Raw: score=unknown(0), refid=str=0001.0A020206.597B2594.0024,ss=1,re=0.000,recu=0.000,reip=0.000,cl=1,cld=1,fgs=0, ip=169.254.1.170, so=2013-06-18 04:22:30, dmn=2013-03-21 17:37:32 X-Mirapoint-Loop-Id: dfc7fcbda838253fb8ad4205b1b956a4 Sender: linux-kernel-owner@vger.kernel.org List-ID: X-Mailing-List: linux-kernel@vger.kernel.org Content-Length: 4746 Lines: 128 Hi Leon, > -----Original Message----- > From: linux-rdma-owner@vger.kernel.org [mailto:linux-rdma- > owner@vger.kernel.org] On Behalf Of Leon Romanovsky > Sent: Friday, July 28, 2017 5:42 AM > To: Salil Mehta > Cc: davem@davemloft.net; Zhuangyuzeng (Yisen); huangdaode; lipeng (Y); > mehta.salil.lnk@gmail.com; netdev@vger.kernel.org; linux- > kernel@vger.kernel.org; linux-rdma@vger.kernel.org; Linuxarm > Subject: Re: [PATCH V4 net-next 2/8] net: hns3: Add support of the > HNAE3 framework > > On Thu, Jul 27, 2017 at 11:44:32PM +0000, Salil Mehta wrote: > > Hi Leon > > > > > -----Original Message----- > > > From: linux-rdma-owner@vger.kernel.org [mailto:linux-rdma- > > > owner@vger.kernel.org] On Behalf Of Leon Romanovsky > > > Sent: Sunday, July 23, 2017 2:16 PM > > > To: Salil Mehta > > > Cc: davem@davemloft.net; Zhuangyuzeng (Yisen); huangdaode; lipeng > (Y); > > > mehta.salil.lnk@gmail.com; netdev@vger.kernel.org; linux- > > > kernel@vger.kernel.org; linux-rdma@vger.kernel.org; Linuxarm > > > Subject: Re: [PATCH V4 net-next 2/8] net: hns3: Add support of the > > > HNAE3 framework > > > > > > On Sat, Jul 22, 2017 at 11:09:36PM +0100, Salil Mehta wrote: > > > > This patch adds the support of the HNAE3 (Hisilicon Network > > > > Acceleration Engine 3) framework support to the HNS3 driver. > > > > > > > > Framework facilitates clients like ENET(HNS3 Ethernet Driver), > RoCE > > > > and user-space Ethernet drivers (like ODP etc.) to register with > > > HNAE3 > > > > devices and their associated operations. > > > > > > > > Signed-off-by: Daode Huang > > > > Signed-off-by: lipeng > > > > Signed-off-by: Salil Mehta > > > > Signed-off-by: Yisen Zhuang > > > > --- > > > > Patch V4: Addressed following comments > > > > 1. Andrew Lunn: > > > > https://lkml.org/lkml/2017/6/17/233 > > > > https://lkml.org/lkml/2017/6/18/105 > > > > 2. Bo Yu: > > > > https://lkml.org/lkml/2017/6/18/112 > > > > 3. Stephen Hamminger: > > > > https://lkml.org/lkml/2017/6/19/778 > > > > Patch V3: Addressed below comments > > > > 1. Andrew Lunn: > > > > https://lkml.org/lkml/2017/6/13/1025 > > > > Patch V2: No change > > > > Patch V1: Initial Submit > > > > --- > > > > drivers/net/ethernet/hisilicon/hns3/hnae3.c | 319 > > > ++++++++++++++++++++ > > > > drivers/net/ethernet/hisilicon/hns3/hnae3.h | 449 > > > ++++++++++++++++++++++++++++ > > > > 2 files changed, 768 insertions(+) > > > > create mode 100644 drivers/net/ethernet/hisilicon/hns3/hnae3.c > > > > create mode 100644 drivers/net/ethernet/hisilicon/hns3/hnae3.h > > > > > > > > diff --git a/drivers/net/ethernet/hisilicon/hns3/hnae3.c > > > b/drivers/net/ethernet/hisilicon/hns3/hnae3.c > > > > new file mode 100644 > > > > index 000000000000..7a11aaff0a23 [...] > > > > + > > > > +#include > > > > +#include > > > > +#include > > > > +#include > > > > +#include > > > > +#include > > > > +#include > > > > + > > > > +#define HNAE_DRIVER_VERSION "1.0" > > > > > > Please no driver versions. > > We need this in ethtool. Most of the driver are using it. > > So please, stop doing copy/paste and take a look how it was implemented > in nfp. > > Related discussion about useless of your driver version. > https://lists.linuxfoundation.org/pipermail/ksummit-discuss/2017- > June/004441.html > https://lists.linuxfoundation.org/pipermail/ksummit-discuss/2017- > June/004428.html Sure, thanks for this. I will have a look > > > > > > > > > > +#define HNAE_DRIVER_NAME "hns3" > > > > +#define HNAE_COPYRIGHT "Copyright(c) 2017 Huawei Corporation." > > > > +#define HNAE_DRIVER_STRING "Hisilicon Network Subsystem Driver" > > > > +#define HNAE_DEFAULT_DEVICE_DESCR "Hisilicon Network Subsystem" > > > > > > You are not subsystem yet. > > Hisilicon Network System is the network related hardware within > > Hip08 SoC of Hisilicon. This does not means HNS is Linux network > > subsystem. > > I understand it, so remove word "subsystem" and use more appropriate > "core", "library", e.t.c I understand this might look bit confusing, but if we change the name of the driver now then we might have to change the name of the folders and earlier driver names as well. Name of the earlier driver would be very difficult to change at this stage since there might be people using it. Do you think if we rename to below it will bring more clarity? +#define HNAE_DRIVER_STRING "Hisilicon Network Subsystem Ethernet Driver" OR +#define HNAE_DRIVER_STRING "HNS3 Ethernet Driver" Please suggest? Thanks! Best regards Salil > > Thanks