[PATCH] net/nfp: fix MTU configuration order
Niklas Söderlund
niklas.soderlund at corigine.com
Wed Mar 8 09:19:27 CET 2023
Hi Stephen,
Thanks for your feedback.
On 2023-03-07 18:44:09 -0800, Stephen Hemminger wrote:
> On Wed, 8 Mar 2023 10:33:18 +0800
> Chaoyong He <chaoyong.he at corigine.com> wrote:
>
> > diff --git a/drivers/net/nfp/nfp_common.c b/drivers/net/nfp/nfp_common.c
> > index 5922bfea8e..5d92b476e2 100644
> > --- a/drivers/net/nfp/nfp_common.c
> > +++ b/drivers/net/nfp/nfp_common.c
> > @@ -1126,9 +1126,9 @@ nfp_net_dev_mtu_set(struct rte_eth_dev *dev, uint16_t mtu)
> > return -EBUSY;
> > }
> >
> > - /* MTU larger then current mbufsize not supported */
> > + /* MTU larger than current mbufsize not supported */
> > if (mtu > hw->flbufsz) {
> > - PMD_DRV_LOG(ERR, "MTU (%u) larger then current mbufsize (%u) not supported",
> > + PMD_DRV_LOG(ERR, "MTU (%u) larger than current mbufsize (%u) not supported",
> > mtu, hw->flbufsz);
> > return -ERANGE;
> > }
>
> Patch looks good but this looks like unrelated whitespace change.
It's a tad unrelated, but not a whitespace change. It fixes a spelling
mistake related to setting the MTU,
s/then/than/g
--
Kind Regards,
Niklas Söderlund
More information about the dev
mailing list