linux.git/drivers/thermal/cpu_cooling.c, branch v3.12

drivers: thermal: add check when unregistering cpu cooling

2013-09-03T13:10:34+00:00

This patch avoids NULL pointer accesses while unregistering
cpu cooling devices, in case a NULL pointer is received.

Cc: Zhang Rui 
Cc: linux-pm@vger.kernel.org
Cc: linux-kernel@vger.kernel.org
Signed-off-by: Eduardo Valentin

Thermal/cpu_cooling: Return directly for the cpu out of allowed_cpus in the cpufreq_thermal_notifier()

2013-08-15T07:25:08+00:00

cpufreq_thermal_notifier() is to change the cpu's cpufreq in the allowed_cpus mask
when associated thermal-cpufreq cdev's cooling state is changed. It's a cpufreq policy
notifier handler and it will be triggered even if those cpus out of allowed_cpus has
changed freq policy.

cpufreq_thermal_notifier() checks the policy->cpu. If it belongs to allowed_cpus,
change max_freq(default to 0) to the desire cpufreq value and pass 0 and max_freq
to cpufreq_verify_within_limits() as cpufreq scope. But if not, do nothing and
max_freq will remain 0. This will cause the cpufreq scope to become 0~0. This
is not right. This patch is to return directly after finding cpu not belonging
to allowed_cpus.

Signed-off-by: Lan Tianyu 
Signed-off-by: Zhang Rui

thermal: cpu_cooling: fix 'descend' check in get_property()

2013-05-28T06:49:31+00:00

The variable 'descend' is initialized as -1 in function get_property(),
and will never get any chance to be updated by the following code.

	if (freq != CPUFREQ_ENTRY_INVALID && descend != -1)
		descend = !!(freq > table[i].frequency);

This makes function get_property() return the wrong frequency for given
cooling level if the frequency table is sorted in ascending.  Fix it
by correcting the 'descend' check in if-condition to 'descend == -1'.

Signed-off-by: Shawn Guo 
Signed-off-by: Zhang Rui

thermal: cpu_cooling: alignment improvements

2013-04-27T01:25:55+00:00

Improve code readiness by changing alignments so that
they match open parenthesis, like checkpatch.pl --strict
suggests.

Signed-off-by: Eduardo Valentin 
Signed-off-by: Zhang Rui

thermal: cpu_cooling: remove checkpatch.pl warning

2013-04-27T01:25:51+00:00

Simple code style fix.

Signed-off-by: Eduardo Valentin 
Signed-off-by: Zhang Rui

thermal: cpu_cooling: remove trailing blank line

2013-04-27T01:25:47+00:00

Remove unnecessary blank line.

Signed-off-by: Eduardo Valentin 
Signed-off-by: Zhang Rui

thermal: cpu_cooling: align on open parenthesis

2013-04-27T01:25:43+00:00

Improve code readiness by remove checkpatch.pl warnings
on get_property function.

Signed-off-by: Eduardo Valentin 
Signed-off-by: Zhang Rui

thermal: cpu_cooling: standardize comment style

2013-04-27T01:25:38+00:00

There are at least three patterns for oneline comments in this
file. This patch changes them to one single pattern

Signed-off-by: Eduardo Valentin 
Signed-off-by: Zhang Rui

thermal: cpu_cooling: standardize end of function

2013-04-27T01:25:32+00:00

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 
Signed-off-by: Zhang Rui

thermal: cpu_cooling: remove trailing white spaces

2013-04-27T01:25:24+00:00

Remove unnecessary white spaces.

Signed-off-by: Eduardo Valentin 
Signed-off-by: Zhang Rui