clk: samsung: Add local variable to match its purpose
authorMarek Szyprowski <m.szyprowski@samsung.com>
Mon, 24 Apr 2017 07:54:12 +0000 (09:54 +0200)
committerSylwester Nawrocki <s.nawrocki@samsung.com>
Wed, 7 Jun 2017 20:47:56 +0000 (22:47 +0200)
Add new variable to avoid using clk pointer for different purposes across
the exynos_register_cpu_clock() function. This will help in future rewrite
for the new clk_hw API.

Signed-off-by: Marek Szyprowski <m.szyprowski@samsung.com>
Reviewed-by: Krzysztof Kozlowski <krzk@kernel.org>
Signed-off-by: Sylwester Nawrocki <s.nawrocki@samsung.com>
drivers/clk/samsung/clk-cpu.c

index 8bf7e80..c6dd83d 100644 (file)
@@ -410,6 +410,7 @@ int __init exynos_register_cpu_clock(struct samsung_clk_provider *ctx,
 {
        struct exynos_cpuclk *cpuclk;
        struct clk_init_data init;
+       struct clk *parent_clk;
        struct clk *clk;
        int ret = 0;
 
@@ -440,15 +441,15 @@ int __init exynos_register_cpu_clock(struct samsung_clk_provider *ctx,
                goto free_cpuclk;
        }
 
-       clk = __clk_lookup(parent);
-       if (!clk) {
+       parent_clk = __clk_lookup(parent);
+       if (!parent_clk) {
                pr_err("%s: could not lookup parent clock %s\n",
                                __func__, parent);
                ret = -EINVAL;
                goto free_cpuclk;
        }
 
-       ret = clk_notifier_register(clk, &cpuclk->clk_nb);
+       ret = clk_notifier_register(parent_clk, &cpuclk->clk_nb);
        if (ret) {
                pr_err("%s: failed to register clock notifier for %s\n",
                                __func__, name);
@@ -476,7 +477,7 @@ int __init exynos_register_cpu_clock(struct samsung_clk_provider *ctx,
 free_cpuclk_data:
        kfree(cpuclk->cfg);
 unregister_clk_nb:
-       clk_notifier_unregister(__clk_lookup(parent), &cpuclk->clk_nb);
+       clk_notifier_unregister(parent_clk, &cpuclk->clk_nb);
 free_cpuclk:
        kfree(cpuclk);
        return ret;