diff options
| author | Jeff McGee <jeff.mcgee@intel.com> | 2014-01-10 15:12:32 -0600 |
|---|---|---|
| committer | Daniel Vetter <daniel.vetter@ffwll.ch> | 2014-01-10 22:31:51 +0100 |
| commit | d0b5fd2bff7ce8ec1ada15da6ca9ed2425cbf1c5 (patch) | |
| tree | d2a9b1f4f1a97535020eb77b31a45dbb9548d900 /tests | |
| parent | 44b5d7b3300dd7c94d09c57c95ab3a407f8b17ef (diff) | |
pm_rps: Fix test to target original min and max
The goal of the test is to confirm that gt_cur_freq_mhz can be forced
to the boundaries of the frequency range by collapsing gt_min_freq_mhz
and gt_max_freq_mhz to the target value. But we miss testing the upper
end of the range by targetting the current value of max after it has
been set equal to min. So fix by targetting orginal max instead of
current max.
This correction exposes a problem in setfreq where min is always set
to target before max, which should fail if the target value is greater
than max. So fix that too.
Signed-off-by: Jeff McGee <jeff.mcgee@intel.com>
Signed-off-by: Daniel Vetter <daniel.vetter@ffwll.ch>
Diffstat (limited to 'tests')
| -rw-r--r-- | tests/pm_rps.c | 13 |
1 files changed, 9 insertions, 4 deletions
diff --git a/tests/pm_rps.c b/tests/pm_rps.c index 69eeb814..c968ecb9 100644 --- a/tests/pm_rps.c +++ b/tests/pm_rps.c @@ -101,8 +101,13 @@ static int do_writeval(FILE *filp, int val, int lerrno) static void setfreq(int val) { - writeval(stuff[MIN].filp, val); - writeval(stuff[MAX].filp, val); + if (val > fmax) { + writeval(stuff[MAX].filp, val); + writeval(stuff[MIN].filp, val); + } else { + writeval(stuff[MIN].filp, val); + writeval(stuff[MAX].filp, val); + } } static void checkit(void) @@ -166,11 +171,11 @@ igt_simple_main dumpit(); checkit(); - setfreq(fmin); + setfreq(origmin); if (verbose) dumpit(); restore_assert(fcur == fmin); - setfreq(fmax); + setfreq(origmax); if (verbose) dumpit(); restore_assert(fcur == fmax); |