From 1fea6ce7dddbb08bde49c6206addbddab4cf029a Mon Sep 17 00:00:00 2001 From: Hajimu UMEMOTO Date: Thu, 18 Aug 2005 16:41:25 +0000 Subject: - don't forget to save freqency when priority is raised. - nuke redundant variable initialization. --- sys/kern/kern_cpu.c | 5 ++--- 1 file changed, 2 insertions(+), 3 deletions(-) diff --git a/sys/kern/kern_cpu.c b/sys/kern/kern_cpu.c index e75092f7b4588..4c22fb685a8fa 100644 --- a/sys/kern/kern_cpu.c +++ b/sys/kern/kern_cpu.c @@ -260,8 +260,7 @@ cf_set_method(device_t dev, const struct cf_level *level, int priority) if (CPUFREQ_CMP(sc->curr_level.total_set.freq, level->total_set.freq)) { CF_DEBUG("skipping freq %d, same as current level %d\n", level->total_set.freq, sc->curr_level.total_set.freq); - sc->curr_priority = priority; - goto out; + goto skip; } /* First, set the absolute frequency via its driver. */ @@ -323,6 +322,7 @@ cf_set_method(device_t dev, const struct cf_level *level, int priority) } } +skip: /* If we were restoring a saved state, reset it to "unused". */ if (level == &sc->saved_level) { CF_DEBUG("resetting saved level\n"); @@ -345,7 +345,6 @@ cf_set_method(device_t dev, const struct cf_level *level, int priority) } sc->curr_level = *level; sc->curr_priority = priority; - error = 0; out: CF_MTX_UNLOCK(&sc->lock); -- cgit v1.3