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@xxxxxxxxx>
> ---
> 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@xxxxxxxxxxxxxxxxxxxxx
> http://lists.freedesktop.org/mailman/listinfo/intel-gfx
--
Daniel Vetter
Software Engineer, Intel Corporation
+41 (0) 79 365 57 48 - http://blog.ffwll.ch
_______________________________________________
Intel-gfx mailing list
Intel-gfx@xxxxxxxxxxxxxxxxxxxxx
http://lists.freedesktop.org/mailman/listinfo/intel-gfx
