> On 2/17/2024 1:54 AM, Chaoyong He wrote: > > Add the support of UDP fragmentation offload feature. > > > > Signed-off-by: Chaoyong He <chaoyong...@corigine.com> > > Reviewed-by: Long Wu <long...@corigine.com> > > Reviewed-by: Peng Zhang <peng.zh...@corigine.com> > > <...> > > > diff --git a/drivers/net/nfp/nfp_net_common.c > > b/drivers/net/nfp/nfp_net_common.c > > index 72c9a41b00..99c319eb2d 100644 > > --- a/drivers/net/nfp/nfp_net_common.c > > +++ b/drivers/net/nfp/nfp_net_common.c > > @@ -312,7 +312,7 @@ nfp_net_log_device_information(const struct > nfp_net_hw *hw) > > hw->ver.major, hw->ver.minor, hw->max_mtu); > > > > PMD_INIT_LOG(INFO, "CAP: %#x", cap); > > - PMD_INIT_LOG(INFO, > "%s%s%s%s%s%s%s%s%s%s%s%s%s%s%s%s%s%s%s%s%s%s%s%s%s%s%s%s", > > + PMD_INIT_LOG(INFO, > > > +"%s%s%s%s%s%s%s%s%s%s%s%s%s%s%s%s%s%s%s%s%s%s%s%s%s%s%s%s > %s", > > > > This seems getting out of hand, I assume this is done like this (instead of > multiple > print lines) to prevent line break, if so what do you think add a new macro > that > doesn't append \n automatically and refactor this code (in a separate patch) ?
Yeah, that's a good point, thanks! I will add it into my to-do list, and send a patch at the suitable time.
