net: ethernet: lantiq_etop: remove redundant device name setup

abajk · kuba-moo · commit 9283477e2891 · 2024-07-15T07:02:22.000-07:00
The same name is set when allocating the netdevice structure in the alloc_etherdev_mq()->alloc_etherrdev_mqs() function. Therefore, there is no need to manually set it. This fixes CheckPatch warnings: WARNING: Prefer strscpy over strcpy - see: KSPP#88 strcpy(dev->name, "eth%d"); Signed-off-by: Aleksander Jan Bajkowski <olek2@wp.pl> Link: https://patch.msgid.link/20240713170920.863171-1-olek2@wp.pl Signed-off-by: Jakub Kicinski <kuba@kernel.org>
diff --git a/drivers/net/ethernet/lantiq_etop.c b/drivers/net/ethernet/lantiq_etop.c
@@ -675,7 +675,6 @@ ltq_etop_probe(struct platform_device *pdev)
 		err = -ENOMEM;
 		goto err_out;
 	}
-	strcpy(dev->name, "eth%d");
 	dev->netdev_ops = &ltq_eth_netdev_ops;
 	dev->ethtool_ops = &ltq_etop_ethtool_ops;
 	priv = netdev_priv(dev);

Original file line number	Diff line number	Diff line change
`@@ -675,7 +675,6 @@ ltq_etop_probe(struct platform_device *pdev)`
`675`	`675`	`err = -ENOMEM;`
`676`	`676`	`goto err_out;`
`677`	`677`	`}`
`678`		`- strcpy(dev->name, "eth%d");`
`679`	`678`	`dev->netdev_ops = &ltq_eth_netdev_ops;`
`680`	`679`	`dev->ethtool_ops = &ltq_etop_ethtool_ops;`
`681`	`680`	`priv = netdev_priv(dev);`