tests/exynos: introduce wait_for_user_input
authorTobias Jakobi <tjakobi@math.uni-bielefeld.de>
Tue, 24 Feb 2015 14:20:44 +0000 (15:20 +0100)
committerEmil Velikov <emil.l.velikov@gmail.com>
Tue, 10 Mar 2015 19:21:33 +0000 (19:21 +0000)
Currently getchar() is used to pause execution after each test.
The user isn't informed if one is supposed to do anything for
the tests to continue, so print a simple message to make this
more clear.

v3: Compactify printf calls as pointed out by
    Emil Velikov <emil.l.velikov@gmail.com>.

Signed-off-by: Tobias Jakobi <tjakobi@math.uni-bielefeld.de>
Reviewed-by: Emil Velikov <emil.l.velikov@gmail.com>
Tested-by: Joonyoung Shim <jy0922.shim@samsung.com>
tests/exynos/exynos_fimg2d_test.c

index 8ef0eb3..d89fc23 100644 (file)
@@ -214,6 +214,14 @@ static void exynos_destroy_buffer(struct exynos_bo *bo)
        exynos_bo_destroy(bo);
 }
 
+static void wait_for_user_input(int last)
+{
+       printf("press <ENTER> to %s\n", last ? "exit test application" :
+                       "skip to next test");
+
+       getchar();
+}
+
 static int g2d_solid_fill_test(struct exynos_device *dev, struct exynos_bo *dst)
 {
        struct g2d_context *ctx;
@@ -644,7 +652,7 @@ int main(int argc, char **argv)
                goto err_rm_fb;
        }
 
-       getchar();
+       wait_for_user_input(0);
 
        src = exynos_create_buffer(dev, screen_width * screen_height * 4, 0);
        if (!src) {
@@ -658,7 +666,7 @@ int main(int argc, char **argv)
                goto err_free_src;
        }
 
-       getchar();
+       wait_for_user_input(0);
 
        ret = test_case.copy_with_scale(dev, src, bo, G2D_IMGBUF_GEM);
        if (ret < 0) {
@@ -666,7 +674,7 @@ int main(int argc, char **argv)
                goto err_free_src;
        }
 
-       getchar();
+       wait_for_user_input(1);
 
        /*
         * The blend test uses the userptr functionality of exynos-drm, which