[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