On Wed, Sep 27, 2017 at 09:40:52AM +0000, De Lara Guarch, Pablo wrote: > > > > -----Original Message----- > > From: dev [mailto:dev-boun...@dpdk.org] On Behalf Of Tomasz Duszynski > > Sent: Tuesday, September 26, 2017 10:40 AM > > To: dev@dpdk.org > > Cc: m...@semihalf.com; d...@marvell.com; nsams...@marvell.com; > > Tomasz Duszynski <t...@semihalf.com> > > 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