d/freebsd-nq
1
0
Fork 0
Code Issues Pull Requests Packages Projects Releases Wiki Activity

Correct an off-by-one error in the number of settings est announces.

Browse Source 
The extraneous "0" state was not fatal but useless.
This commit is contained in:
Nate Lawson 2005-02-24 20:20:11 +00:00
parent eaed22643f
commit 0c5897722c
1 changed files with 3 additions and 4 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

7
sys/i386/cpufreq/est.c
View File

@ -717,15 +717,14 @@ est_settings(device_t dev, struct cf_setting *sets, int *count)
return (E2BIG);
i = 0;
for (f = sc->freq_list; f->freq != 0; f++) {
for (f = sc->freq_list; f->freq != 0; f++, i++) {
sets[i].freq = f->freq;
sets[i].volts = f->volts;
sets[i].power = CPUFREQ_VAL_UNKNOWN;
sets[i].lat = EST_TRANS_LAT;
sets[i].dev = dev;
i++;
}
*count = i + 1;
*count = i;
return (0);
}
@ -751,7 +750,7 @@ est_set(device_t dev, const struct cf_setting *set)
msr = (msr & ~0xffff) | f->id16;
wrmsr(MSR_PERF_CTL, msr);
/* Wait a short while for the new setting. Is this necessary? */
/* Wait a short while for the new setting. XXX Is this necessary? */
DELAY(EST_TRANS_LAT);
return (0);