diff options
author | Jinyoung Park <jinyoungp@nvidia.com> | 2014-02-05 13:08:46 +0900 |
---|---|---|
committer | Harry Hong <hhong@nvidia.com> | 2014-02-24 16:28:35 -0800 |
commit | 841c79d1344f9e1496c1f5d73bd648ff78b3b838 (patch) | |
tree | 082a3f0d70ba9e2ae9553a37e8e8c359fbb822f4 /drivers | |
parent | 9f0a392e1710ec6e76a68e91a83338f4a9f75a2c (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.c | 9 |
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; |