[PATCH v4 17/27] bus/vdev: make driver-only headers private
David Marchand
david.marchand at redhat.com
Mon Aug 29 10:12:18 CEST 2022
On Mon, Aug 29, 2022 at 9:18 AM Ruifeng Wang <Ruifeng.Wang at 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.wang at arm.com>
Thanks Ruifeng.
--
David Marchand
More information about the dev
mailing list