diff options
| author | Matti Vaittinen <matti.vaittinen@fi.rohmeurope.com> | 2018-12-04 12:39:06 +0100 |
|---|---|---|
| committer | Stephen Boyd <sboyd@kernel.org> | 2018-12-05 18:22:33 +0100 |
| commit | 7265c3cbbf9f038016b358b8e529a6a367f1af53 (patch) | |
| tree | fa8c06764c4cd9095b27a41d4caa705d248899d0 | |
| parent | clk: clk-twl6040: Free of_provider at remove (diff) | |
| download | linux-7265c3cbbf9f038016b358b8e529a6a367f1af53.tar.xz linux-7265c3cbbf9f038016b358b8e529a6a367f1af53.zip | |
clk: apcs-msm8916: simplify probe cleanup by using devm
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>
| -rw-r--r-- | drivers/clk/qcom/apcs-msm8916.c | 6 |
1 files changed, 2 insertions, 4 deletions
diff --git a/drivers/clk/qcom/apcs-msm8916.c b/drivers/clk/qcom/apcs-msm8916.c index b1cc8dbcd327..a6c89a310b18 100644 --- a/drivers/clk/qcom/apcs-msm8916.c +++ b/drivers/clk/qcom/apcs-msm8916.c @@ -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; } |