| Message ID | 1409869371-7558-1-git-send-email-rodrigo.vivi@intel.com (mailing list archive) |
|---|---|
| State | New, archived |
| Headers | show |
On Thu, Sep 04, 2014 at 06:22:47PM -0400, Rodrigo Vivi wrote: > Lets just do this small sleep to allow human eyes to see what is happening, > but let's avoid any kind of interference on the actual test. > > Signed-off-by: Rodrigo Vivi <rodrigo.vivi@intel.com> > --- > tests/kms_psr_sink_crc.c | 6 +++++- > 1 file changed, 5 insertions(+), 1 deletion(-) > > diff --git a/tests/kms_psr_sink_crc.c b/tests/kms_psr_sink_crc.c > index 0917a7f..bbe18f0 100644 > --- a/tests/kms_psr_sink_crc.c > +++ b/tests/kms_psr_sink_crc.c > @@ -327,6 +327,11 @@ static void get_sink_crc(data_t *data, char *crc) { > igt_require(ret > 0); > > fclose(file); > + > + /* The important value was already taken. > + * Now give a time for human eyes > + */ > + usleep(300000); We have igt_wait_for_keypress already, which is unfortunately not terribly userfriendly (since the env-var is undocumented). Maybe we should apply a bit of polish there. Have you considered using that one? Just fyi really. -Daniel > } > > static void test_crc(data_t *data) > @@ -342,7 +347,6 @@ static void test_crc(data_t *data) > 1, 1) == 0); > } > > - usleep(300000); > igt_assert(wait_psr_entry(data, 10)); > get_sink_crc(data, ref_crc); > > -- > 1.9.3 > > _______________________________________________ > Intel-gfx mailing list > Intel-gfx@lists.freedesktop.org > http://lists.freedesktop.org/mailman/listinfo/intel-gfx
diff --git a/tests/kms_psr_sink_crc.c b/tests/kms_psr_sink_crc.c index 0917a7f..bbe18f0 100644 --- a/tests/kms_psr_sink_crc.c +++ b/tests/kms_psr_sink_crc.c @@ -327,6 +327,11 @@ static void get_sink_crc(data_t *data, char *crc) { igt_require(ret > 0); fclose(file); + + /* The important value was already taken. + * Now give a time for human eyes + */ + usleep(300000); } static void test_crc(data_t *data) @@ -342,7 +347,6 @@ static void test_crc(data_t *data) 1, 1) == 0); } - usleep(300000); igt_assert(wait_psr_entry(data, 10)); get_sink_crc(data, ref_crc);
Lets just do this small sleep to allow human eyes to see what is happening, but let's avoid any kind of interference on the actual test. Signed-off-by: Rodrigo Vivi <rodrigo.vivi@intel.com> --- tests/kms_psr_sink_crc.c | 6 +++++- 1 file changed, 5 insertions(+), 1 deletion(-)