Commit 79491e53 authored by Eduardo Valentin's avatar Eduardo Valentin Committed by Zhang Rui

thermal: cpu_cooling: standardize end of function

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: default avatarEduardo Valentin <eduardo.valentin@ti.com>
Signed-off-by: default avatarZhang Rui <rui.zhang@intel.com>
parent 79d26401
...@@ -64,6 +64,11 @@ static struct cpufreq_cooling_device *notify_device; ...@@ -64,6 +64,11 @@ static struct cpufreq_cooling_device *notify_device;
* get_idr - function to get a unique id. * get_idr - function to get a unique id.
* @idr: struct idr * handle used to create a id. * @idr: struct idr * handle used to create a id.
* @id: int * value generated by this function. * @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) static int get_idr(struct idr *idr, int *id)
{ {
...@@ -75,6 +80,7 @@ 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)) if (unlikely(ret < 0))
return ret; return ret;
*id = ret; *id = ret;
return 0; return 0;
} }
...@@ -105,6 +111,7 @@ static void release_idr(struct idr *idr, int id) ...@@ -105,6 +111,7 @@ static void release_idr(struct idr *idr, int id)
static int is_cpufreq_valid(int cpu) static int is_cpufreq_valid(int cpu)
{ {
struct cpufreq_policy policy; struct cpufreq_policy policy;
return !cpufreq_get_policy(&policy, cpu); return !cpufreq_get_policy(&policy, cpu);
} }
...@@ -200,6 +207,7 @@ static int get_property(unsigned int cpu, unsigned long input, ...@@ -200,6 +207,7 @@ static int get_property(unsigned int cpu, unsigned long input,
} }
j++; j++;
} }
return -EINVAL; return -EINVAL;
} }
...@@ -220,6 +228,7 @@ unsigned long cpufreq_cooling_get_level(unsigned int cpu, unsigned int freq) ...@@ -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)) if (get_property(cpu, (unsigned long)freq, &val, GET_LEVEL))
return THERMAL_CSTATE_INVALID; return THERMAL_CSTATE_INVALID;
return (unsigned long)val; return (unsigned long)val;
} }
EXPORT_SYMBOL_GPL(cpufreq_cooling_get_level); EXPORT_SYMBOL_GPL(cpufreq_cooling_get_level);
...@@ -244,6 +253,7 @@ static unsigned int get_cpu_frequency(unsigned int cpu, unsigned long 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); ret = get_property(cpu, level, &freq, GET_FREQ);
if (ret) if (ret)
return 0; return 0;
return freq; return freq;
} }
...@@ -372,6 +382,7 @@ static int cpufreq_get_cur_state(struct thermal_cooling_device *cdev, ...@@ -372,6 +382,7 @@ static int cpufreq_get_cur_state(struct thermal_cooling_device *cdev,
struct cpufreq_cooling_device *cpufreq_device = cdev->devdata; struct cpufreq_cooling_device *cpufreq_device = cdev->devdata;
*state = cpufreq_device->cpufreq_state; *state = cpufreq_device->cpufreq_state;
return 0; return 0;
} }
...@@ -474,6 +485,7 @@ struct thermal_cooling_device *cpufreq_cooling_register( ...@@ -474,6 +485,7 @@ struct thermal_cooling_device *cpufreq_cooling_register(
cpufreq_dev_count++; cpufreq_dev_count++;
mutex_unlock(&cooling_cpufreq_lock); mutex_unlock(&cooling_cpufreq_lock);
return cool_dev; return cool_dev;
} }
EXPORT_SYMBOL_GPL(cpufreq_cooling_register); EXPORT_SYMBOL_GPL(cpufreq_cooling_register);
......
Markdown is supported
0%
or
You are about to add 0 people to the discussion. Proceed with caution.
Finish editing this message first!
Please register or to comment