[dpdk-dev] [PATCH v2 08/10] config: add dpaaX build support in common linuxapp

[Hemant Agrawal]

 Hi Thomas,
> 
> As you know the config options should be declared in common_base and
> overwritten in common_linuxapp.
> 
> That's why there is no need to add comments in common_linuxapp.
> And some of the options should be only in common_base, like
> 	CONFIG_RTE_LIBRTE_DPAA_HWDEBUG=n
> 
> Please try to move all the options in common_base with their comments, and
> only overwrite what is needed for Linux compilation.
> 
[Hemant] Thanks for the review.  I will fix these in v3.

Regards,
Hemant 

> Thanks
> 
> 
> 01/03/2018 08:33, Hemant Agrawal:
> > --- a/config/common_linuxapp
> > +++ b/config/common_linuxapp
> > @@ -23,3 +23,40 @@ CONFIG_RTE_LIBRTE_NFP_PMD=y
> > CONFIG_RTE_LIBRTE_POWER=y  CONFIG_RTE_VIRTIO_USER=y
> > CONFIG_RTE_PROC_INFO=y
> > +#
> > +# ARMv8 Specific driver compilation flags #
> > +
> > +# Compile NXP DPAA Bus
> > +CONFIG_RTE_LIBRTE_DPAA_BUS=y
> > +CONFIG_RTE_LIBRTE_DPAA_HWDEBUG=n
> > +
> > +# Compile NXP DPAA Mempool
> > +CONFIG_RTE_LIBRTE_DPAA_MEMPOOL=y
> > +
> > +# Compile bust-oriented NXP DPAA PMD
> > +CONFIG_RTE_LIBRTE_DPAA_PMD=y
> > +
> > +# Compile schedule-oriented NXP DPAA Event Dev PMD
> > +CONFIG_RTE_LIBRTE_PMD_DPAA_EVENTDEV=y
> > +
> > +# Compile NXP DPAA caam - crypto driver
> > +CONFIG_RTE_LIBRTE_PMD_DPAA_SEC=y
> > +CONFIG_RTE_LIBRTE_DPAA_MAX_CRYPTODEV=4
> > +CONFIG_RTE_DPAA_SEC_PMD_MAX_NB_SESSIONS=2048
> > +
> > +# Compile NXP DPAA2 FSL-MC Bus
> > +CONFIG_RTE_LIBRTE_FSLMC_BUS=y
> > +
> > +# Compile NXP DPAA2 Mempool
> > +CONFIG_RTE_LIBRTE_DPAA2_MEMPOOL=y
> > +
> > +# Compile burst-oriented NXP DPAA2 PMD driver
> > +CONFIG_RTE_LIBRTE_DPAA2_PMD=y
> > +
> > +# Compile schedule-oriented NXP DPAA2 EVENTDEV driver
> > +CONFIG_RTE_LIBRTE_PMD_DPAA2_EVENTDEV=y
> > +
> > +# Compile NXP DPAA2 crypto sec driver for CAAM HW
> > +CONFIG_RTE_LIBRTE_PMD_DPAA2_SEC=y
> > +CONFIG_RTE_DPAA2_SEC_PMD_MAX_NB_SESSIONS=2048
> 
> 
>