From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: Received: from mga05.intel.com (mga05.intel.com [192.55.52.43]) by dpdk.org (Postfix) with ESMTP id 517BD2C4F for ; Thu, 15 Dec 2016 15:41:15 +0100 (CET) Received: from fmsmga003.fm.intel.com ([10.253.24.29]) by fmsmga105.fm.intel.com with ESMTP; 15 Dec 2016 06:41:14 -0800 X-ExtLoop1: 1 X-IronPort-AV: E=Sophos;i="5.33,352,1477983600"; d="scan'208";a="798337780" Received: from fyigit-mobl1.ger.corp.intel.com (HELO [10.237.220.29]) ([10.237.220.29]) by FMSMGA003.fm.intel.com with ESMTP; 15 Dec 2016 06:41:13 -0800 To: Hemant Agrawal , "dev@dpdk.org" References: <1480875447-23680-1-git-send-email-hemant.agrawal@nxp.com> <1480875447-23680-10-git-send-email-hemant.agrawal@nxp.com> Cc: "thomas.monjalon@6wind.com" , "Richardson, Bruce" , "shreyansh.jain@nxp.com" From: Ferruh Yigit Message-ID: <4ced5dc6-c2d7-a4d9-b7a1-29476efd9791@intel.com> Date: Thu, 15 Dec 2016 14:41:12 +0000 User-Agent: Mozilla/5.0 (Windows NT 10.0; WOW64; rv:45.0) Gecko/20100101 Thunderbird/45.5.1 MIME-Version: 1.0 In-Reply-To: Content-Type: text/plain; charset=windows-1252 Content-Transfer-Encoding: 7bit Subject: Re: [dpdk-dev] [PATCH 09/32] lib/ether: add rte_device in rte_eth_dev X-BeenThere: dev@dpdk.org X-Mailman-Version: 2.1.15 Precedence: list List-Id: DPDK patches and discussions List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , X-List-Received-Date: Thu, 15 Dec 2016 14:41:16 -0000 On 12/7/2016 6:41 AM, Hemant Agrawal wrote: > On 12/7/2016 1:18 AM, Ferruh Yigit wrote: >> On 12/4/2016 6:17 PM, Hemant Agrawal wrote: >>> Signed-off-by: Hemant Agrawal >>> --- >>> lib/librte_ether/rte_ethdev.h | 1 + >>> 1 file changed, 1 insertion(+) >>> >>> diff --git a/lib/librte_ether/rte_ethdev.h b/lib/librte_ether/rte_ethdev.h >>> index 3c45a1f..6f5673f 100644 >>> --- a/lib/librte_ether/rte_ethdev.h >>> +++ b/lib/librte_ether/rte_ethdev.h >>> @@ -1626,6 +1626,7 @@ struct rte_eth_dev { >>> eth_rx_burst_t rx_pkt_burst; /**< Pointer to PMD receive function. */ >>> eth_tx_burst_t tx_pkt_burst; /**< Pointer to PMD transmit function. */ >>> struct rte_eth_dev_data *data; /**< Pointer to device data */ >>> + struct rte_device *device; >> >> I believe this change should not be part of a PMD patchset. This change >> is more generic than the PMD. >> >> Won't Shreyansh's patch already do this? > > I agree that this patch is not a fit for this PMD patchset, Shreyansh's > patch is not yet doing it. He will be taking care of it next. > > So till Shreyansh provide the support, we need it. If you need it, what do you think sending this as a separate patch? And when accepted, your driver can use it? > >> >>> const struct eth_driver *driver;/**< Driver for this device */ >>> const struct eth_dev_ops *dev_ops; /**< Functions exported by PMD */ >>> struct rte_pci_device *pci_dev; /**< PCI info. supplied by probing */ >>> >> >> > >