diff options
author | Jeff Garzik <jeff@garzik.org> | 2006-10-20 17:31:00 -0400 |
---|---|---|
committer | Dave Jones <davej@redhat.com> | 2006-10-21 01:33:12 -0400 |
commit | 914f7c31b0bea0ccf3bf474d0b99d803f7985097 (patch) | |
tree | 06c8ff4bf0f12170904bf681ffaedd14f368a7c8 /drivers/cpufreq/cpufreq_conservative.c | |
parent | 95dd722700dc9bbb000d51cab07dde48720e9178 (diff) |
[CPUFREQ] handle sysfs errors
Signed-off-by: Jeff Garzik <jeff@garzik.org>
Signed-off-by: Andrew Morton <akpm@osdl.org>
Signed-off-by: Dave Jones <davej@redhat.com>
Diffstat (limited to 'drivers/cpufreq/cpufreq_conservative.c')
-rw-r--r-- | drivers/cpufreq/cpufreq_conservative.c | 10 |
1 files changed, 9 insertions, 1 deletions
diff --git a/drivers/cpufreq/cpufreq_conservative.c b/drivers/cpufreq/cpufreq_conservative.c index c4c578defabf..8fe13ec1e50b 100644 --- a/drivers/cpufreq/cpufreq_conservative.c +++ b/drivers/cpufreq/cpufreq_conservative.c | |||
@@ -453,6 +453,7 @@ static int cpufreq_governor_dbs(struct cpufreq_policy *policy, | |||
453 | unsigned int cpu = policy->cpu; | 453 | unsigned int cpu = policy->cpu; |
454 | struct cpu_dbs_info_s *this_dbs_info; | 454 | struct cpu_dbs_info_s *this_dbs_info; |
455 | unsigned int j; | 455 | unsigned int j; |
456 | int rc; | ||
456 | 457 | ||
457 | this_dbs_info = &per_cpu(cpu_dbs_info, cpu); | 458 | this_dbs_info = &per_cpu(cpu_dbs_info, cpu); |
458 | 459 | ||
@@ -469,6 +470,13 @@ static int cpufreq_governor_dbs(struct cpufreq_policy *policy, | |||
469 | break; | 470 | break; |
470 | 471 | ||
471 | mutex_lock(&dbs_mutex); | 472 | mutex_lock(&dbs_mutex); |
473 | |||
474 | rc = sysfs_create_group(&policy->kobj, &dbs_attr_group); | ||
475 | if (rc) { | ||
476 | mutex_unlock(&dbs_mutex); | ||
477 | return rc; | ||
478 | } | ||
479 | |||
472 | for_each_cpu_mask(j, policy->cpus) { | 480 | for_each_cpu_mask(j, policy->cpus) { |
473 | struct cpu_dbs_info_s *j_dbs_info; | 481 | struct cpu_dbs_info_s *j_dbs_info; |
474 | j_dbs_info = &per_cpu(cpu_dbs_info, j); | 482 | j_dbs_info = &per_cpu(cpu_dbs_info, j); |
@@ -481,7 +489,7 @@ static int cpufreq_governor_dbs(struct cpufreq_policy *policy, | |||
481 | this_dbs_info->enable = 1; | 489 | this_dbs_info->enable = 1; |
482 | this_dbs_info->down_skip = 0; | 490 | this_dbs_info->down_skip = 0; |
483 | this_dbs_info->requested_freq = policy->cur; | 491 | this_dbs_info->requested_freq = policy->cur; |
484 | sysfs_create_group(&policy->kobj, &dbs_attr_group); | 492 | |
485 | dbs_enable++; | 493 | dbs_enable++; |
486 | /* | 494 | /* |
487 | * Start the timerschedule work, when this governor | 495 | * Start the timerschedule work, when this governor |