On Wed, Sep 27, 2017 at 09:40:52AM +0000, De Lara Guarch, Pablo wrote: > > > > -----Original Message----- > > From: dev [mailto:[email protected]] On Behalf Of Tomasz Duszynski > > Sent: Tuesday, September 26, 2017 10:40 AM > > To: [email protected] > > Cc: [email protected]; [email protected]; [email protected]; > > Tomasz Duszynski <[email protected]> > > 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 PMDs > 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ński
