From mboxrd@z Thu Jan 1 00:00:00 1970 From: Tomasz Duszynski Subject: Re: [PATCH 0/8] add net/crypto mrvl pmd drivers Date: Wed, 27 Sep 2017 12:59:26 +0200 Message-ID: <20170927105926.GA9473@tdu> References: <1506418805-12117-1-git-send-email-tdu@semihalf.com> Mime-Version: 1.0 Content-Type: text/plain; charset=utf-8 Content-Transfer-Encoding: quoted-printable Cc: Tomasz Duszynski , "dev@dpdk.org" , "mw@semihalf.com" , "dima@marvell.com" , "nsamsono@marvell.com" To: "De Lara Guarch, Pablo" Return-path: Received: from mail-wr0-f178.google.com (mail-wr0-f178.google.com [209.85.128.178]) by dpdk.org (Postfix) with ESMTP id CE1196904 for ; Wed, 27 Sep 2017 12:59:28 +0200 (CEST) Received: by mail-wr0-f178.google.com with SMTP id 54so1099072wrz.10 for ; Wed, 27 Sep 2017 03:59:28 -0700 (PDT) Content-Disposition: inline In-Reply-To: List-Id: DPDK patches and discussions List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , Errors-To: dev-bounces@dpdk.org Sender: "dev" On Wed, Sep 27, 2017 at 09:40:52AM +0000, De Lara Guarch, Pablo wrote: > > > > -----Original Message----- > > From: dev [mailto:dev-bounces@dpdk.org] On Behalf Of Tomasz Duszynski > > Sent: Tuesday, September 26, 2017 10:40 AM > > To: dev@dpdk.org > > Cc: mw@semihalf.com; dima@marvell.com; nsamsono@marvell.com; > > Tomasz Duszynski > > Subject: [dpdk-dev] [PATCH 0/8] add net/crypto mrvl pmd drivers > > > > Hello, > > > > This patch series introduces net/crypto drivers for Marvell Armada 7k/8k > > SoCs along with documentation and crypto driver pmd tests. > > > > Below you can find the list of features which net/crypto pmds support. > > > > Could you split this patchset into two? I think the network and crypto PM= Ds should be separated, > as they are targetting different subtrees. > > Thanks, > Pablo If that's a hard requirement I am fine with splitting the patch series. Drivers were put into the same patchset because they have common CONFIG dependency. E.g. applying just crypto pmd patches and changing CONFIG_RTE_LIBRTE_PMD_MRVL_CRYPTO to 'y' would break compilation as it depends on CONFIG introduced by net pmd patches. -- - Tomasz Duszy=C5=84ski