[igt-dev] [PATCH 1/6] tests/kms_vrr: Move fb0 and fb1 to an array
Sean Paul
sean at poorly.run
Fri Aug 25 18:36:16 UTC 2023
From: Sean Paul <seanpaul at chromium.org>
A bit of cleanup, no functional changes.
Cc: Mark Yacoub <markyacoub at chromium.org>
Signed-off-by: Sean Paul <seanpaul at chromium.org>
---
tests/kms_vrr.c | 19 +++++++++----------
1 file changed, 9 insertions(+), 10 deletions(-)
diff --git a/tests/kms_vrr.c b/tests/kms_vrr.c
index d7ede6513..83a91b543 100644
--- a/tests/kms_vrr.c
+++ b/tests/kms_vrr.c
@@ -105,8 +105,7 @@ typedef struct data {
igt_display_t display;
int drm_fd;
igt_plane_t *primary;
- igt_fb_t fb0;
- igt_fb_t fb1;
+ igt_fb_t fb[2];
range_t range;
} data_t;
@@ -273,13 +272,13 @@ static void prepare_test(data_t *data, igt_output_t *output, enum pipe pipe)
/* Prepare resources */
igt_create_color_fb(data->drm_fd, mode.hdisplay, mode.vdisplay,
DRM_FORMAT_XRGB8888, DRM_FORMAT_MOD_LINEAR,
- 0.50, 0.50, 0.50, &data->fb0);
+ 0.50, 0.50, 0.50, &data->fb[0]);
igt_create_color_fb(data->drm_fd, mode.hdisplay, mode.vdisplay,
DRM_FORMAT_XRGB8888, DRM_FORMAT_MOD_LINEAR,
- 0.50, 0.50, 0.50, &data->fb1);
+ 0.50, 0.50, 0.50, &data->fb[1]);
- cr = igt_get_cairo_ctx(data->drm_fd, &data->fb0);
+ cr = igt_get_cairo_ctx(data->drm_fd, &data->fb[0]);
igt_paint_color(cr, 0, 0, mode.hdisplay / 10, mode.vdisplay / 10,
1.00, 0.00, 0.00);
@@ -288,7 +287,7 @@ static void prepare_test(data_t *data, igt_output_t *output, enum pipe pipe)
/* Take care of any required modesetting before the test begins. */
data->primary = igt_output_get_plane_type(output, DRM_PLANE_TYPE_PRIMARY);
- igt_plane_set_fb(data->primary, &data->fb0);
+ igt_plane_set_fb(data->primary, &data->fb[0]);
/* Clear vrr_enabled state before enabling it, because
* it might be left enabled if the previous test fails.
@@ -338,7 +337,7 @@ flip_and_measure(data_t *data, igt_output_t *output, enum pipe pipe,
vtest_ns_t vtest_ns = get_test_rate_ns(data->range);
/* Align with the flip completion event to speed up convergence. */
- do_flip(data, &data->fb0);
+ do_flip(data, &data->fb[0]);
start_ns = last_event_ns = target_ns = get_kernel_event_ns(data,
DRM_EVENT_FLIP_COMPLETE);
@@ -347,7 +346,7 @@ flip_and_measure(data_t *data, igt_output_t *output, enum pipe pipe,
int64_t diff_ns;
front = !front;
- do_flip(data, front ? &data->fb1 : &data->fb0);
+ do_flip(data, front ? &data->fb[1] : &data->fb[0]);
/* We need to cpture flip event instead of vblank event,
* because vblank is triggered after each frame, but depending
@@ -497,8 +496,8 @@ test_basic(data_t *data, enum pipe pipe, igt_output_t *output, uint32_t flags)
igt_output_override_mode(output, NULL);
igt_display_commit2(&data->display, COMMIT_ATOMIC);
- igt_remove_fb(data->drm_fd, &data->fb1);
- igt_remove_fb(data->drm_fd, &data->fb0);
+ igt_remove_fb(data->drm_fd, &data->fb[1]);
+ igt_remove_fb(data->drm_fd, &data->fb[0]);
}
/* Runs tests on outputs that are VRR capable. */
--
Sean Paul, Software Engineer, Google / Chromium OS
More information about the igt-dev
mailing list