On Mon, Aug 24, 2026 at 06:23:22PM +0200, David Marchand wrote:
> On Fri, 14 Aug 2026 at 17:39, Bruce Richardson
> <[email protected]> wrote:
> > diff --git a/drivers/net/intel/ixgbe/ixgbe_ethdev.c 
> > b/drivers/net/intel/ixgbe/ixgbe_ethdev.c
> > index 3d4f4fb7fc..8e44bb3a09 100644
> > --- a/drivers/net/intel/ixgbe/ixgbe_ethdev.c
> > +++ b/drivers/net/intel/ixgbe/ixgbe_ethdev.c
> > @@ -36,15 +36,15 @@
> >  #include <rte_os_shim.h>
> >
> >  #include "ixgbe_logs.h"
> > -#include "base/ixgbe_api.h"
> > -#include "base/ixgbe_vf.h"
> > -#include "base/ixgbe_common.h"
> > +#include "ixgbe_api.h"
> > +#include "ixgbe_vf.h"
> > +#include "ixgbe_common.h"
> >  #include "ixgbe_ethdev.h"
> >  #include "ixgbe_bypass.h"
> >  #include "ixgbe_rxtx.h"
> > -#include "base/ixgbe_type.h"
> > -#include "base/ixgbe_phy.h"
> > -#include "base/ixgbe_osdep.h"
> > +#include "ixgbe_type.h"
> > +#include "ixgbe_phy.h"
> > +#include "ixgbe_osdep.h"
> >  #include "ixgbe_regs.h"
> 
> While I agree with the base/ixgbe_osdep.h -> ixgbe_osdep.h conversion,
> I don't get why you *need* to change other base headers inclusions
> above.
> 
> The rest of the series looks good to me.
>
I don't need to, I just felt that the base specifiers were unnecessary.
I'll revert those changes, though, to keep all files consistent in this
driver.

/Bruce 

Reply via email to