diff options
author | Maarten Lankhorst <maarten.lankhorst@linux.intel.com> | 2018-01-30 17:21:53 +0100 |
---|---|---|
committer | Maarten Lankhorst <maarten.lankhorst@linux.intel.com> | 2018-02-05 13:41:34 +0100 |
commit | 08e8621aaaed6853ecaede6325512e2775b78dea (patch) | |
tree | 2279224c7d6e8cfc46a087d79b9f478da4d222c1 /tests/kms_plane_scaling.c | |
parent | 48d9da5636d54d37261b3fd2fa25aec70746b999 (diff) |
tests/kms_plane_scaling: Test all pixel formats with clamping and clipping too
Run across all combinations of pixel formats to ensure better testing.
Signed-off-by: Maarten Lankhorst <maarten.lankhorst@linux.intel.com>
Reviewed-by: Mika Kahola <mika.kahola@intel.com>
Diffstat (limited to 'tests/kms_plane_scaling.c')
-rw-r--r-- | tests/kms_plane_scaling.c | 61 |
1 files changed, 43 insertions, 18 deletions
diff --git a/tests/kms_plane_scaling.c b/tests/kms_plane_scaling.c index 18a42035..208f9262 100644 --- a/tests/kms_plane_scaling.c +++ b/tests/kms_plane_scaling.c @@ -58,14 +58,10 @@ static int get_num_scalers(uint32_t devid, enum pipe pipe) return 1; } -static void cleanup_crtc(data_t *data) +static void cleanup_fbs(data_t *data) { int i; - igt_display_reset(&data->display); - igt_pipe_crc_free(data->pipe_crc); - data->pipe_crc = NULL; - for (i = 0; i < ARRAY_SIZE(data->fb); i++) { if (!data->fb[i].fb_id) continue; @@ -75,6 +71,15 @@ static void cleanup_crtc(data_t *data) } } +static void cleanup_crtc(data_t *data) +{ + igt_display_reset(&data->display); + igt_pipe_crc_free(data->pipe_crc); + data->pipe_crc = NULL; + + cleanup_fbs(data); +} + static void prepare_crtc(data_t *data, igt_output_t *output, enum pipe pipe, igt_plane_t *plane, drmModeModeInfo *mode) { @@ -401,28 +406,20 @@ test_plane_scaling_on_pipe(data_t *d, enum pipe pipe, igt_output_t *output) } static void -test_scaler_with_clipping_clamping_scenario(data_t *d, enum pipe pipe, igt_output_t *output) +__test_scaler_with_clipping_clamping_scenario(data_t *d, drmModeModeInfo *mode, + uint32_t f1, uint32_t f2) { - drmModeModeInfo *mode; - - igt_require(get_num_scalers(d->devid, pipe) >= 2); - - mode = igt_output_get_mode(output); - d->plane1 = &d->display.pipes[pipe].planes[0]; - prepare_crtc(d, output, pipe, d->plane1, mode); + cleanup_fbs(d); igt_create_pattern_fb(d->drm_fd, - mode->hdisplay, mode->vdisplay, - DRM_FORMAT_XRGB8888, + mode->hdisplay, mode->vdisplay, f1, LOCAL_I915_FORMAT_MOD_X_TILED, &d->fb[1]); igt_create_pattern_fb(d->drm_fd, - mode->hdisplay, mode->vdisplay, - DRM_FORMAT_XRGB8888, + mode->hdisplay, mode->vdisplay, f2, LOCAL_I915_FORMAT_MOD_Y_TILED, &d->fb[2]); igt_plane_set_fb(d->plane1, &d->fb[1]); - d->plane2 = igt_output_get_plane(output, 1); igt_plane_set_fb(d->plane2, &d->fb[2]); igt_fb_set_position(&d->fb[1], d->plane1, 0, 0); @@ -440,6 +437,34 @@ test_scaler_with_clipping_clamping_scenario(data_t *d, enum pipe pipe, igt_outpu igt_display_commit2(&d->display, COMMIT_ATOMIC); } +static void +test_scaler_with_clipping_clamping_scenario(data_t *d, enum pipe pipe, igt_output_t *output) +{ + drmModeModeInfo *mode; + + igt_require(get_num_scalers(d->devid, pipe) >= 2); + + mode = igt_output_get_mode(output); + d->plane1 = &d->display.pipes[pipe].planes[0]; + d->plane2 = &d->display.pipes[pipe].planes[1]; + prepare_crtc(d, output, pipe, d->plane1, mode); + + for (int i = 0; i < d->plane1->drm_plane->count_formats; i++) { + unsigned f1 = d->plane1->drm_plane->formats[i]; + if (!can_draw(f1)) + continue; + + for (int j = 0; j < d->plane2->drm_plane->count_formats; j++) { + unsigned f2 = d->plane2->drm_plane->formats[j]; + + if (!can_draw(f2)) + continue; + + __test_scaler_with_clipping_clamping_scenario(d, mode, f1, f2); + } + } +} + static void find_connected_pipe(igt_display_t *display, bool second, enum pipe *pipe, igt_output_t **output) { enum pipe first = PIPE_NONE; |