]> git.dujemihanovic.xyz Git - u-boot.git/commitdiff
net: fec: Only unregister MII bus if we registered it
authorSean Anderson <sean.anderson@seco.com>
Thu, 15 Apr 2021 17:06:09 +0000 (13:06 -0400)
committerStefano Babic <sbabic@denx.de>
Sun, 2 May 2021 10:46:54 +0000 (12:46 +0200)
If we fail to probe for whatever reason, we cannot unregister/free the
MII bus unless we registered it with fec_get_miibus. This fixes FECs
sharing an MDIO bus from destroying it, preventing the other FEC from
using it.

Fixes: 6a895d039b ("net: Update eQos driver and FEC driver to use eth phy interfaces")
Signed-off-by: Sean Anderson <sean.anderson@seco.com>
Reviewed-by: Ramon Fried <rfried.dev@gmail.com>
drivers/net/fec_mxc.c

index 59f5a14e54e819d1f078286539d305716ade076e..4fd5c01b4ae19080203ae68b8d7ff013407aa91d 100644 (file)
@@ -1355,6 +1355,7 @@ static void fec_gpio_reset(struct fec_priv *priv)
 
 static int fecmxc_probe(struct udevice *dev)
 {
+       bool dm_mii_bus = true;
        struct eth_pdata *pdata = dev_get_plat(dev);
        struct fec_priv *priv = dev_get_priv(dev);
        struct mii_dev *bus = NULL;
@@ -1462,6 +1463,7 @@ static int fecmxc_probe(struct udevice *dev)
 #endif
 
        if (!bus) {
+               dm_mii_bus = false;
 #ifdef CONFIG_FEC_MXC_MDIO_BASE
                bus = fec_get_miibus((ulong)CONFIG_FEC_MXC_MDIO_BASE,
                                     dev_seq(dev));
@@ -1507,8 +1509,10 @@ static int fecmxc_probe(struct udevice *dev)
        return 0;
 
 err_phy:
-       mdio_unregister(bus);
-       free(bus);
+       if (!dm_mii_bus) {
+               mdio_unregister(bus);
+               free(bus);
+       }
 err_mii:
 err_timeout:
        fec_free_descs(priv);