diff options
| author | Arkadiusz Hiler <arkadiusz.hiler@intel.com> | 2017-06-21 10:15:41 +0300 |
|---|---|---|
| committer | Arkadiusz Hiler <arkadiusz.hiler@intel.com> | 2017-06-21 10:15:41 +0300 |
| commit | d517ee729ddbe2bb8a5c6cc10e7ce35d73a8bd82 (patch) | |
| tree | a4485997c81ec296a7ed4254be33645669b4c6cd | |
| parent | fea6f86a0ac977e2b9da8b72aabbda8e3ef89aec (diff) | |
Revert "igt/pm_rps: Remove remaining assert on CUR <= MAX"
This reverts commit ae8d240b4d742a51f99e7fbe428392041aec160b.
This constrain still should be in effect, as noted by Michal Winiarski.
There's no client boost in the scenario when the check happens, so CUR
freq should never exceed the MAX.
Cc: Jeff Mcgee <jeff.mcgee@intel.com>
Cc: Radoslaw Szwichtenberg <radoslaw.szwichtenberg@intel.com>
Cc: Michal Winiarski <michal.winiarski@intel.com>
| -rw-r--r-- | tests/pm_rps.c | 1 |
1 files changed, 1 insertions, 0 deletions
diff --git a/tests/pm_rps.c b/tests/pm_rps.c index 1a322459..f0455e78 100644 --- a/tests/pm_rps.c +++ b/tests/pm_rps.c @@ -145,6 +145,7 @@ static int do_writeval(FILE *filp, int val, int lerrno, bool readback_check) static void checkit(const int *freqs) { igt_assert_lte(freqs[MIN], freqs[MAX]); + igt_assert_lte(freqs[CUR], freqs[MAX]); igt_assert_lte(freqs[RPn], freqs[CUR]); igt_assert_lte(freqs[RPn], freqs[MIN]); igt_assert_lte(freqs[MAX], freqs[RP0]); |