diff options
author | Mohammed Thasleem <mohammed.thasleem@intel.com> | 2022-07-07 14:52:11 +0530 |
---|---|---|
committer | Bhanuprakash Modem <bhanuprakash.modem@intel.com> | 2022-07-08 10:51:28 +0530 |
commit | 40f6e44b76059527dec2978a25e868be49dc076a (patch) | |
tree | db7bf657ffba92961e9d385728d2e996632d7a65 /tests/i915 | |
parent | 6725767a84109e934ca138d94df0482386c44532 (diff) |
tests/i915/kms_legacy_colorkey: Create dynamic subtests
Modified kms_legacy_colorkey to include dynamic test cases.
v2: Updated igt_dynamic subtest names of invalid-plane.
v3: Minor changes.
v4: Changes in discription.
Signed-off-by: Mohammed Thasleem <mohammed.thasleem@intel.com>
Reviewed-by: Bhanuprakash Modem <bhanuprakash.modem@intel.com>
Diffstat (limited to 'tests/i915')
-rw-r--r-- | tests/i915/kms_legacy_colorkey.c | 50 |
1 files changed, 36 insertions, 14 deletions
diff --git a/tests/i915/kms_legacy_colorkey.c b/tests/i915/kms_legacy_colorkey.c index de9610a7..5b2fd64b 100644 --- a/tests/i915/kms_legacy_colorkey.c +++ b/tests/i915/kms_legacy_colorkey.c @@ -43,28 +43,50 @@ static void test_plane(uint32_t plane_id, int expected_ret) sizeof(ckey)) == expected_ret); } -igt_simple_main +igt_main { - drm_fd = drm_open_driver_master(DRIVER_INTEL); + igt_fixture { + drm_fd = drm_open_driver_master(DRIVER_INTEL); - kmstest_set_vt_graphics_mode(); + kmstest_set_vt_graphics_mode(); - igt_display_require(&display, drm_fd); + igt_display_require(&display, drm_fd); + for_each_pipe(&display, p) { + for_each_plane_on_pipe(&display, p, plane) { + max_id = max(max_id, plane->drm_plane->plane_id); + } + } + + } - for_each_pipe(&display, p) { - for_each_plane_on_pipe(&display, p, plane) { - bool is_valid = (plane->type == DRM_PLANE_TYPE_PRIMARY || - plane->type == DRM_PLANE_TYPE_CURSOR); - test_plane(plane->drm_plane->plane_id, - is_valid ? -ENOENT : 0); + igt_describe("Test to check the legacy set colorkey ioctl " + "only works for sprite planes.\n"); + igt_subtest_with_dynamic("basic") { + for_each_pipe(&display, p) { + igt_dynamic_f("pipe-%s", kmstest_pipe_name(p)) { + for_each_plane_on_pipe(&display, p, plane) { + bool is_valid = (plane->type == DRM_PLANE_TYPE_PRIMARY || + plane->type == DRM_PLANE_TYPE_CURSOR); - max_id = max(max_id, plane->drm_plane->plane_id); + test_plane(plane->drm_plane->plane_id, + is_valid ? -ENOENT : 0); + max_id = max(max_id, plane->drm_plane->plane_id); + } + } } } /* try some invalid IDs too */ - test_plane(0, -ENOENT); - test_plane(max_id + 1, -ENOENT); + igt_describe("Check invalid plane id's, zero and outrange\n"); + igt_subtest_with_dynamic("invalid-plane") { + igt_dynamic("zero-id") + test_plane(0, -ENOENT); + igt_dynamic("outrange-id") + test_plane(max_id + 1, -ENOENT); + } - igt_display_fini(&display); + igt_fixture { + igt_display_fini(&display); + close(drm_fd); + } } |