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