net: mvmdio: defer probe of orion-mdio if a clock is not ready
authorJosua Mayer <josua@solid-run.com>
Tue, 9 Jul 2019 13:01:01 +0000 (15:01 +0200)
committerDavid S. Miller <davem@davemloft.net>
Tue, 9 Jul 2019 20:03:04 +0000 (13:03 -0700)
Defer probing of the orion-mdio interface when getting a clock returns
EPROBE_DEFER. This avoids locking up the Armada 8k SoC when mdio is used
before all clocks have been enabled.

Signed-off-by: Josua Mayer <josua@solid-run.com>
Reviewed-by: Andrew Lunn <andrew@lunn.ch>
Signed-off-by: David S. Miller <davem@davemloft.net>
drivers/net/ethernet/marvell/mvmdio.c

index eba18065a4dafad2deffd513af3fc12c02ebf04e..f660cc2b82583fdc6d20cd1e0761dc71b294603b 100644 (file)
@@ -321,6 +321,10 @@ static int orion_mdio_probe(struct platform_device *pdev)
 
        for (i = 0; i < ARRAY_SIZE(dev->clk); i++) {
                dev->clk[i] = of_clk_get(pdev->dev.of_node, i);
+               if (PTR_ERR(dev->clk[i]) == -EPROBE_DEFER) {
+                       ret = -EPROBE_DEFER;
+                       goto out_clk;
+               }
                if (IS_ERR(dev->clk[i]))
                        break;
                clk_prepare_enable(dev->clk[i]);
@@ -366,6 +370,7 @@ out_mdio:
        if (dev->err_interrupt > 0)
                writel(0, dev->regs + MVMDIO_ERR_INT_MASK);
 
+out_clk:
        for (i = 0; i < ARRAY_SIZE(dev->clk); i++) {
                if (IS_ERR(dev->clk[i]))
                        break;