23/02/2018 07:25, Hemant Agrawal:
> Hi Thomas,
> On 2/23/2018 4:23 AM, Thomas Monjalon wrote:
> > The bus and mempool dependencies should be declared after the PMD
> > libraries needing them.
> > Moreover there is no need to disable the PMDs at the Makefile level,
> > in case the dependencies are not met.
> > Such dependencies should be handled at configuration time.
> > The other side effect of this clean-up is to take into account
> > the mempool option CONFIG_RTE_LIBRTE_DPAA2_MEMPOOL.
> > Signed-off-by: Thomas Monjalon <tho...@monjalon.net>
> > ---
> > mk/rte.app.mk | 20 ++++----------------
> > 1 file changed, 4 insertions(+), 16 deletions(-)
> This patch looks good, however I think we (NXP) need to also set some
> dependency check in makefile to avoid dpaaX PMD compilations, if bus is
> not available. I am working on a patch for the same.
Why do you think such check is necessary?
If the PMD is enable but not the bus, it is a configuration error.
A good configuration system would resolve it automatically,
but currently we do the configuration manually, so the user must fix
its configuration file.