On Thu, Feb 13, 2020 at 1:07 PM Hemant Agrawal <hemant.agra...@nxp.com> wrote: > > I could not find a dependency in the makefiles for this. > > But meson has this dependency. > > > > drivers/event/dpaa/meson.build > > deps += ['pmd_dpaa', 'pmd_dpaa_sec'] > > > > > [Hemant] Makefile do have dependency set, but at a different place. I have > done a small change. Please check if you can :
I must have missed it then. Can you describe how this dependency is expressed in the makefiles? > > diff --git a/drivers/event/dpaa/Makefile b/drivers/event/dpaa/Makefile > index 2f53efdf9e..093d3a38cf 100644 > --- a/drivers/event/dpaa/Makefile > +++ b/drivers/event/dpaa/Makefile > @@ -21,8 +21,6 @@ CFLAGS += -I$(RTE_SDK)/drivers/bus/dpaa/include/ > CFLAGS += -I$(RTE_SDK)/drivers/mempool/dpaa > CFLAGS += -I$(RTE_SDK)/drivers/common/dpaax > CFLAGS += -I$(RTE_SDK)/lib/librte_eal/common/include > - > -LDLIBS += -lrte_pmd_dpaa_sec > CFLAGS += -I$(RTE_SDK)/drivers/crypto/dpaa_sec > > EXPORT_MAP := rte_pmd_dpaa_event_version.map > @@ -34,6 +32,7 @@ LDLIBS += -lrte_bus_dpaa > LDLIBS += -lrte_mempool_dpaa > LDLIBS += -lrte_eal -lrte_mbuf -lrte_mempool -lrte_ring > LDLIBS += -lrte_ethdev -lrte_net -lrte_kvargs > +LDLIBS += -lrte_pmd_dpaa_sec > LDLIBS += -lrte_eventdev -lrte_pmd_dpaa -lrte_bus_vdev > LDLIBS += -lrte_common_dpaax I don't see how this would affect the building order between drivers/event/dpaa and drivers/crypto/dpaa_sec. -- David Marchand