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 > > >
