kernel.git/drivers/thermal/cpu_cooling.c, branch linux-3.11.yHosts the 0x221E linux distro kernel.
https://git.0xinfinity.dev/distro/kernel.git/atom?h=linux-3.11.y2013-05-28T06:49:31Zthermal: cpu_cooling: fix 'descend' check in get_property()2013-05-28T06:49:31ZShawn Guo2013-05-28T06:22:32Zurn:sha1:24c7a381720843f17efb42de81f7e85aefd6f616
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 <shawn.guo@linaro.org>
Signed-off-by: Zhang Rui <rui.zhang@intel.com>
thermal: cpu_cooling: alignment improvements2013-04-27T01:25:55ZEduardo Valentin2013-04-17T17:12:11Zurn:sha1:5fda7f680a44c65ce44fbe37f254b8a82f49d241
Improve code readiness by changing alignments so that
they match open parenthesis, like checkpatch.pl --strict
suggests.
Signed-off-by: Eduardo Valentin <eduardo.valentin@ti.com>
Signed-off-by: Zhang Rui <rui.zhang@intel.com>
thermal: cpu_cooling: remove checkpatch.pl warning2013-04-27T01:25:51ZEduardo Valentin2013-04-17T17:12:06Zurn:sha1:ef5e2124ec3d54182bd826411d864e0e28fc00d4
Simple code style fix.
Signed-off-by: Eduardo Valentin <eduardo.valentin@ti.com>
Signed-off-by: Zhang Rui <rui.zhang@intel.com>
thermal: cpu_cooling: remove trailing blank line2013-04-27T01:25:47ZEduardo Valentin2013-04-17T17:12:04Zurn:sha1:e45a6430025811c9ffc8932389adf5912f8ba4fd
Remove unnecessary blank line.
Signed-off-by: Eduardo Valentin <eduardo.valentin@ti.com>
Signed-off-by: Zhang Rui <rui.zhang@intel.com>
thermal: cpu_cooling: align on open parenthesis2013-04-27T01:25:43ZEduardo Valentin2013-04-17T17:12:03Zurn:sha1:d8892a01b13c8ea3ad74fb0c56beb96e92a2c65e
Improve code readiness by remove checkpatch.pl warnings
on get_property function.
Signed-off-by: Eduardo Valentin <eduardo.valentin@ti.com>
Signed-off-by: Zhang Rui <rui.zhang@intel.com>
thermal: cpu_cooling: standardize comment style2013-04-27T01:25:38ZEduardo Valentin2013-04-17T17:12:02Zurn:sha1:1b9e35265903c2e0e484dc224e8f7de506e3353f
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 <eduardo.valentin@ti.com>
Signed-off-by: Zhang Rui <rui.zhang@intel.com>
thermal: cpu_cooling: standardize end of function2013-04-27T01:25:32ZEduardo Valentin2013-04-17T17:11:59Zurn:sha1:79491e53dcd73175473e3293a574f5e006b468be
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>
thermal: cpu_cooling: remove trailing white spaces2013-04-27T01:25:24ZEduardo Valentin2013-04-17T17:11:55Zurn:sha1:79d264016ac011b74497b553022d2fc45bf9d9f2
Remove unnecessary white spaces.
Signed-off-by: Eduardo Valentin <eduardo.valentin@ti.com>
Signed-off-by: Zhang Rui <rui.zhang@intel.com>
thermal: cpu_cooling: remove unused symbols2013-04-24T16:34:44ZEduardo Valentin2013-04-17T17:12:19Zurn:sha1:67d0b2a8267caa4e653714bec79013fa6b7dd508
The list is not needed so far. Thus removing it.
Signed-off-by: Eduardo Valentin <eduardo.valentin@ti.com>
Acked-by: Amit Daniel Kachhap <amit.daniel@samsung.com>
Signed-off-by: Zhang Rui <rui.zhang@intel.com>
thermal: cpu_cooling: remove not needed curl brackets2013-04-24T16:34:43ZEduardo Valentin2013-04-17T17:12:18Zurn:sha1:2d9bf71c12be9caadd9a8276a78f3a3df8e9e852
Just for style purposes, remove extra curl brackets.
Signed-off-by: Eduardo Valentin <eduardo.valentin@ti.com>
Acked-by: Amit Daniel Kachhap <amit.daniel@samsung.com>
Signed-off-by: Zhang Rui <rui.zhang@intel.com>