Donate to e Foundation | Murena handsets with /e/OS | Own a part of Murena! Learn more

Commit bc9c21f8 authored by Arnaud Patard's avatar Arnaud Patard Committed by Greg Kroah-Hartman
Browse files

drivers/net/ethernet/marvell/mvmdio.c: Fix non OF case 

commit d934423ac26ed373dfe089734d505dca5ff679b6 upstream.

Orion5.x systems are still using machine files and not device-tree.
Commit 96cb4342 ("net: mvmdio: allow up to three clocks to be
specified for orion-mdio") has replaced devm_clk_get() with of_clk_get(),
leading to a oops at boot and not working network, as reported in
https://lists.debian.org/debian-arm/2019/07/msg00088.html and possibly in
https://bugs.debian.org/cgi-bin/bugreport.cgi?bug=908712.

Link: https://lists.debian.org/debian-arm/2019/07/msg00088.html


Fixes: 96cb4342 ("net: mvmdio: allow up to three clocks to be specified for orion-mdio")
Signed-off-by: default avatarArnaud Patard <arnaud.patard@rtp-net.org>
Reviewed-by: default avatarAndrew Lunn <andrew@lunn.ch>
Signed-off-by: default avatarDavid S. Miller <davem@davemloft.net>
Signed-off-by: default avatarGreg Kroah-Hartman <gregkh@linuxfoundation.org>
parent 9fd231b2

drivers/net/ethernet/marvell/mvmdio.c

+16 −6
Original line number Diff line number Diff line
@@ -319,6 +319,7 @@ static int orion_mdio_probe(struct platform_device *pdev) 


	init_waitqueue_head(&dev->smi_busy_wait);



	if (pdev->dev.of_node) {

		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) {
@@ -329,6 +330,15 @@ static int orion_mdio_probe(struct platform_device *pdev) 
				break;

			clk_prepare_enable(dev->clk[i]);

		}

	} else {

		dev->clk[0] = clk_get(&pdev->dev, NULL);

		if (PTR_ERR(dev->clk[0]) == -EPROBE_DEFER) {

			ret = -EPROBE_DEFER;

			goto out_clk;

		}

		if (!IS_ERR(dev->clk[0]))

			clk_prepare_enable(dev->clk[0]);

	}



	dev->err_interrupt = platform_get_irq(pdev, 0);

	if (dev->err_interrupt > 0 &&