Re: [dpdk-dev] [PATCH v2] mempool/dpaa2: add DPAA2 hardware offloaded mempool

[Thomas Monjalon <thomas.monjalon@6wind.com>]

2017-04-11 11:28, Hemant Agrawal:
> On 4/11/2017 1:28 AM, Olivier MATZ wrote:
> > Hemant Agrawal <hemant.agrawal@nxp.com> wrote:
> >> --- a/drivers/bus/Makefile
> >> +++ b/drivers/bus/Makefile
> >> @@ -33,6 +33,10 @@ include $(RTE_SDK)/mk/rte.vars.mk
> >>
> >>  core-libs := librte_eal librte_mbuf librte_mempool librte_ring librte_ether
> >>
> >> +ifeq ($(CONFIG_RTE_LIBRTE_DPAA2_MEMPOOL),y)
> >> +CONFIG_RTE_LIBRTE_FSLMC_BUS = $(CONFIG_RTE_LIBRTE_DPAA2_MEMPOOL)
> >> +endif
> >> +
> >>  DIRS-$(CONFIG_RTE_LIBRTE_FSLMC_BUS) += fslmc
> >>  DEPDIRS-fslmc = ${core-libs}
> >>
> >
> > What's the purpose of this?
> > Not sure we are allowed to modify the configs in the Makefiles.
> 
> DPAA2_MEMPOOL will not work without the DPAA2 mempool hw instance 
> detected on FSLMC_BUS.
> So, it is required that if you are enabling DPAA2_MEMPOOL, FSLMC_BUS is 
> to be enabled.
> 
> Currently the config structure do not provide such dependency definitions.
> 
> This was done based on the suggestions on the initial patches from 
> Ferruh and Jerin.

Please do not do that.
We do not change the configuration in the back of the user.
This kind of dependency should be managed in the configuration step
which do not exist yet.

You can use $(error) to stop the compilation instead.