On Mon, Aug 29, 2022 at 9:18 AM Ruifeng Wang <ruifeng.w...@arm.com> wrote: > > diff --git a/app/test/test_vdev.c b/app/test/test_vdev.c index > > 5eeff3106d..3cc3abe70b > > 100644 > > --- a/app/test/test_vdev.c > > +++ b/app/test/test_vdev.c > > @@ -9,7 +9,7 @@ > > #include <rte_common.h> > > #include <rte_kvargs.h> > > #include <rte_bus.h> > > -#include <rte_bus_vdev.h> > > +#include <bus_vdev_driver.h> > This line of change is unnecessary? Application uses rte header.
This unit test looks into rte_bus objects (you'll notice a change later in the series for including bus_driver.h) but, here, it only uses "standard" API for the rest. I'll fix this include, in next revision. > The rest looks good to me. > Reviewed-by: Ruifeng Wang <ruifeng.w...@arm.com> Thanks Ruifeng. -- David Marchand