Skip to content

Commit 29c9e85

Browse files
Andi Shytiwsakernel
authored andcommitted
i2c: at91-core: Use devm_clk_get_enabled()
Replace the pair of functions, devm_clk_get() and clk_prepare_enable(), with a single function devm_clk_get_enabled(). Signed-off-by: Andi Shyti <[email protected]> Acked-by: Nicolas Ferre <[email protected]> [wsa: rebased] Signed-off-by: Wolfram Sang <[email protected]>
1 parent ba63f99 commit 29c9e85

File tree

1 file changed

+3
-7
lines changed

1 file changed

+3
-7
lines changed

drivers/i2c/busses/i2c-at91-core.c

Lines changed: 3 additions & 7 deletions
Original file line numberDiff line numberDiff line change
@@ -221,11 +221,10 @@ static int at91_twi_probe(struct platform_device *pdev)
221221

222222
platform_set_drvdata(pdev, dev);
223223

224-
dev->clk = devm_clk_get(dev->dev, NULL);
224+
dev->clk = devm_clk_get_enabled(dev->dev, NULL);
225225
if (IS_ERR(dev->clk))
226-
return dev_err_probe(dev->dev, PTR_ERR(dev->clk), "no clock defined\n");
227-
228-
clk_prepare_enable(dev->clk);
226+
return dev_err_probe(dev->dev, PTR_ERR(dev->clk),
227+
"failed to enable clock\n");
229228

230229
snprintf(dev->adapter.name, sizeof(dev->adapter.name), "AT91");
231230
i2c_set_adapdata(&dev->adapter, dev);
@@ -254,8 +253,6 @@ static int at91_twi_probe(struct platform_device *pdev)
254253

255254
rc = i2c_add_numbered_adapter(&dev->adapter);
256255
if (rc) {
257-
clk_disable_unprepare(dev->clk);
258-
259256
pm_runtime_disable(dev->dev);
260257
pm_runtime_set_suspended(dev->dev);
261258

@@ -272,7 +269,6 @@ static void at91_twi_remove(struct platform_device *pdev)
272269
struct at91_twi_dev *dev = platform_get_drvdata(pdev);
273270

274271
i2c_del_adapter(&dev->adapter);
275-
clk_disable_unprepare(dev->clk);
276272

277273
pm_runtime_disable(dev->dev);
278274
pm_runtime_set_suspended(dev->dev);

0 commit comments

Comments
 (0)