diff options
| author | Maarten Lankhorst <maarten.lankhorst@linux.intel.com> | 2018-02-02 11:00:50 +0100 |
|---|---|---|
| committer | Maarten Lankhorst <maarten.lankhorst@linux.intel.com> | 2018-02-12 13:13:01 +0100 |
| commit | aa5068c58947f31252cad69d80686e40bbce29db (patch) | |
| tree | 557e4984186b3b5abc17b8d1980190e1b4c9c233 /tests/kms_panel_fitting.c | |
| parent | e62de1c7c4ecde0bea5256faec0e8793810f9410 (diff) | |
tests: Always call igt_remove_fb without checking.
This cleans up the tests slightly.
Signed-off-by: Maarten Lankhorst <maarten.lankhorst@linux.intel.com>
Reviewed-by: Mika Kahola <mika.kahola@intel.com>
Tested-by: Vidya Srinivas <vidya.srinivas@intel.com>
Diffstat (limited to 'tests/kms_panel_fitting.c')
| -rw-r--r-- | tests/kms_panel_fitting.c | 15 |
1 files changed, 3 insertions, 12 deletions
diff --git a/tests/kms_panel_fitting.c b/tests/kms_panel_fitting.c index e0ca6cca..b3cee223 100644 --- a/tests/kms_panel_fitting.c +++ b/tests/kms_panel_fitting.c @@ -51,10 +51,7 @@ static void prepare_crtc(data_t *data, igt_output_t *output, enum pipe pipe, igt_output_set_pipe(output, pipe); /* before allocating, free if any older fb */ - if (data->fb1.fb_id) { - igt_remove_fb(data->drm_fd, &data->fb1); - data->fb1.fb_id = 0; - } + igt_remove_fb(data->drm_fd, &data->fb1); /* allocate fb for plane 1 */ igt_create_pattern_fb(data->drm_fd, @@ -83,14 +80,8 @@ static void cleanup_crtc(data_t *data, igt_output_t *output, igt_plane_t *plane) { igt_display_t *display = &data->display; - if (data->fb1.fb_id) { - igt_remove_fb(data->drm_fd, &data->fb1); - data->fb1.fb_id = 0; - } - if (data->fb2.fb_id) { - igt_remove_fb(data->drm_fd, &data->fb2); - data->fb2.fb_id = 0; - } + igt_remove_fb(data->drm_fd, &data->fb1); + igt_remove_fb(data->drm_fd, &data->fb2); if (plane->type != DRM_PLANE_TYPE_PRIMARY) { igt_plane_t *primary; |