Now that we have support for Y-tiled buffers, add another
iteration of tests for Y-tiled buffers.

Signed-off-by: Praveen Paneri <praveen.pan...@intel.com>
---
 tests/kms_fbc_crc.c | 71 +++++++++++++++++++++++++++++------------------------
 1 file changed, 39 insertions(+), 32 deletions(-)

diff --git a/tests/kms_fbc_crc.c b/tests/kms_fbc_crc.c
index 96af06a..e0f9bf6 100644
--- a/tests/kms_fbc_crc.c
+++ b/tests/kms_fbc_crc.c
@@ -318,12 +318,10 @@ static void prepare_crtc(data_t *data)
        igt_output_set_pipe(output, data->pipe);
 }
 
-static void create_fbs(data_t *data, bool tiled, struct igt_fb *fbs)
+static void create_fbs(data_t *data, uint64_t tiling, struct igt_fb *fbs)
 {
        int rc;
        drmModeModeInfo *mode = igt_output_get_mode(data->output);
-       uint64_t tiling = tiled ? LOCAL_I915_FORMAT_MOD_X_TILED :
-                                 LOCAL_DRM_FORMAT_MOD_NONE;
 
        rc = igt_create_color_fb(data->drm_fd, mode->hdisplay, mode->vdisplay,
                                 DRM_FORMAT_XRGB8888, tiling,
@@ -344,8 +342,8 @@ static void get_ref_crcs(data_t *data)
        struct igt_fb fbs[4];
        int i;
 
-       create_fbs(data, false, &fbs[0]);
-       create_fbs(data, false, &fbs[2]);
+       create_fbs(data, LOCAL_DRM_FORMAT_MOD_NONE, &fbs[0]);
+       create_fbs(data, LOCAL_DRM_FORMAT_MOD_NONE, &fbs[2]);
 
        fill_mmap_gtt(data, fbs[2].gem_handle, 0xff);
        fill_mmap_gtt(data, fbs[3].gem_handle, 0xff);
@@ -366,7 +364,7 @@ static void get_ref_crcs(data_t *data)
                igt_remove_fb(data->drm_fd, &fbs[i]);
 }
 
-static bool prepare_test(data_t *data, enum test_mode test_mode)
+static bool prepare_test(data_t *data, enum test_mode test_mode, uint64_t 
tiling)
 {
        igt_display_t *display = &data->display;
        igt_output_t *output = data->output;
@@ -374,7 +372,7 @@ static bool prepare_test(data_t *data, enum test_mode 
test_mode)
 
        data->primary = igt_output_get_plane_type(data->output, 
DRM_PLANE_TYPE_PRIMARY);
 
-       create_fbs(data, true, data->fb);
+       create_fbs(data, tiling, data->fb);
 
        igt_pipe_crc_free(data->pipe_crc);
        data->pipe_crc = NULL;
@@ -484,32 +482,41 @@ static void run_test(data_t *data, enum test_mode mode)
 
        reset_display(data);
 
-       for_each_pipe_with_valid_output(display, data->pipe, data->output) {
-               prepare_crtc(data);
-
-               igt_info("Beginning %s on pipe %s, connector %s\n",
-                         igt_subtest_name(),
-                         kmstest_pipe_name(data->pipe),
-                         igt_output_name(data->output));
-
-               if (!prepare_test(data, mode)) {
-                       igt_info("%s on pipe %s, connector %s: SKIPPED\n",
-                                 igt_subtest_name(),
-                                 kmstest_pipe_name(data->pipe),
-                                 igt_output_name(data->output));
-                       continue;
+       for (int tiling = I915_TILING_X;
+            tiling <= I915_TILING_Y; tiling++) {
+               for_each_pipe_with_valid_output(display,
+                                               data->pipe, data->output) {
+                       prepare_crtc(data);
+
+                       igt_info("Beginning %s on pipe %s, connector "
+                                       "%s, %s-tiled\n",
+                                       igt_subtest_name(),
+                                       kmstest_pipe_name(data->pipe),
+                                       igt_output_name(data->output),
+                                       (tiling == I915_TILING_X) ? "x":"y" );
+
+                       if (!prepare_test(data, mode,
+                                         igt_fb_tiling_to_mod(tiling))) {
+                               igt_info("%s on pipe %s, connector %s: 
SKIPPED\n",
+                                               igt_subtest_name(),
+                                               kmstest_pipe_name(data->pipe),
+                                               igt_output_name(data->output));
+                               continue;
+                       }
+
+                       valid_tests++;
+
+                       test_crc(data, mode);
+
+                       igt_info("%s on pipe %s, connector %s"
+                                       "%s-tiled: PASSED\n",
+                                       igt_subtest_name(),
+                                       kmstest_pipe_name(data->pipe),
+                                       igt_output_name(data->output),
+                                       (tiling == I915_TILING_X) ? "x":"y" );
+
+                       finish_crtc(data, mode);
                }
-
-               valid_tests++;
-
-               test_crc(data, mode);
-
-               igt_info("%s on pipe %s, connector %s: PASSED\n",
-                         igt_subtest_name(),
-                         kmstest_pipe_name(data->pipe),
-                         igt_output_name(data->output));
-
-               finish_crtc(data, mode);
        }
 
        igt_require_f(valid_tests, "no valid crtc/connector combinations 
found\n");
-- 
2.7.4

_______________________________________________
Intel-gfx mailing list
Intel-gfx@lists.freedesktop.org
https://lists.freedesktop.org/mailman/listinfo/intel-gfx

Reply via email to