ARM: PNX4008: convert i2c-pnx to use clk API enable/disable calls

clk_set_rate() is not supposed to be used to turn clocks on and off.
That's what clk_enable/clk_disable is for.

Acked-by: Vitaly Wool <vitalywool@gmail.com>
Signed-off-by: Russell King <rmk+kernel@arm.linux.org.uk>

1 files changed, 9 insertions, 9 deletions

diff --git a/drivers/i2c/busses/i2c-pnx.c b/drivers/i2c/busses/i2c-pnx.c
index 98462671cdf7..29f91774c4df 100644
--- a/drivers/i2c/busses/i2c-pnx.c
+++ b/drivers/i2c/busses/i2c-pnx.c
@@ -555,8 +555,8 @@ static int i2c_pnx_controller_suspend(struct platform_device *pdev,
         struct i2c_pnx_data *i2c_pnx = platform_get_drvdata(pdev);
         struct i2c_pnx_algo_data *alg_data = i2c_pnx->adapter->algo_data;
 
-        /* FIXME: disable clock? */
-        clk_set_rate(alg_data->clk, 1);
+        /* FIXME: shouldn't this be clk_disable? */
+        clk_enable(alg_data->clk);
 
         return 0;
 }
@@ -566,9 +566,7 @@ static int i2c_pnx_controller_resume(struct platform_device *pdev)
         struct i2c_pnx_data *i2c_pnx = platform_get_drvdata(pdev);
         struct i2c_pnx_algo_data *alg_data = i2c_pnx->adapter->algo_data;
 
-        clk_set_rate(alg_data->clk, 1);
-
-        return 0;
+        return clk_enable(alg_data->clk);
 }
 #else
 #define i2c_pnx_controller_suspend      NULL
@@ -630,7 +628,9 @@ static int __devinit i2c_pnx_probe(struct platform_device *pdev)
                 goto out_release;
         }
 
-        clk_set_rate(alg_data->clk, 1);
+        ret = clk_enable(alg_data->clk);
+        if (ret)
+                goto out_unmap;
 
         /*
          * Clock Divisor High This value is the number of system clocks
@@ -650,7 +650,7 @@ static int __devinit i2c_pnx_probe(struct platform_device *pdev)
         iowrite32(mcntrl_reset, I2C_REG_CTL(alg_data));
         if (wait_reset(I2C_PNX_TIMEOUT, alg_data)) {
                 ret = -ENODEV;
-                goto out_unmap;
+                goto out_clock;
         }
         init_completion(&alg_data->mif.complete);
 
@@ -676,7 +676,7 @@ static int __devinit i2c_pnx_probe(struct platform_device *pdev)
 out_irq:
         free_irq(alg_data->irq, i2c_pnx->adapter);
 out_clock:
-        clk_set_rate(alg_data->clk, 0);
+        clk_disable(alg_data->clk);
 out_unmap:
         iounmap((void *)alg_data->ioaddr);
 out_release:
@@ -697,7 +697,7 @@ static int __devexit i2c_pnx_remove(struct platform_device *pdev)
 
         free_irq(alg_data->irq, i2c_pnx->adapter);
         i2c_del_adapter(adap);
-        clk_set_rate(alg_data->clk, 0);
+        clk_disable(alg_data->clk);
         iounmap((void *)alg_data->ioaddr);
         release_mem_region(alg_data->base, I2C_PNX_REGION_SIZE);
         clk_put(alg_data->clk);