Commit 8529fc0a authored by Rahul Tanwar's avatar Rahul Tanwar Committed by Stephen Boyd
Browse files

clk: intel: Use devm_clk_hw_register() instead of clk_hw_register() 



To ensure that clks are unregistered in case of any failure, use
devm_clk_hw_register() instead of clk_hw_register().

Signed-off-by: default avatarRahul Tanwar <rahul.tanwar@linux.intel.com>
Link: https://lore.kernel.org/r/7ef7009b4e9f986fd6dfbf487c0e85de68a4ba9b.1594880946.git.rahul.tanwar@linux.intel.com


Signed-off-by: default avatarStephen Boyd <sboyd@kernel.org>
parent b3a9e3b9

drivers/clk/x86/clk-cgu-pll.c

+1 −1
Original line number Diff line number Diff line
@@ -128,7 +128,7 @@ lgm_clk_register_pll(struct lgm_clk_provider *ctx, 
	pll->hw.init = &init;



	hw = &pll->hw;

	ret = clk_hw_register(dev, hw);

	ret = devm_clk_hw_register(dev, hw);

	if (ret)

		return ERR_PTR(ret);

drivers/clk/x86/clk-cgu.c

+4 −4
Original line number Diff line number Diff line
@@ -119,7 +119,7 @@ lgm_clk_register_mux(struct lgm_clk_provider *ctx, 
	mux->hw.init = &init;



	hw = &mux->hw;

	ret = clk_hw_register(dev, hw);

	ret = devm_clk_hw_register(dev, hw);

	if (ret)

		return ERR_PTR(ret);
@@ -247,7 +247,7 @@ lgm_clk_register_divider(struct lgm_clk_provider *ctx, 
	div->hw.init = &init;



	hw = &div->hw;

	ret = clk_hw_register(dev, hw);

	ret = devm_clk_hw_register(dev, hw);

	if (ret)

		return ERR_PTR(ret);
@@ -361,7 +361,7 @@ lgm_clk_register_gate(struct lgm_clk_provider *ctx, 
	gate->hw.init = &init;



	hw = &gate->hw;

	ret = clk_hw_register(dev, hw);

	ret = devm_clk_hw_register(dev, hw);

	if (ret)

		return ERR_PTR(ret);
@@ -624,7 +624,7 @@ int lgm_clk_register_ddiv(struct lgm_clk_provider *ctx, 
		ddiv->hw.init = &init;



		hw = &ddiv->hw;

		ret = clk_hw_register(dev, hw);

		ret = devm_clk_hw_register(dev, hw);

		if (ret) {

			dev_err(dev, "register clk: %s failed!\n", list->name);

			return ret;

CRA Git | Maintained and supported by SUSTech CRA and CCSE