[igt-dev] [PATCH i-g-t] test/kms_plane_alpha_blend: clearer validation for coverage-7efc
Melissa Wen
mwen at igalia.com
Wed May 25 17:09:39 UTC 2022
Inspired by previous change in alpha-7efc, get a clearer validation for
the swappable property of plane_alpha and fg.alpha in the coverage blend
formula. Get rid of the 8-increment alpha loop by refactoring
coverage-7efc in favor of one-shot check format, avoiding doubts related
to bit precision and rounding issue.
Signed-off-by: Melissa Wen <mwen at igalia.com>
---
tests/kms_plane_alpha_blend.c | 23 +++++++++++------------
1 file changed, 11 insertions(+), 12 deletions(-)
diff --git a/tests/kms_plane_alpha_blend.c b/tests/kms_plane_alpha_blend.c
index 8e54dfd6..001a1479 100644
--- a/tests/kms_plane_alpha_blend.c
+++ b/tests/kms_plane_alpha_blend.c
@@ -420,20 +420,18 @@ static void coverage_7efc(data_t *data, enum pipe pipe, igt_plane_t *plane)
igt_pipe_crc_start(data->pipe_crc);
/* for coverage, plane alpha and fb alpha should be swappable, so swap fb and alpha */
- for (i = 0; i < 256; i += 8) {
- igt_plane_set_prop_value(plane, IGT_PLANE_ALPHA, ((i/2) << 8) | (i/2));
- igt_plane_set_fb(plane, &data->argb_fb_cov_fc);
- igt_display_commit2(display, COMMIT_ATOMIC);
+ igt_plane_set_prop_value(plane, IGT_PLANE_ALPHA, 0x7e7e);
+ igt_plane_set_fb(plane, &data->argb_fb_cov_fc);
+ igt_display_commit2(display, COMMIT_ATOMIC);
- igt_pipe_crc_get_current(display->drm_fd, data->pipe_crc, &ref_crc);
+ igt_pipe_crc_get_current(display->drm_fd, data->pipe_crc, &ref_crc);
- igt_plane_set_prop_value(plane, IGT_PLANE_ALPHA, (i << 8) | i);
- igt_plane_set_fb(plane, &data->argb_fb_cov_7e);
- igt_display_commit2(display, COMMIT_ATOMIC);
+ igt_plane_set_prop_value(plane, IGT_PLANE_ALPHA, 0xfcfc);
+ igt_plane_set_fb(plane, &data->argb_fb_cov_7e);
+ igt_display_commit2(display, COMMIT_ATOMIC);
- igt_pipe_crc_get_current(display->drm_fd, data->pipe_crc, &crc);
- igt_assert_crc_equal(&ref_crc, &crc);
- }
+ igt_pipe_crc_get_current(display->drm_fd, data->pipe_crc, &crc);
+ igt_assert_crc_equal(&ref_crc, &crc);
igt_pipe_crc_stop(data->pipe_crc);
}
@@ -537,7 +535,8 @@ static void run_subtests(data_t *data, enum pipe pipe)
igt_subtest_f("pipe-%s-alpha-7efc", kmstest_pipe_name(pipe))
run_test_on_pipe_planes(data, pipe, false, true, alpha_7efc);
- igt_describe("Tests pipe coverage blending properties.");
+ igt_describe("Uses alpha values 0x7e and 0xfc to validate fg.alpha and "
+ "plane_alpha are swappable on coverage blend mode.");
igt_subtest_f("pipe-%s-coverage-7efc", kmstest_pipe_name(pipe))
run_test_on_pipe_planes(data, pipe, true, true, coverage_7efc);
--
2.35.1
More information about the igt-dev
mailing list