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...@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

Reply via email to