drivers/staging/iio/adc/spear_adc.c: use clk_prepare_enable and clk_disable_unprepare
authorJulia Lawall <Julia.Lawall@lip6.fr>
Sun, 26 Aug 2012 16:00:00 +0000 (17:00 +0100)
committerJonathan Cameron <jic23@kernel.org>
Mon, 3 Sep 2012 19:26:41 +0000 (20:26 +0100)
Clk_prepare_enable and clk_disable_unprepare combine clk_prepare and
clk_enable, and clk_disable and clk_unprepare.  They make the code more
concise, and ensure that clk_unprepare is called when clk_enable fails.

A simplified version of the semantic patch that introduces calls to these
functions is as follows: (http://coccinelle.lip6.fr/)

// <smpl>
@@
expression e;
@@

- clk_prepare(e);
- clk_enable(e);
+ clk_prepare_enable(e);

@@
expression e;
@@

- clk_disable(e);
- clk_unprepare(e);
+ clk_disable_unprepare(e);
// </smpl>

Signed-off-by: Julia Lawall <Julia.Lawall@lip6.fr>
Reviewed-by: Viresh Kumar <viresh.kumar@linaro.org>
Signed-off-by: Jonathan Cameron <jic23@kernel.org>
drivers/staging/iio/adc/spear_adc.c

index 675c427c02ade52528676a3a9546347dd03da0b9..0b83e2e1f41096c13d8f7a4a3a6291f08578ea5b 100644 (file)
@@ -330,36 +330,30 @@ static int __devinit spear_adc_probe(struct platform_device *pdev)
                goto errout3;
        }
 
-       ret = clk_prepare(info->clk);
-       if (ret) {
-               dev_err(dev, "failed preparing clock\n");
-               goto errout4;
-       }
-
-       ret = clk_enable(info->clk);
+       ret = clk_prepare_enable(info->clk);
        if (ret) {
                dev_err(dev, "failed enabling clock\n");
-               goto errout5;
+               goto errout4;
        }
 
        irq = platform_get_irq(pdev, 0);
        if ((irq < 0) || (irq >= NR_IRQS)) {
                dev_err(dev, "failed getting interrupt resource\n");
                ret = -EINVAL;
-               goto errout6;
+               goto errout5;
        }
 
        ret = devm_request_irq(dev, irq, spear_adc_isr, 0, MOD_NAME, info);
        if (ret < 0) {
                dev_err(dev, "failed requesting interrupt\n");
-               goto errout6;
+               goto errout5;
        }
 
        if (of_property_read_u32(np, "sampling-frequency",
                                 &info->sampling_freq)) {
                dev_err(dev, "sampling-frequency missing in DT\n");
                ret = -EINVAL;
-               goto errout6;
+               goto errout5;
        }
 
        /*
@@ -389,16 +383,14 @@ static int __devinit spear_adc_probe(struct platform_device *pdev)
 
        ret = iio_device_register(iodev);
        if (ret)
-               goto errout6;
+               goto errout5;
 
        dev_info(dev, "SPEAR ADC driver loaded, IRQ %d\n", irq);
 
        return 0;
 
-errout6:
-       clk_disable(info->clk);
 errout5:
-       clk_unprepare(info->clk);
+       clk_disable_unprepare(info->clk);
 errout4:
        clk_put(info->clk);
 errout3:
@@ -416,8 +408,7 @@ static int __devexit spear_adc_remove(struct platform_device *pdev)
 
        iio_device_unregister(iodev);
        platform_set_drvdata(pdev, NULL);
-       clk_disable(info->clk);
-       clk_unprepare(info->clk);
+       clk_disable_unprepare(info->clk);
        clk_put(info->clk);
        iounmap(info->adc_base_spear6xx);
        iio_device_free(iodev);