From mboxrd@z Thu Jan 1 00:00:00 1970 From: Thomas Monjalon Subject: Re: [PATCH] pmd: change initialization to indicate pci drivers Date: Fri, 12 Jun 2015 11:13:52 +0200 Message-ID: <10636183.XYdRXAllQb@xps13> References: <1432914471-20061-1-git-send-email-stephen@networkplumber.org> Mime-Version: 1.0 Content-Type: text/plain; charset="us-ascii" Content-Transfer-Encoding: 7Bit Cc: dev@dpdk.org To: Stephen Hemminger Return-path: Received: from mail-wg0-f47.google.com (mail-wg0-f47.google.com [74.125.82.47]) by dpdk.org (Postfix) with ESMTP id 27AE9E72 for ; Fri, 12 Jun 2015 11:14:51 +0200 (CEST) Received: by wgme6 with SMTP id e6so20108973wgm.2 for ; Fri, 12 Jun 2015 02:14:51 -0700 (PDT) In-Reply-To: <1432914471-20061-1-git-send-email-stephen@networkplumber.org> 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-05-29 08:47, Stephen Hemminger: > Upcoming drivers will need to be able to support other bus types. > This is a transparent change to how struct eth_driver is initialized. > It has not function or ABI layout impact, but makes adding a later > bus type (Xen, Hyper-V, ...) much easier. > > Signed-off-by: Stpehen Hemminger There is a (fixed) typo in this line. Please use --signoff. Acked-by: Thomas Monjalon Applied, thanks Next step would be to avoid this strange assumption: struct eth_driver { struct rte_pci_driver pci_drv; /**< The PMD is also a PCI driver. */