> -----Original Message----- > From: [email protected] <[email protected]> > Sent: 2023年12月7日 7:16 > To: David S. Miller <[email protected]>; Eric Dumazet > <[email protected]>; Jakub Kicinski <[email protected]>; Paolo Abeni > <[email protected]>; Shay Agroskin <[email protected]>; Arthur > Kiyanovski <[email protected]>; David Arinzon > <[email protected]>; Noam Dagan <[email protected]>; Saeed > Bishara <[email protected]>; Rasesh Mody <[email protected]>; > Sudarsana Kalluru <[email protected]>; [email protected]; > Dimitris Michailidis <[email protected]>; Yisen Zhuang > <[email protected]>; Salil Mehta <[email protected]>; Jesse > Brandeburg <[email protected]>; Tony Nguyen > <[email protected]>; Louis Peens <[email protected]>; > Shannon Nelson <[email protected]>; Brett Creeley > <[email protected]>; [email protected]; K. Y. Srinivasan > <[email protected]>; Haiyang Zhang <[email protected]>; Wei Liu > <[email protected]>; Dexuan Cui <[email protected]>; Ronak Doshi > <[email protected]>; VMware PV-Drivers Reviewers > <[email protected]>; Andy Whitcroft <[email protected]>; Joe > Perches <[email protected]>; Dwaipayan Ray <[email protected]>; > Lukas Bulwahn <[email protected]>; Hauke Mehrtens > <[email protected]>; Andrew Lunn <[email protected]>; Florian Fainelli > <[email protected]>; Vladimir Oltean <[email protected]>; Arınç ÜNAL > <[email protected]>; Daniel Golle <[email protected]>; Landen > Chao <[email protected]>; DENG Qingfang <[email protected]>; > Sean Wang <[email protected]>; Matthias Brugger > <[email protected]>; AngeloGioacchino Del Regno > <[email protected]>; Linus Walleij > <[email protected]>; Alvin Šipraga <[email protected]>; Wei Fang > <[email protected]>; Shenwei Wang <[email protected]>; Clark > Wang <[email protected]>; dl-linux-imx <[email protected]>; Lars > Povlsen <[email protected]>; Steen Hegelund > <[email protected]>; Daniel Machon > <[email protected]>; [email protected]; Jiawen > Wu <[email protected]>; Mengyuan Lou > <[email protected]>; Heiner Kallweit <[email protected]>; > Russell King <[email protected]>; Alexei Starovoitov <[email protected]>; > Daniel Borkmann <[email protected]>; Jesper Dangaard Brouer > <[email protected]>; John Fastabend <[email protected]> > Cc: [email protected]; [email protected]; Nick Desaulniers > <[email protected]>; Nathan Chancellor <[email protected]>; Kees > Cook <[email protected]>; [email protected]; > [email protected]; [email protected]; > [email protected]; [email protected]; > [email protected]; Justin Stitt <[email protected]> > Subject: [PATCH net-next v5 3/3] net: Convert some ethtool_sprintf() to > ethtool_puts() > > This patch converts some basic cases of ethtool_sprintf() to ethtool_puts(). > > The conversions are used in cases where ethtool_sprintf() was being used > with just two arguments: > | ethtool_sprintf(&data, buffer[i].name); > or when it's used with format string: "%s" > | ethtool_sprintf(&data, "%s", buffer[i].name); > which both now become: > | ethtool_puts(&data, buffer[i].name); > > Signed-off-by: Justin Stitt <[email protected]> > --- > drivers/net/ethernet/freescale/fec_main.c | 4 +- > diff --git a/drivers/net/ethernet/freescale/fec_main.c > b/drivers/net/ethernet/freescale/fec_main.c > index c3b7694a7485..bae9536de767 100644 > --- a/drivers/net/ethernet/freescale/fec_main.c > +++ b/drivers/net/ethernet/freescale/fec_main.c > @@ -2932,10 +2932,10 @@ static void fec_enet_get_strings(struct > net_device *netdev, > switch (stringset) { > case ETH_SS_STATS: > for (i = 0; i < ARRAY_SIZE(fec_stats); i++) { > - ethtool_sprintf(&data, "%s", fec_stats[i].name); > + ethtool_puts(&data, fec_stats[i].name); > } > for (i = 0; i < ARRAY_SIZE(fec_xdp_stat_strs); i++) { > - ethtool_sprintf(&data, "%s", fec_xdp_stat_strs[i]); > + ethtool_puts(&data, fec_xdp_stat_strs[i]); > } > page_pool_ethtool_stats_get_strings(data); >
Reviewed-by: Wei Fang <[email protected]> _______________________________________________ Intel-wired-lan mailing list [email protected] https://lists.osuosl.org/mailman/listinfo/intel-wired-lan
