summaryrefslogtreecommitdiff
path: root/drivers
diff options
context:
space:
mode:
authorJinyoung Park <jinyoungp@nvidia.com>2014-02-05 13:08:46 +0900
committerHarry Hong <hhong@nvidia.com>2014-02-24 16:28:35 -0800
commit841c79d1344f9e1496c1f5d73bd648ff78b3b838 (patch)
tree082a3f0d70ba9e2ae9553a37e8e8c359fbb822f4 /drivers
parent9f0a392e1710ec6e76a68e91a83338f4a9f75a2c (diff)
cpufreq: conservative: Move sysfs_remove_group
Move sysfs_remove_group into mutex lock protection. Bug 1449493 Change-Id: I65976f91f9acceeaca7ebe897f41f46be198b28e Signed-off-by: Jinyoung Park <jinyoungp@nvidia.com> Reviewed-on: http://git-master/r/373379 GVS: Gerrit_Virtual_Submit Reviewed-by: David Yu (Engrg-SW) <davyu@nvidia.com> Tested-by: David Yu (Engrg-SW) <davyu@nvidia.com> Reviewed-by: Harry Hong <hhong@nvidia.com>
Diffstat (limited to 'drivers')
-rw-r--r--drivers/cpufreq/cpufreq_conservative.c9
1 files changed, 4 insertions, 5 deletions
diff --git a/drivers/cpufreq/cpufreq_conservative.c b/drivers/cpufreq/cpufreq_conservative.c
index 7ae376e70adc..38cba68fb8f2 100644
--- a/drivers/cpufreq/cpufreq_conservative.c
+++ b/drivers/cpufreq/cpufreq_conservative.c
@@ -571,15 +571,14 @@ static int cpufreq_governor_dbs(struct cpufreq_policy *policy,
* Stop the timerschedule work, when this governor
* is used for first time
*/
- if (dbs_enable == 0)
+ if (dbs_enable == 0) {
cpufreq_unregister_notifier(
&dbs_cpufreq_notifier_block,
CPUFREQ_TRANSITION_NOTIFIER);
-
- mutex_unlock(&dbs_mutex);
- if (!dbs_enable)
sysfs_remove_group(cpufreq_global_kobject,
- &dbs_attr_group);
+ &dbs_attr_group);
+ }
+ mutex_unlock(&dbs_mutex);
break;