Skip to content

Commit

Permalink
clk: lmk04832: Use managed of_clk_add_hw_provider()
Browse files Browse the repository at this point in the history
Use the managed `devm_of_clk_add_hw_provider()` instead of
`of_clk_add_hw_provider()`.

This makes sure the provider gets automatically removed on unbind.

Signed-off-by: Lars-Peter Clausen <lars@metafoo.de>
Link: https://lore.kernel.org/r/20230410014502.27929-6-lars@metafoo.de
Signed-off-by: Stephen Boyd <sboyd@kernel.org>
  • Loading branch information
Lars-Peter Clausen authored and Stephen Boyd committed Apr 10, 2023
1 parent 56d144d commit 56fc9a3
Showing 1 changed file with 2 additions and 3 deletions.
5 changes: 2 additions & 3 deletions drivers/clk/clk-lmk04832.c
Original file line number Diff line number Diff line change
Expand Up @@ -1522,8 +1522,8 @@ static int lmk04832_probe(struct spi_device *spi)
}

lmk->clk_data->num = info->num_channels;
ret = of_clk_add_hw_provider(lmk->dev->of_node, of_clk_hw_onecell_get,
lmk->clk_data);
ret = devm_of_clk_add_hw_provider(lmk->dev, of_clk_hw_onecell_get,
lmk->clk_data);
if (ret) {
dev_err(lmk->dev, "failed to add provider (%d)\n", ret);
goto err_disable_vco;
Expand All @@ -1547,7 +1547,6 @@ static void lmk04832_remove(struct spi_device *spi)
struct lmk04832 *lmk = spi_get_drvdata(spi);

clk_disable_unprepare(lmk->oscin);
of_clk_del_provider(spi->dev.of_node);
}

static const struct spi_device_id lmk04832_id[] = {
Expand Down

0 comments on commit 56fc9a3

Please sign in to comment.