diff options
author | Rosen Penev <rosenp@gmail.com> | 2024-10-11 13:02:25 -0700 |
---|---|---|
committer | Jakub Kicinski <kuba@kernel.org> | 2024-10-14 17:57:30 -0700 |
commit | 2a22bead433e2f6b4340bbd5c8e992fbb17cbb52 (patch) | |
tree | ec021178d5d59b3cf028a53a891e72b2b6d6f15c | |
parent | 9de722c144d2ee67fb227fc527effbe77cf3ce30 (diff) | |
download | lwn-2a22bead433e2f6b4340bbd5c8e992fbb17cbb52.tar.gz lwn-2a22bead433e2f6b4340bbd5c8e992fbb17cbb52.zip |
net: mtk_eth_soc: use ethtool_puts
Allows simplifying get_strings and avoids manual pointer manipulation.
Tested on Belkin RT1800.
Signed-off-by: Rosen Penev <rosenp@gmail.com>
Reviewed-by: Gerhard Engleder <gerhard@engleder-embedded.com>
Link: https://patch.msgid.link/20241011200225.7403-1-rosenp@gmail.com
Signed-off-by: Jakub Kicinski <kuba@kernel.org>
-rw-r--r-- | drivers/net/ethernet/mediatek/mtk_eth_soc.c | 6 |
1 files changed, 2 insertions, 4 deletions
diff --git a/drivers/net/ethernet/mediatek/mtk_eth_soc.c b/drivers/net/ethernet/mediatek/mtk_eth_soc.c index a476a94a607d..e35a2e4d5525 100644 --- a/drivers/net/ethernet/mediatek/mtk_eth_soc.c +++ b/drivers/net/ethernet/mediatek/mtk_eth_soc.c @@ -4329,10 +4329,8 @@ static void mtk_get_strings(struct net_device *dev, u32 stringset, u8 *data) case ETH_SS_STATS: { struct mtk_mac *mac = netdev_priv(dev); - for (i = 0; i < ARRAY_SIZE(mtk_ethtool_stats); i++) { - memcpy(data, mtk_ethtool_stats[i].str, ETH_GSTRING_LEN); - data += ETH_GSTRING_LEN; - } + for (i = 0; i < ARRAY_SIZE(mtk_ethtool_stats); i++) + ethtool_puts(&data, mtk_ethtool_stats[i].str); if (mtk_page_pool_enabled(mac->hw)) page_pool_ethtool_stats_get_strings(data); break; |