clk: apcs-msm8916: simplify probe cleanup by using devm
authorMatti Vaittinen <matti.vaittinen@fi.rohmeurope.com>
Tue, 4 Dec 2018 11:39:06 +0000 (13:39 +0200)
committerStephen Boyd <sboyd@kernel.org>
Wed, 5 Dec 2018 17:22:33 +0000 (09:22 -0800)
use devm variant for of_provider registration.

Signed-off-by: Matti Vaittinen <matti.vaittinen@fi.rohmeurope.com>
[sboyd@kernel.org: Drop unused parent pointer]
Signed-off-by: Stephen Boyd <sboyd@kernel.org>
drivers/clk/qcom/apcs-msm8916.c

index b1cc8db..a6c89a3 100644 (file)
@@ -96,8 +96,8 @@ static int qcom_apcs_msm8916_clk_probe(struct platform_device *pdev)
                goto err;
        }
 
-       ret = of_clk_add_hw_provider(parent->of_node, of_clk_hw_simple_get,
-                                    &a53cc->clkr.hw);
+       ret = devm_of_clk_add_hw_provider(dev, of_clk_hw_simple_get,
+                                         &a53cc->clkr.hw);
        if (ret) {
                dev_err(dev, "failed to add clock provider: %d\n", ret);
                goto err;
@@ -115,10 +115,8 @@ err:
 static int qcom_apcs_msm8916_clk_remove(struct platform_device *pdev)
 {
        struct clk_regmap_mux_div *a53cc = platform_get_drvdata(pdev);
-       struct device *parent = pdev->dev.parent;
 
        clk_notifier_unregister(a53cc->pclk, &a53cc->clk_nb);
-       of_clk_del_provider(parent->of_node);
 
        return 0;
 }