summaryrefslogtreecommitdiffstats
path: root/drivers/cpufreq
diff options
context:
space:
mode:
authorDave Jones <davej@redhat.com>2006-04-01 22:07:07 -0500
committerDave Jones <davej@redhat.com>2006-04-01 22:07:07 -0500
commitb82fbe6c4232365272bde6f2c3f8fd9dd4dcd73a (patch)
tree230b1b0c7de857c065c5ff5839ff986213329d96 /drivers/cpufreq
parent87c32271380e630955de365656f67b0a54b75b19 (diff)
downloadop-kernel-dev-b82fbe6c4232365272bde6f2c3f8fd9dd4dcd73a.zip
op-kernel-dev-b82fbe6c4232365272bde6f2c3f8fd9dd4dcd73a.tar.gz
[CPUFREQ] Remove pointless check in conservative governor.
< 0 checks on unsigned variables are pointless. Signed-off-by: Dave Jones <davej@redhat.com>
Diffstat (limited to 'drivers/cpufreq')
-rw-r--r--drivers/cpufreq/cpufreq_conservative.c6
1 files changed, 2 insertions, 4 deletions
diff --git a/drivers/cpufreq/cpufreq_conservative.c b/drivers/cpufreq/cpufreq_conservative.c
index 037f6bf..e07a354 100644
--- a/drivers/cpufreq/cpufreq_conservative.c
+++ b/drivers/cpufreq/cpufreq_conservative.c
@@ -176,8 +176,7 @@ static ssize_t store_up_threshold(struct cpufreq_policy *unused,
ret = sscanf (buf, "%u", &input);
mutex_lock(&dbs_mutex);
- if (ret != 1 || input > 100 || input < 0 ||
- input <= dbs_tuners_ins.down_threshold) {
+ if (ret != 1 || input > 100 || input <= dbs_tuners_ins.down_threshold) {
mutex_unlock(&dbs_mutex);
return -EINVAL;
}
@@ -196,8 +195,7 @@ static ssize_t store_down_threshold(struct cpufreq_policy *unused,
ret = sscanf (buf, "%u", &input);
mutex_lock(&dbs_mutex);
- if (ret != 1 || input > 100 || input < 0 ||
- input >= dbs_tuners_ins.up_threshold) {
+ if (ret != 1 || input > 100 || input >= dbs_tuners_ins.up_threshold) {
mutex_unlock(&dbs_mutex);
return -EINVAL;
}
OpenPOWER on IntegriCloud