diff options
author | Fabio Estevam <fabio.estevam@freescale.com> | 2014-10-17 17:14:01 -0300 |
---|---|---|
committer | Brian Norris <computersforpeace@gmail.com> | 2014-10-22 01:35:41 -0700 |
commit | 77adc085aee0c96dc3536403b197707316c7c241 (patch) | |
tree | b1006e00ff03424a1a1e6a19475a325efee76b6f /drivers | |
parent | b26171e392c7d3952b6eeb6da62a861c5317e438 (diff) | |
download | op-kernel-dev-77adc085aee0c96dc3536403b197707316c7c241.zip op-kernel-dev-77adc085aee0c96dc3536403b197707316c7c241.tar.gz |
mtd: fsl-quadspi: Let the clocks error path be clearer
When clk_prepare_enable(q->clk) fails it is clearer to disable the previous
acquired clock (q->clk_en) in the error path rather than doing it locally.
So disable q->clk_en in the error path only.
Signed-off-by: Fabio Estevam <fabio.estevam@freescale.com>
Acked-by: Huang Shijie <shijie8@gmail.com>
Signed-off-by: Brian Norris <computersforpeace@gmail.com>
Diffstat (limited to 'drivers')
-rw-r--r-- | drivers/mtd/spi-nor/fsl-quadspi.c | 4 |
1 files changed, 2 insertions, 2 deletions
diff --git a/drivers/mtd/spi-nor/fsl-quadspi.c b/drivers/mtd/spi-nor/fsl-quadspi.c index 94e7355..b6aab38 100644 --- a/drivers/mtd/spi-nor/fsl-quadspi.c +++ b/drivers/mtd/spi-nor/fsl-quadspi.c @@ -849,9 +849,8 @@ static int fsl_qspi_probe(struct platform_device *pdev) ret = clk_prepare_enable(q->clk); if (ret) { - clk_disable_unprepare(q->clk_en); dev_err(dev, "can not enable the qspi clock\n"); - goto map_failed; + goto clk_failed; } /* find the irq */ @@ -965,6 +964,7 @@ last_init_failed: irq_failed: clk_disable_unprepare(q->clk); +clk_failed: clk_disable_unprepare(q->clk_en); map_failed: dev_err(dev, "Freescale QuadSPI probe failed\n"); |