[openwrt/openwrt] bcm4908: fix -EPROBE_DEFER support in bcm4908_enet

LEDE Commits lede-commits at lists.infradead.org
Tue Sep 20 22:54:30 PDT 2022


rmilecki pushed a commit to openwrt/openwrt.git, branch master:
https://git.openwrt.org/1722e23ffcf9038114142a2129c29eb3cdec8ff9

commit 1722e23ffcf9038114142a2129c29eb3cdec8ff9
Author: Rafał Miłecki <rafal at milecki.pl>
AuthorDate: Wed Sep 21 07:53:52 2022 +0200

    bcm4908: fix -EPROBE_DEFER support in bcm4908_enet
    
    Signed-off-by: Rafał Miłecki <rafal at milecki.pl>
---
 ...-bcm4908_enet-reset-DMA-rings-sw-indexes.patch} |  4 +-
 ...m-bcm4908_enet-handle-EPROBE_DEFER-when-g.patch | 55 ++++++++++++++++++++++
 2 files changed, 58 insertions(+), 1 deletion(-)

diff --git a/target/linux/bcm4908/patches-5.10/170-net-broadcom-bcm4908_enet-reset-DMA-rings-sw-indexes.patch b/target/linux/bcm4908/patches-5.10/077-v5.14-net-broadcom-bcm4908_enet-reset-DMA-rings-sw-indexes.patch
similarity index 89%
rename from target/linux/bcm4908/patches-5.10/170-net-broadcom-bcm4908_enet-reset-DMA-rings-sw-indexes.patch
rename to target/linux/bcm4908/patches-5.10/077-v5.14-net-broadcom-bcm4908_enet-reset-DMA-rings-sw-indexes.patch
index 7e82230f9a..02290c8e64 100644
--- a/target/linux/bcm4908/patches-5.10/170-net-broadcom-bcm4908_enet-reset-DMA-rings-sw-indexes.patch
+++ b/target/linux/bcm4908/patches-5.10/077-v5.14-net-broadcom-bcm4908_enet-reset-DMA-rings-sw-indexes.patch
@@ -1,5 +1,6 @@
+From ddeacc4f6494e07cbb6f033627926623f3e7a9d0 Mon Sep 17 00:00:00 2001
 From: =?UTF-8?q?Rafa=C5=82=20Mi=C5=82ecki?= <rafal at milecki.pl>
-Date: Tue, 22 Jun 2021 07:05:04 +0200
+Date: Tue, 22 Jun 2021 07:24:15 +0200
 Subject: [PATCH] net: broadcom: bcm4908_enet: reset DMA rings sw indexes
  properly
 MIME-Version: 1.0
@@ -15,6 +16,7 @@ down & up sequence.
 
 Fixes: 4feffeadbcb2 ("net: broadcom: bcm4908enet: add BCM4908 controller driver")
 Signed-off-by: Rafał Miłecki <rafal at milecki.pl>
+Signed-off-by: David S. Miller <davem at davemloft.net>
 ---
  drivers/net/ethernet/broadcom/bcm4908_enet.c | 6 +++---
  1 file changed, 3 insertions(+), 3 deletions(-)
diff --git a/target/linux/bcm4908/patches-5.10/078-v6.1-net-broadcom-bcm4908_enet-handle-EPROBE_DEFER-when-g.patch b/target/linux/bcm4908/patches-5.10/078-v6.1-net-broadcom-bcm4908_enet-handle-EPROBE_DEFER-when-g.patch
new file mode 100644
index 0000000000..702b7641fd
--- /dev/null
+++ b/target/linux/bcm4908/patches-5.10/078-v6.1-net-broadcom-bcm4908_enet-handle-EPROBE_DEFER-when-g.patch
@@ -0,0 +1,55 @@
+From e93a766da57fff3273bcb618edf5dfca1fb86b89 Mon Sep 17 00:00:00 2001
+From: =?UTF-8?q?Rafa=C5=82=20Mi=C5=82ecki?= <rafal at milecki.pl>
+Date: Thu, 15 Sep 2022 15:30:13 +0200
+Subject: [PATCH] net: broadcom: bcm4908_enet: handle -EPROBE_DEFER when
+ getting MAC
+MIME-Version: 1.0
+Content-Type: text/plain; charset=UTF-8
+Content-Transfer-Encoding: 8bit
+
+Reading MAC from OF may return -EPROBE_DEFER if underlaying NVMEM device
+isn't ready yet. In such case pass that error code up and "wait" to be
+probed later.
+
+Signed-off-by: Rafał Miłecki <rafal at milecki.pl>
+Link: https://lore.kernel.org/r/20220915133013.2243-1-zajec5@gmail.com
+Signed-off-by: Jakub Kicinski <kuba at kernel.org>
+---
+ drivers/net/ethernet/broadcom/bcm4908_enet.c | 13 +++++++++----
+ 1 file changed, 9 insertions(+), 4 deletions(-)
+
+--- a/drivers/net/ethernet/broadcom/bcm4908_enet.c
++++ b/drivers/net/ethernet/broadcom/bcm4908_enet.c
+@@ -714,7 +714,9 @@ static int bcm4908_enet_probe(struct pla
+ 		return err;
+ 
+ 	SET_NETDEV_DEV(netdev, &pdev->dev);
+-	of_get_mac_address(dev->of_node, netdev->dev_addr);
++	err = of_get_mac_address(dev->of_node, netdev->dev_addr);
++	if (err == -EPROBE_DEFER)
++		goto err_dma_free;
+ 	if (!is_valid_ether_addr(netdev->dev_addr))
+ 		eth_hw_addr_random(netdev);
+ 	netdev->netdev_ops = &bcm4908_enet_netdev_ops;
+@@ -725,14 +727,17 @@ static int bcm4908_enet_probe(struct pla
+ 	netif_napi_add(netdev, &enet->rx_ring.napi, bcm4908_enet_poll_rx, NAPI_POLL_WEIGHT);
+ 
+ 	err = register_netdev(netdev);
+-	if (err) {
+-		bcm4908_enet_dma_free(enet);
+-		return err;
+-	}
++	if (err)
++		goto err_dma_free;
+ 
+ 	platform_set_drvdata(pdev, enet);
+ 
+ 	return 0;
++
++err_dma_free:
++	bcm4908_enet_dma_free(enet);
++
++	return err;
+ }
+ 
+ static int bcm4908_enet_remove(struct platform_device *pdev)




More information about the lede-commits mailing list