On Fri, Aug 24, 2018 at 01:32:47PM +0100, Ferruh Yigit wrote: > On 8/24/2018 1:16 PM, Tomasz Duszynski wrote: > > From: Liron Himi <lir...@marvell.com> > > > > Add MVEP (Marvell Embedded Processors) to drivers/common which > > will keep code reused by current and future MRVL PMDs. > > Right now we have only common DMA memory initialization routines there. > > > > Signed-off-by: Liron Himi <lir...@marvell.com> > > Signed-off-by: Tomasz Duszynski <t...@semihalf.com> > > Reviewed-by: Natalie Samsonov <nsams...@marvell.com> > > --- > > config/common_base | 5 +++ > > devtools/test-build.sh | 2 ++ > > drivers/common/Makefile | 4 +++ > > drivers/common/meson.build | 2 +- > > drivers/common/mvep/Makefile | 38 +++++++++++++++++++++ > > drivers/common/mvep/meson.build | 19 +++++++++++ > > drivers/common/mvep/mvep_common.c | 45 > > +++++++++++++++++++++++++ > > drivers/common/mvep/rte_common_mvep_version.map | 6 ++++ > > drivers/common/mvep/rte_mvep_common.h | 20 +++++++++++ > > mk/rte.app.mk | 4 +++ > > 10 files changed, 144 insertions(+), 1 deletion(-) > > create mode 100644 drivers/common/mvep/Makefile > > create mode 100644 drivers/common/mvep/meson.build > > create mode 100644 drivers/common/mvep/mvep_common.c > > create mode 100644 drivers/common/mvep/rte_common_mvep_version.map > > create mode 100644 drivers/common/mvep/rte_mvep_common.h > > > > diff --git a/config/common_base b/config/common_base > > index 4bcbaf9..978592d 100644 > > --- a/config/common_base > > +++ b/config/common_base > > @@ -395,6 +395,11 @@ CONFIG_RTE_LIBRTE_BOND_DEBUG_ALB_L1=n > > CONFIG_RTE_LIBRTE_PMD_FAILSAFE=y > > > > # > > +# Compile Marvell Embedded Processors Common > > +# > > +CONFIG_RTE_LIBRTE_MVEP_COMMON=n > > Do you need a new config option for common, why not enable it if one of the > consumers of common code enabled?
Good point. Will rework that in v2. -- - Tomasz DuszyĆski