clk: intel: Use devm_clk_hw_register() instead of clk_hw_register()
authorRahul Tanwar <rahul.tanwar@linux.intel.com>
Thu, 16 Jul 2020 06:30:30 +0000 (14:30 +0800)
committerStephen Boyd <sboyd@kernel.org>
Fri, 24 Jul 2020 08:55:18 +0000 (01:55 -0700)
To ensure that clks are unregistered in case of any failure, use
devm_clk_hw_register() instead of clk_hw_register().

Signed-off-by: Rahul Tanwar <rahul.tanwar@linux.intel.com>
Link: https://lore.kernel.org/r/7ef7009b4e9f986fd6dfbf487c0e85de68a4ba9b.1594880946.git.rahul.tanwar@linux.intel.com
Signed-off-by: Stephen Boyd <sboyd@kernel.org>
drivers/clk/x86/clk-cgu-pll.c
drivers/clk/x86/clk-cgu.c

index c03cc6b85b9f1678e76a35f87c6b692c0c55a60a..3179557b5f784f5394bc196924251538a2cf601d 100644 (file)
@@ -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);
 
index 56af0e04ec1e32c5cc36969912318f216cc1ea7f..88ebeb53b10905ea463d7e6b528c954a0f2ce2ce 100644 (file)
@@ -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;