[PATCH 3/3] net: xilinx: Force a probe failure if it cannot setup phy
Andy Chiu
andy.chiu at sifive.com
Thu Jan 20 08:34:20 CET 2022
Or we may get load access faults afterward.
The `phydev` field on axi-ethernet’s private struct is not set on a
failed phy_connect():
axi_emac_probe()
=> axiemac_phy_init()
=> priv->phydev = phy_connect() <--- may fail
However, all of the following calls on `axi_emac_ops` assume a valid
`phydev` pointer. For example:
axiemac_start()
=> setup_phy()
=> phy_startup()
=> if (phydev->drv->startup) <--- deref of phydev
return phydev->drv->startup(phydev);
Thus, it would be better to fail at the driver probe and let u-boot
handle the rest (e.g. probe the driver again if needed), rather than
having access faults.
Signed-off-by: Andy Chiu <andy.chiu at sifive.com>
Reviewed-by: Greentime Hu <greentime.hu at sifive.com>
---
drivers/net/xilinx_axi_emac.c | 10 ++++++++--
1 file changed, 8 insertions(+), 2 deletions(-)
diff --git a/drivers/net/xilinx_axi_emac.c b/drivers/net/xilinx_axi_emac.c
index dbda6c70d8..08322aff88 100644
--- a/drivers/net/xilinx_axi_emac.c
+++ b/drivers/net/xilinx_axi_emac.c
@@ -782,10 +782,16 @@ static int axiemac_setup_emac(struct udevice *dev)
ret = mdio_register_seq(priv->bus, dev_seq(dev));
if (ret)
- return ret;
+ goto fail;
- axiemac_phy_init(dev);
+ ret = axiemac_phy_init(dev);
+ if (ret)
+ goto fail_free_mdio;
+fail_free_mdio:
+ mdio_unregister(priv->bus);
+ mdio_free(priv->bus);
+fail:
return ret;
}
--
2.34.1
More information about the U-Boot
mailing list