[PATCH net-next v3 0/2] net: ethernet: litex: minor improvment for the codebase

[PATCH net-next v3 0/2] net: ethernet: litex: minor improvment for the codebase

[Inochi Amaoto]

Improve the litex code for using the device managed function to register
netdev and replace all the "pdev->dev" with dev pointer instead.

Change from v2:
- https://lore.kernel.org/netdev/20260119040446.741970-1-inochiama@gmail.com
1. Apply Andrew' tag
2. patch 2: wrap the line to match the characters limit

Change from v1:
- https://lore.kernel.org/netdev/20260116003150.183070-1-inochiama@gmail.com/
1. separate the original patch into two patches.

Inochi Amaoto (2):
  net: ethernet: litex: use devm_register_netdev() to register netdev
  net: ethernet: litex: use device pointer to simplify code.

 drivers/net/ethernet/litex/litex_liteeth.c | 21 +++++++--------------
 1 file changed, 7 insertions(+), 14 deletions(-)

--
2.53.0

[PATCH net-next v3 1/2] net: ethernet: litex: use devm_register_netdev() to register netdev

[Inochi Amaoto]

Use devm_register_netdev to avoid unnecessary remove() callback in
platform_driver structure.

Signed-off-by: Inochi Amaoto <inochiama@gmail.com>
Reviewed-by: Andrew Lunn <andrew@lunn.ch>
---
 drivers/net/ethernet/litex/litex_liteeth.c | 11 ++---------
 1 file changed, 2 insertions(+), 9 deletions(-)

diff --git a/drivers/net/ethernet/litex/litex_liteeth.c b/drivers/net/ethernet/litex/litex_liteeth.c
index 829a4b828f8e..67ad1058c2ab 100644
--- a/drivers/net/ethernet/litex/litex_liteeth.c
+++ b/drivers/net/ethernet/litex/litex_liteeth.c
@@ -232,6 +232,7 @@ static void liteeth_setup_slots(struct liteeth *priv)

 static int liteeth_probe(struct platform_device *pdev)
 {
+	struct device *dev = &pdev->dev;
 	struct net_device *netdev;
 	void __iomem *buf_base;
 	struct liteeth *priv;
@@ -282,7 +283,7 @@ static int liteeth_probe(struct platform_device *pdev)

 	netdev->netdev_ops = &liteeth_netdev_ops;

-	err = register_netdev(netdev);
+	err = devm_register_netdev(dev, netdev);
 	if (err) {
 		dev_err(&pdev->dev, "Failed to register netdev %d\n", err);
 		return err;
@@ -294,13 +295,6 @@ static int liteeth_probe(struct platform_device *pdev)
 	return 0;
 }

-static void liteeth_remove(struct platform_device *pdev)
-{
-	struct net_device *netdev = platform_get_drvdata(pdev);
-
-	unregister_netdev(netdev);
-}
-
 static const struct of_device_id liteeth_of_match[] = {
 	{ .compatible = "litex,liteeth" },
 	{ }
@@ -309,7 +303,6 @@ MODULE_DEVICE_TABLE(of, liteeth_of_match);

 static struct platform_driver liteeth_driver = {
 	.probe = liteeth_probe,
-	.remove = liteeth_remove,
 	.driver = {
 		.name = DRV_NAME,
 		.of_match_table = liteeth_of_match,
--
2.53.0

[PATCH net-next v3 2/2] net: ethernet: litex: use device pointer to simplify code.

[Inochi Amaoto]

As there is already a device pointer in the probe function, replace
all "&pdev->dev" pattern with this predefined device pointer.

Signed-off-by: Inochi Amaoto <inochiama@gmail.com>
Reviewed-by: Andrew Lunn <andrew@lunn.ch>
---
 drivers/net/ethernet/litex/litex_liteeth.c | 10 +++++-----
 1 file changed, 5 insertions(+), 5 deletions(-)

diff --git a/drivers/net/ethernet/litex/litex_liteeth.c b/drivers/net/ethernet/litex/litex_liteeth.c
index 67ad1058c2ab..108d0a0db206 100644
--- a/drivers/net/ethernet/litex/litex_liteeth.c
+++ b/drivers/net/ethernet/litex/litex_liteeth.c
@@ -238,7 +238,7 @@ static int liteeth_probe(struct platform_device *pdev)
 	struct liteeth *priv;
 	int irq, err;

-	netdev = devm_alloc_etherdev(&pdev->dev, sizeof(*priv));
+	netdev = devm_alloc_etherdev(dev, sizeof(*priv));
 	if (!netdev)
 		return -ENOMEM;

@@ -247,9 +247,9 @@ static int liteeth_probe(struct platform_device *pdev)

 	priv = netdev_priv(netdev);
 	priv->netdev = netdev;
-	priv->dev = &pdev->dev;
+	priv->dev = dev;

-	netdev->tstats = devm_netdev_alloc_pcpu_stats(&pdev->dev,
+	netdev->tstats = devm_netdev_alloc_pcpu_stats(dev,
 						      struct pcpu_sw_netstats);
 	if (!netdev->tstats)
 		return -ENOMEM;
@@ -277,7 +277,7 @@ static int liteeth_probe(struct platform_device *pdev)
 	priv->tx_base = buf_base + priv->num_rx_slots * priv->slot_size;
 	priv->tx_slot = 0;

-	err = of_get_ethdev_address(pdev->dev.of_node, netdev);
+	err = of_get_ethdev_address(dev->of_node, netdev);
 	if (err)
 		eth_hw_addr_random(netdev);

@@ -285,7 +285,7 @@ static int liteeth_probe(struct platform_device *pdev)

 	err = devm_register_netdev(dev, netdev);
 	if (err) {
-		dev_err(&pdev->dev, "Failed to register netdev %d\n", err);
+		dev_err(dev, "Failed to register netdev %d\n", err);
 		return err;
 	}

--
2.53.0

Re: [PATCH net-next v3 0/2] net: ethernet: litex: minor improvment for the codebase

[patchwork-bot+netdevbpf]

Hello:

This series was applied to netdev/net-next.git (main)
by Jakub Kicinski <kuba@kernel.org>:

On Fri, 27 Feb 2026 08:33:46 +0800 you wrote:
> Improve the litex code for using the device managed function to register
> netdev and replace all the "pdev->dev" with dev pointer instead.
> 
> Change from v2:
> - https://lore.kernel.org/netdev/20260119040446.741970-1-inochiama@gmail.com
> 1. Apply Andrew' tag
> 2. patch 2: wrap the line to match the characters limit
> 
> [...]

Here is the summary with links:
  - [net-next,v3,1/2] net: ethernet: litex: use devm_register_netdev() to register netdev
    (no matching commit)
  - [net-next,v3,2/2] net: ethernet: litex: use device pointer to simplify code.
    https://git.kernel.org/netdev/net-next/c/621e3634dfab

You are awesome, thank you!
-- 
Deet-doot-dot, I am a bot.
https://korg.docs.kernel.org/patchwork/pwbot.html