From mboxrd@z Thu Jan  1 00:00:00 1970
From: Shreyansh Jain <shreyansh.jain@nxp.com>
Subject: Re: [PATCH v2] ethdev: remove driver name from device
	private data
Date: Wed, 7 Jun 2017 10:22:42 +0530
Message-ID: <5f9f3914-ab05-11a7-2fe5-1eb143c47f90@nxp.com>
References: <20170526111502.92063-1-ferruh.yigit@intel.com>
 <20170606151008.62680-1-ferruh.yigit@intel.com>
Mime-Version: 1.0
Content-Type: text/plain; charset="utf-8"; format=flowed
Content-Transfer-Encoding: 7bit
Cc: Stephen Hurd <stephen.hurd@broadcom.com>, Ajit Khaparde
 <ajit.khaparde@broadcom.com>, Declan Doherty <declan.doherty@intel.com>,
 Rahul Lakkireddy <rahul.lakkireddy@chelsio.com>, Hemant Agrawal
 <hemant.agrawal@nxp.com>, Helin Zhang <helin.zhang@intel.com>, Jingjing Wu
 <jingjing.wu@intel.com>, Wenzhuo Lu <wenzhuo.lu@intel.com>, Konstantin
 Ananyev <konstantin.ananyev@intel.com>, Bruce Richardson
 <bruce.richardson@intel.com>, Pascal Mazon <pascal.mazon@6wind.com>,
 Shrikrishna Khare <skhare@vmware.com>, Jianfeng Tan <jianfeng.tan@intel.com>,
 Thomas Monjalon <thomas@monjalon.net>, <dev@dpdk.org>, Gaetan Rivet
 <gaetan.rivet@6wind.com>, Jan Blunck <jblunck@infradead.org>
To: Ferruh Yigit <ferruh.yigit@intel.com>
Return-path: <dev-bounces@dpdk.org>
Received: from NAM03-BY2-obe.outbound.protection.outlook.com
 (mail-by2nam03on0078.outbound.protection.outlook.com [104.47.42.78])
 by dpdk.org (Postfix) with ESMTP id 718C42B91
 for <dev@dpdk.org>; Wed,  7 Jun 2017 06:44:41 +0200 (CEST)
In-Reply-To: <20170606151008.62680-1-ferruh.yigit@intel.com>
Content-Language: en-US
List-Id: DPDK patches and discussions <dev.dpdk.org>
List-Unsubscribe: <http://dpdk.org/ml/options/dev>,
 <mailto:dev-request@dpdk.org?subject=unsubscribe>
List-Archive: <http://dpdk.org/ml/archives/dev/>
List-Post: <mailto:dev@dpdk.org>
List-Help: <mailto:dev-request@dpdk.org?subject=help>
List-Subscribe: <http://dpdk.org/ml/listinfo/dev>,
 <mailto:dev-request@dpdk.org?subject=subscribe>
Errors-To: dev-bounces@dpdk.org
Sender: "dev" <dev-bounces@dpdk.org>

On Tuesday 06 June 2017 08:40 PM, Ferruh Yigit wrote:
> rte_driver->name has the driver name and all physical and virtual
> devices has access to it.
> 
> Previously it was not possible for virtual ethernet devices to access
> rte_driver->name field (because eth_dev used to keep only pci_dev),
> and it was required to save driver name in the device private struct.
> 
> After re-works on bus and vdev, it is possible for all bus types to
> access rte_driver.
> 
> It is able to remove the driver name from ethdev device private data and
> use eth_dev->device->driver->name.
> 
> Signed-off-by: Ferruh Yigit <ferruh.yigit@intel.com>
> ---
> Cc: Gaetan Rivet <gaetan.rivet@6wind.com>
> Cc: Jan Blunck <jblunck@infradead.org>
> 
> v2:
> * rebase on latest next-net
> ---
>   drivers/net/bnxt/bnxt_ethdev.c         | 2 +-
>   drivers/net/bonding/rte_eth_bond_api.c | 4 ++--
>   drivers/net/cxgbe/sge.c                | 6 +++---
>   drivers/net/dpaa2/dpaa2_ethdev.c       | 1 -
>   drivers/net/i40e/i40e_ethdev.c         | 3 +--
>   drivers/net/i40e/i40e_fdir.c           | 2 +-
>   drivers/net/ixgbe/ixgbe_ethdev.c       | 2 +-
>   drivers/net/ring/rte_eth_ring.c        | 1 -
>   drivers/net/tap/rte_eth_tap.c          | 1 -
>   drivers/net/vmxnet3/vmxnet3_ethdev.c   | 2 +-
>   drivers/net/xenvirt/rte_eth_xenvirt.c  | 1 -
>   lib/librte_ether/rte_ethdev.c          | 8 ++++----
>   lib/librte_ether/rte_ethdev.h          | 1 -
>   lib/librte_ether/rte_ethdev_pci.h      | 1 -
>   lib/librte_ether/rte_ethdev_vdev.h     | 1 -
>   15 files changed, 14 insertions(+), 22 deletions(-)
> 

Apologies for delay in responding. I am OK with respect to the dpaa2 
change. Otherwise as well:

Acked-by: Shreyansh Jain <shreyansh.jain@nxp.com>