net: ethernet: slicoss: remove redundant increment of pointer data

ColinIanKing · kuba-moo · commit 3bdd85e2e350 · 2023-07-28T15:37:08.000-07:00
The pointer data is being incremented but this change to the pointer is not used afterwards. The increment is redundant and can be removed. Signed-off-by: Colin Ian King <colin.i.king@gmail.com> Acked-by: Lino Sanfilippo <LinoSanfilippo@gmx.de> Link: https://lore.kernel.org/r/20230726164522.369206-1-colin.i.king@gmail.com Signed-off-by: Jakub Kicinski <kuba@kernel.org>
diff --git a/drivers/net/ethernet/alacritech/slicoss.c b/drivers/net/ethernet/alacritech/slicoss.c
@@ -1520,10 +1520,8 @@ static void slic_get_ethtool_stats(struct net_device *dev,
 
 static void slic_get_strings(struct net_device *dev, u32 stringset, u8 *data)
 {
-	if (stringset == ETH_SS_STATS) {
+	if (stringset == ETH_SS_STATS)
 		memcpy(data, slic_stats_strings, sizeof(slic_stats_strings));
-		data += sizeof(slic_stats_strings);
-	}
 }
 
 static void slic_get_drvinfo(struct net_device *dev,

Original file line number	Diff line number	Diff line change
`@@ -1520,10 +1520,8 @@ static void slic_get_ethtool_stats(struct net_device *dev,`
`1520`	`1520`
`1521`	`1521`	`static void slic_get_strings(struct net_device dev, u32 stringset, u8 data)`
`1522`	`1522`	`{`
`1523`		`- if (stringset == ETH_SS_STATS) {`
	`1523`	`+ if (stringset == ETH_SS_STATS)`
`1524`	`1524`	`memcpy(data, slic_stats_strings, sizeof(slic_stats_strings));`
`1525`		`- data += sizeof(slic_stats_strings);`
`1526`		`- }`
`1527`	`1525`	`}`
`1528`	`1526`
`1529`	`1527`	`static void slic_get_drvinfo(struct net_device *dev,`