thermal: cpu_cooling: standardize end of function
authorEduardo Valentin <eduardo.valentin@ti.com>
Wed, 17 Apr 2013 17:11:59 +0000 (17:11 +0000)
committerZhang Rui <rui.zhang@intel.com>
Sat, 27 Apr 2013 01:25:32 +0000 (09:25 +0800)
Just for code readiness, this patch makes all functions
on this file to have a blank line before their returns.
Now, some functions follow this pattern, and others will
not have a blank line. So, this patch makes it a single
pattern.

Signed-off-by: Eduardo Valentin <eduardo.valentin@ti.com>
Signed-off-by: Zhang Rui <rui.zhang@intel.com>
drivers/thermal/cpu_cooling.c

index 4b9be532cb14a8abf6cf525b4cda0bde64b4bb06..bb32ca0056c4c369bfaa4cdb6df0b70470e8c6b2 100644 (file)
@@ -64,6 +64,11 @@ static struct cpufreq_cooling_device *notify_device;
  * get_idr - function to get a unique id.
  * @idr: struct idr * handle used to create a id.
  * @id: int * value generated by this function.
+ *
+ * This function will populate @id with an unique
+ * id, using the idr API.
+ *
+ * Return: 0 on success, an error code on failure.
  */
 static int get_idr(struct idr *idr, int *id)
 {
@@ -75,6 +80,7 @@ static int get_idr(struct idr *idr, int *id)
        if (unlikely(ret < 0))
                return ret;
        *id = ret;
+
        return 0;
 }
 
@@ -105,6 +111,7 @@ static void release_idr(struct idr *idr, int id)
 static int is_cpufreq_valid(int cpu)
 {
        struct cpufreq_policy policy;
+
        return !cpufreq_get_policy(&policy, cpu);
 }
 
@@ -200,6 +207,7 @@ static int get_property(unsigned int cpu, unsigned long input,
                }
                j++;
        }
+
        return -EINVAL;
 }
 
@@ -220,6 +228,7 @@ unsigned long cpufreq_cooling_get_level(unsigned int cpu, unsigned int freq)
 
        if (get_property(cpu, (unsigned long)freq, &val, GET_LEVEL))
                return THERMAL_CSTATE_INVALID;
+
        return (unsigned long)val;
 }
 EXPORT_SYMBOL_GPL(cpufreq_cooling_get_level);
@@ -244,6 +253,7 @@ static unsigned int get_cpu_frequency(unsigned int cpu, unsigned long level)
        ret = get_property(cpu, level, &freq, GET_FREQ);
        if (ret)
                return 0;
+
        return freq;
 }
 
@@ -372,6 +382,7 @@ static int cpufreq_get_cur_state(struct thermal_cooling_device *cdev,
        struct cpufreq_cooling_device *cpufreq_device = cdev->devdata;
 
        *state = cpufreq_device->cpufreq_state;
+
        return 0;
 }
 
@@ -474,6 +485,7 @@ struct thermal_cooling_device *cpufreq_cooling_register(
        cpufreq_dev_count++;
 
        mutex_unlock(&cooling_cpufreq_lock);
+
        return cool_dev;
 }
 EXPORT_SYMBOL_GPL(cpufreq_cooling_register);