CPUFREQ: Loongson1: Use devm_kzalloc() instead of global structure
authorKelvin Cheung <keguang.zhang@gmail.com>
Tue, 12 Apr 2016 10:40:18 +0000 (18:40 +0800)
committerRalf Baechle <ralf@linux-mips.org>
Fri, 13 May 2016 12:02:08 +0000 (14:02 +0200)
This patch uses devm_kzalloc() instead of global structure.

Signed-off-by: Kelvin Cheung <keguang.zhang@gmail.com>
Acked-by: Viresh Kumar <viresh.kumar@linaro.org>
Cc: Rafael J. Wysocki <rjw@rjwysocki.net>
Cc: linux-pm@vger.kernel.org
Cc: linux-mips@linux-mips.org
Cc: linux-kernel@vger.kernel.org
Patchwork: https://patchwork.linux-mips.org/patch/13055/
Signed-off-by: Ralf Baechle <ralf@linux-mips.org>
drivers/cpufreq/loongson1-cpufreq.c

index 1b63b4a3313370fea128acd39626231f78b47d0a..f0d0156109f507a0bd23c8d7f065f61ea6e7068f 100644 (file)
@@ -20,7 +20,7 @@
 #include <cpufreq.h>
 #include <loongson1.h>
 
-static struct {
+struct ls1x_cpufreq {
        struct device *dev;
        struct clk *clk;        /* CPU clk */
        struct clk *mux_clk;    /* MUX of CPU clk */
@@ -28,7 +28,9 @@ static struct {
        struct clk *osc_clk;    /* OSC clk */
        unsigned int max_freq;
        unsigned int min_freq;
-} ls1x_cpufreq;
+};
+
+static struct ls1x_cpufreq *cpufreq;
 
 static int ls1x_cpufreq_notifier(struct notifier_block *nb,
                                 unsigned long val, void *data)
@@ -46,6 +48,7 @@ static struct notifier_block ls1x_cpufreq_notifier_block = {
 static int ls1x_cpufreq_target(struct cpufreq_policy *policy,
                               unsigned int index)
 {
+       struct device *cpu_dev = get_cpu_device(policy->cpu);
        unsigned int old_freq, new_freq;
 
        old_freq = policy->cur;
@@ -60,25 +63,26 @@ static int ls1x_cpufreq_target(struct cpufreq_policy *policy,
         *  - Reparent CPU clk back to CPU DIV clk
         */
 
-       dev_dbg(ls1x_cpufreq.dev, "%u KHz --> %u KHz\n", old_freq, new_freq);
-       clk_set_parent(policy->clk, ls1x_cpufreq.osc_clk);
+       clk_set_parent(policy->clk, cpufreq->osc_clk);
        __raw_writel(__raw_readl(LS1X_CLK_PLL_DIV) | RST_CPU_EN | RST_CPU,
                     LS1X_CLK_PLL_DIV);
        __raw_writel(__raw_readl(LS1X_CLK_PLL_DIV) & ~(RST_CPU_EN | RST_CPU),
                     LS1X_CLK_PLL_DIV);
-       clk_set_rate(ls1x_cpufreq.mux_clk, new_freq * 1000);
-       clk_set_parent(policy->clk, ls1x_cpufreq.mux_clk);
+       clk_set_rate(cpufreq->mux_clk, new_freq * 1000);
+       clk_set_parent(policy->clk, cpufreq->mux_clk);
+       dev_dbg(cpu_dev, "%u KHz --> %u KHz\n", old_freq, new_freq);
 
        return 0;
 }
 
 static int ls1x_cpufreq_init(struct cpufreq_policy *policy)
 {
+       struct device *cpu_dev = get_cpu_device(policy->cpu);
        struct cpufreq_frequency_table *freq_tbl;
        unsigned int pll_freq, freq;
        int steps, i, ret;
 
-       pll_freq = clk_get_rate(ls1x_cpufreq.pll_clk) / 1000;
+       pll_freq = clk_get_rate(cpufreq->pll_clk) / 1000;
 
        steps = 1 << DIV_CPU_WIDTH;
        freq_tbl = kcalloc(steps, sizeof(*freq_tbl), GFP_KERNEL);
@@ -87,18 +91,17 @@ static int ls1x_cpufreq_init(struct cpufreq_policy *policy)
 
        for (i = 0; i < (steps - 1); i++) {
                freq = pll_freq / (i + 1);
-               if ((freq < ls1x_cpufreq.min_freq) ||
-                   (freq > ls1x_cpufreq.max_freq))
+               if ((freq < cpufreq->min_freq) || (freq > cpufreq->max_freq))
                        freq_tbl[i].frequency = CPUFREQ_ENTRY_INVALID;
                else
                        freq_tbl[i].frequency = freq;
-               dev_dbg(ls1x_cpufreq.dev,
+               dev_dbg(cpu_dev,
                        "cpufreq table: index %d: frequency %d\n", i,
                        freq_tbl[i].frequency);
        }
        freq_tbl[i].frequency = CPUFREQ_TABLE_END;
 
-       policy->clk = ls1x_cpufreq.clk;
+       policy->clk = cpufreq->clk;
        ret = cpufreq_generic_init(policy, freq_tbl, 0);
        if (ret)
                kfree(freq_tbl);
@@ -141,51 +144,56 @@ static int ls1x_cpufreq_probe(struct platform_device *pdev)
        if (!pdata || !pdata->clk_name || !pdata->osc_clk_name)
                return -EINVAL;
 
-       ls1x_cpufreq.dev = &pdev->dev;
+       cpufreq =
+           devm_kzalloc(&pdev->dev, sizeof(struct ls1x_cpufreq), GFP_KERNEL);
+       if (!cpufreq)
+               return -ENOMEM;
+
+       cpufreq->dev = &pdev->dev;
 
        clk = devm_clk_get(&pdev->dev, pdata->clk_name);
        if (IS_ERR(clk)) {
-               dev_err(ls1x_cpufreq.dev, "unable to get %s clock\n",
+               dev_err(&pdev->dev, "unable to get %s clock\n",
                        pdata->clk_name);
                ret = PTR_ERR(clk);
                goto out;
        }
-       ls1x_cpufreq.clk = clk;
+       cpufreq->clk = clk;
 
        clk = clk_get_parent(clk);
        if (IS_ERR(clk)) {
-               dev_err(ls1x_cpufreq.dev, "unable to get parent of %s clock\n",
-                       __clk_get_name(ls1x_cpufreq.clk));
+               dev_err(&pdev->dev, "unable to get parent of %s clock\n",
+                       __clk_get_name(cpufreq->clk));
                ret = PTR_ERR(clk);
                goto out;
        }
-       ls1x_cpufreq.mux_clk = clk;
+       cpufreq->mux_clk = clk;
 
        clk = clk_get_parent(clk);
        if (IS_ERR(clk)) {
-               dev_err(ls1x_cpufreq.dev, "unable to get parent of %s clock\n",
-                       __clk_get_name(ls1x_cpufreq.mux_clk));
+               dev_err(&pdev->dev, "unable to get parent of %s clock\n",
+                       __clk_get_name(cpufreq->mux_clk));
                ret = PTR_ERR(clk);
                goto out;
        }
-       ls1x_cpufreq.pll_clk = clk;
+       cpufreq->pll_clk = clk;
 
        clk = devm_clk_get(&pdev->dev, pdata->osc_clk_name);
        if (IS_ERR(clk)) {
-               dev_err(ls1x_cpufreq.dev, "unable to get %s clock\n",
+               dev_err(&pdev->dev, "unable to get %s clock\n",
                        pdata->osc_clk_name);
                ret = PTR_ERR(clk);
                goto out;
        }
-       ls1x_cpufreq.osc_clk = clk;
+       cpufreq->osc_clk = clk;
 
-       ls1x_cpufreq.max_freq = pdata->max_freq;
-       ls1x_cpufreq.min_freq = pdata->min_freq;
+       cpufreq->max_freq = pdata->max_freq;
+       cpufreq->min_freq = pdata->min_freq;
 
        ret = cpufreq_register_driver(&ls1x_cpufreq_driver);
        if (ret) {
-               dev_err(ls1x_cpufreq.dev,
-                       "failed to register cpufreq driver: %d\n", ret);
+               dev_err(&pdev->dev,
+                       "failed to register CPUFreq driver: %d\n", ret);
                goto out;
        }
 
@@ -195,8 +203,7 @@ static int ls1x_cpufreq_probe(struct platform_device *pdev)
        if (!ret)
                goto out;
 
-       dev_err(ls1x_cpufreq.dev, "failed to register cpufreq notifier: %d\n",
-               ret);
+       dev_err(&pdev->dev, "failed to register cpufreq notifier: %d\n", ret);
 
        cpufreq_unregister_driver(&ls1x_cpufreq_driver);
 out: