MLK-14498-4 dma: imx-sdma: remove the redundant clock operation

Remove the redundant clock operation.

Signed-off-by: Fugang Duan <fugang.duan@nxp.com>

diff --git a/drivers/dma/imx-sdma.c b/drivers/dma/imx-sdma.c
index a440556..8775bb3 100644 (file)
--- a/drivers/dma/imx-sdma.c
+++ b/drivers/dma/imx-sdma.c
@@ -1352,8 +1352,12 @@ static int sdma_alloc_chan_resources(struct dma_chan *chan)
        struct imx_dma_data default_data;
        int prio, ret;
 
-       clk_enable(sdmac->sdma->clk_ipg);
-       clk_enable(sdmac->sdma->clk_ahb);
+       ret = clk_enable(sdmac->sdma->clk_ipg);
+       if (ret)
+               return ret;
+       ret = clk_enable(sdmac->sdma->clk_ahb);
+       if (ret)
+               goto disable_clk_ipg;
 
        /*
         * dmatest(memcpy) will never call slave_config before prep, so we need
@@ -1391,13 +1395,6 @@ static int sdma_alloc_chan_resources(struct dma_chan *chan)
        sdmac->src_dualfifo = data->src_dualfifo;
        sdmac->dst_dualfifo = data->dst_dualfifo;
 
-       ret = clk_enable(sdmac->sdma->clk_ipg);
-       if (ret)
-               return ret;
-       ret = clk_enable(sdmac->sdma->clk_ahb);
-       if (ret)
-               goto disable_clk_ipg;
-
        ret = sdma_set_channel_priority(sdmac, prio);
        if (ret)
                goto disable_clk_ahb;