diff options
| author | Dave Jones <davej@redhat.com> | 2006-04-01 22:07:07 -0500 |
|---|---|---|
| committer | Dave Jones <davej@redhat.com> | 2006-04-01 22:07:07 -0500 |
| commit | b82fbe6c4232365272bde6f2c3f8fd9dd4dcd73a (patch) | |
| tree | 230b1b0c7de857c065c5ff5839ff986213329d96 /drivers/cpufreq | |
| parent | 87c32271380e630955de365656f67b0a54b75b19 (diff) | |
| download | talos-obmc-linux-b82fbe6c4232365272bde6f2c3f8fd9dd4dcd73a.tar.gz talos-obmc-linux-b82fbe6c4232365272bde6f2c3f8fd9dd4dcd73a.zip | |
[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.c | 6 |
1 files changed, 2 insertions, 4 deletions
diff --git a/drivers/cpufreq/cpufreq_conservative.c b/drivers/cpufreq/cpufreq_conservative.c index 037f6bf4543c..e07a35487bde 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; } |
