From mboxrd@z Thu Jan 1 00:00:00 1970 From: Thomas Monjalon Subject: Re: [PATCH v7 00/28] remove pci driver from vdevs Date: Mon, 02 Nov 2015 08:35:44 +0100 Message-ID: <1846921.aMtNxlC2Q7@xps13> References: <533710CFB86FA344BFBF2D6802E6028621B5A5E8@SHSMSX101.ccr.corp.intel.com> Mime-Version: 1.0 Content-Type: text/plain; charset="us-ascii" Content-Transfer-Encoding: 7Bit Cc: dev@dpdk.org To: "Iremonger, Bernard" Return-path: Received: from mail-wm0-f44.google.com (mail-wm0-f44.google.com [74.125.82.44]) by dpdk.org (Postfix) with ESMTP id 0BA3CFE5 for ; Mon, 2 Nov 2015 08:37:02 +0100 (CET) Received: by wmeg8 with SMTP id g8so52733571wme.1 for ; Sun, 01 Nov 2015 23:37:02 -0800 (PST) In-Reply-To: <533710CFB86FA344BFBF2D6802E6028621B5A5E8@SHSMSX101.ccr.corp.intel.com> List-Id: patches and discussions about DPDK List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , Errors-To: dev-bounces@dpdk.org Sender: "dev" 2015-11-02 05:28, Qiu, Michael: > Hi, Bernard > > Could we merge some patch together? I see lots of patches are simple and > doing the same thing but in different NIC, merge them almost have no > affect of review, what's more it will make reviewers more comfortable > with less and simple patches. > > Then we could have a clean patch set with almost 5 patches, not totally 28. > > librte_eal: add RTE_KDRV_NONE for vdevs > librte_ether: add fields from rte_pci_driver to rte_eth_dev_data > drivers: copy pci device info to eth_dev data > librte_ether: remove branches on pci_dev > drivers: remove pci device Yes I agree with Michael