On Mon, May 26, 2014 at 11:28 AM, Viresh Kumar [off-list ref] wrote:

On 23 May 2014 19:07, Thomas Abraham [off-list ref] wrote:

quoted

From: Thomas Abraham <redacted>

Commit 6f19efc0 ("cpufreq: Add boost frequency support in core") adds
support for CPU boost mode. This patch adds support for finding available
boost frequencies from device tree and marking them as usable in boost mode.

Cc: Nishanth Menon <nm@ti.com>
Cc: Lukasz Majewski <redacted>
Signed-off-by: Thomas Abraham <redacted>
---
 drivers/cpufreq/cpufreq_opp.c |   44 +++++++++++++++++++++++++++++++++++++++++
 1 file changed, 44 insertions(+)

diff --git a/drivers/cpufreq/cpufreq_opp.c b/drivers/cpufreq/cpufreq_opp.c
index c0c6f4a..2b3905b 100644
--- a/drivers/cpufreq/cpufreq_opp.c
+++ b/drivers/cpufreq/cpufreq_opp.c

@@ -19,6 +19,7 @@
 #include <linux/pm_opp.h>
 #include <linux/rcupdate.h>
 #include <linux/slab.h>
+#include <linux/of.h>

 /**
  * dev_pm_opp_init_cpufreq_table() - create a cpufreq table for a device

@@ -51,6 +52,10 @@ int dev_pm_opp_init_cpufreq_table(struct device *dev,
        struct cpufreq_frequency_table *freq_table = NULL;
        int i, max_opps, ret = 0;
        unsigned long rate;
+#ifdef CONFIG_CPU_FREQ_BOOST_SW
+       int j, len;
+       u32 *boost_freqs = NULL;
+#endif

        rcu_read_lock();

@@ -82,6 +87,45 @@ int dev_pm_opp_init_cpufreq_table(struct device *dev,

        *table = &freq_table[0];

+#ifdef CONFIG_CPU_FREQ_BOOST_SW
+       if (of_find_property(dev->of_node, "boost-frequencies", &len)) {

Maybe:
if(!of_find_property(...))
        goto out;

To get rid of extra indentation levels below..

quoted

+               struct cpufreq_frequency_table *ft;

Declare at the top with boost_freqs, etc..

quoted

+               if (len == 0 || (len & (sizeof(u32) - 1)) != 0) {

s/len == 0/!len

And use IS_ALIGNED() instead of the right hand side of ||

quoted

+                       dev_err(dev, "%s: invalid boost frequency\n", __func__);
+                       ret = -EINVAL;
+                       goto out;
+               }
+
+               boost_freqs = kzalloc(len, GFP_KERNEL);

Can we do a devm_kzalloc instead? And why not kmalloc BTW ?

This is a temporary storage that is freed at the end of this function.
So devm_kzalloc is not used. Yes, kmalloc can be used.

quoted

+               if (!boost_freqs) {
+                       dev_warn(dev, "%s: no memory for boost freq table\n",

dev_err ?

quoted

+                                       __func__);
+                       ret = -ENOMEM;
+                       goto out;
+               }
+               of_property_read_u32_array(dev->of_node, "boost-frequencies",
+                       boost_freqs, len / sizeof(u32));

Create int count = len / sizeof(u32) instead.. You have used this
multiple times.

quoted

+               for (j = 0; j < len / sizeof(u32); j++) {
+                       ft = *table;
+                       for (i = 0; ft->frequency != CPUFREQ_TABLE_END; i++) {

See if new macros can be used here instead. cpufreq_for_each_valid_entry().

quoted

+                               if (boost_freqs[j] == ft->frequency) {
+                                       ft->flags |= CPUFREQ_BOOST_FREQ;
+                                       break;
+                               }
+                               ft++;
+                       }
+
+                       if (ft->frequency == CPUFREQ_TABLE_END)
+                               pr_err("%s: invalid boost frequency %d\n",
+                                       __func__, boost_freqs[j]);

Maybe a pr_debug on the else part as well ? With boost freqs ..

quoted

+               }
+       }
+
+       kfree(boost_freqs);
+#endif
+
 out:
        rcu_read_unlock();
        if (ret)
--
1.7.9.5

--
To unsubscribe from this list: send the line "unsubscribe linux-samsung-soc" in
the body of a message to majordomo@vger.kernel.org
More majordomo info at  http://vger.kernel.org/majordomo-info.html