fec: Fix the order for enabling/disabling the clocks
authorFabio Estevam <fabio.estevam@freescale.com>
Sun, 21 Jul 2013 16:25:00 +0000 (13:25 -0300)
committerDavid S. Miller <davem@davemloft.net>
Mon, 22 Jul 2013 23:50:31 +0000 (16:50 -0700)
On fec_probe the clocks are enabled in the following order:

clk_ahb -> clk_ipg -> clk_enet_out -> clk_ptp

, so in the error and remove paths we should disabled them in the opposite
order.

Also fix the order in the suspend/resume functions.

Signed-off-by: Fabio Estevam <fabio.estevam@freescale.com>
Signed-off-by: David S. Miller <davem@davemloft.net>
drivers/net/ethernet/freescale/fec_main.c

index beaa5762e0414b30b18c4f274fa95bb3f5d41519..5c34fd834d03f43810a86cb529bcf93523390b4f 100644 (file)
@@ -2179,12 +2179,12 @@ failed_init:
        if (fep->reg_phy)
                regulator_disable(fep->reg_phy);
 failed_regulator:
-       clk_disable_unprepare(fep->clk_ahb);
-       clk_disable_unprepare(fep->clk_ipg);
-       if (fep->clk_enet_out)
-               clk_disable_unprepare(fep->clk_enet_out);
        if (fep->clk_ptp)
                clk_disable_unprepare(fep->clk_ptp);
+       if (fep->clk_enet_out)
+               clk_disable_unprepare(fep->clk_enet_out);
+       clk_disable_unprepare(fep->clk_ipg);
+       clk_disable_unprepare(fep->clk_ahb);
 failed_clk:
 failed_ioremap:
        free_netdev(ndev);
@@ -2216,8 +2216,8 @@ fec_drv_remove(struct platform_device *pdev)
                ptp_clock_unregister(fep->ptp_clock);
        if (fep->clk_enet_out)
                clk_disable_unprepare(fep->clk_enet_out);
-       clk_disable_unprepare(fep->clk_ahb);
        clk_disable_unprepare(fep->clk_ipg);
+       clk_disable_unprepare(fep->clk_ahb);
        free_netdev(ndev);
 
        return 0;
@@ -2236,8 +2236,8 @@ fec_suspend(struct device *dev)
        }
        if (fep->clk_enet_out)
                clk_disable_unprepare(fep->clk_enet_out);
-       clk_disable_unprepare(fep->clk_ahb);
        clk_disable_unprepare(fep->clk_ipg);
+       clk_disable_unprepare(fep->clk_ahb);
 
        if (fep->reg_phy)
                regulator_disable(fep->reg_phy);
@@ -2258,10 +2258,10 @@ fec_resume(struct device *dev)
                        return ret;
        }
 
-       if (fep->clk_enet_out)
-               clk_prepare_enable(fep->clk_enet_out);
        clk_prepare_enable(fep->clk_ahb);
        clk_prepare_enable(fep->clk_ipg);
+       if (fep->clk_enet_out)
+               clk_prepare_enable(fep->clk_enet_out);
        if (netif_running(ndev)) {
                fec_restart(ndev, fep->full_duplex);
                netif_device_attach(ndev);