]> git.ipfire.org Git - thirdparty/openwrt.git/commitdiff
realtek: early ethernet probe in dsa setup
authorMarkus Stockhausen <markus.stockhausen@gmx.de>
Wed, 3 Sep 2025 08:02:51 +0000 (04:02 -0400)
committerRobert Marko <robimarko@gmail.com>
Sun, 7 Sep 2025 09:37:59 +0000 (11:37 +0200)
The ethernet and mdio code will be splitted. The dsa driver depends
on proper loading of both, before switch setup can start. Sadly there
are severe cleanup issues in the probe() function if one of the
required devices is not available.

As a temporary workaround provide a dedicated check function that
verifies if the ethernet platform device driver is loaded and can
be used.

Signed-off-by: Markus Stockhausen <markus.stockhausen@gmx.de>
Link: https://github.com/openwrt/openwrt/pull/19942
Signed-off-by: Robert Marko <robimarko@gmail.com>
target/linux/realtek/files-6.12/drivers/net/dsa/rtl83xx/common.c

index eef52e4ac206f12b42c591ae3f419d5e0a22bef8..80b36ec5a8e8c61427c26db6c629b8f14094ab6b 100644 (file)
@@ -1455,6 +1455,45 @@ static int rtl83xx_fib_event(struct notifier_block *this, unsigned long event, v
        return NOTIFY_DONE;
 }
 
+/*
+ * TODO: This check is usually built into the DSA initialization functions. After carving
+ * out the mdio driver from the ethernet driver, there are two drivers that must be loaded
+ * before the DSA setup can start. This driver has severe issues with handling of deferred
+ * probing. For now provide this function for early dependency checks.
+ */
+static int rtldsa_ethernet_loaded(struct platform_device *pdev)
+{
+       struct device_node *dn = pdev->dev.of_node;
+       struct device_node *ports, *port;
+       int ret = -EPROBE_DEFER;
+
+       ports = of_get_child_by_name(dn, "ports");
+       if (!ports)
+               return -ENODEV;
+
+       for_each_child_of_node(ports, port) {
+               struct device_node *eth_np;
+               struct platform_device *eth_pdev;
+
+               eth_np = of_parse_phandle(port, "ethernet", 0);
+               if (!eth_np)
+                       continue;
+
+               eth_pdev = of_find_device_by_node(eth_np);
+               of_node_put(eth_np);
+
+               if (!eth_pdev)
+                       continue;
+
+               if (eth_pdev->dev.driver)
+                       ret = 0;
+       }
+
+       of_node_put(ports);
+
+       return ret;     
+}
+
 static int __init rtl83xx_sw_probe(struct platform_device *pdev)
 {
        int i, err = 0;
@@ -1467,6 +1506,10 @@ static int __init rtl83xx_sw_probe(struct platform_device *pdev)
                dev_err(dev, "No DT found\n");
                return -EINVAL;
        }
+       
+       err = rtldsa_ethernet_loaded(pdev);
+       if (err)
+               return err;
 
        /* Initialize access to RTL switch tables */
        rtl_table_init();