On Thu, Jan 04, 2018 at 03:12:43PM +0100, Maarten Lankhorst wrote: > New way of doing things is calling igt_display_reset() before any > setup code. This way if the configuration stays the same, the initial > modeset will be a noop. > > Signed-off-by: Maarten Lankhorst <[email protected]>
Also for the remaining patches: Reviewed-by: Daniel Vetter <[email protected]> > --- > tests/kms_vblank.c | 11 ++--------- > 1 file changed, 2 insertions(+), 9 deletions(-) > > diff --git a/tests/kms_vblank.c b/tests/kms_vblank.c > index 362221c03677..d2154f1c7605 100644 > --- a/tests/kms_vblank.c > +++ b/tests/kms_vblank.c > @@ -74,6 +74,8 @@ static void prepare_crtc(data_t *data, int fd, igt_output_t > *output) > igt_display_t *display = &data->display; > igt_plane_t *primary; > > + igt_display_reset(display); > + > /* select the pipe we want to use */ > igt_output_set_pipe(output, data->pipe); > > @@ -95,16 +97,7 @@ static void prepare_crtc(data_t *data, int fd, > igt_output_t *output) > > static void cleanup_crtc(data_t *data, int fd, igt_output_t *output) > { > - igt_display_t *display = &data->display; > - igt_plane_t *primary; > - > igt_remove_fb(fd, &data->primary_fb); > - > - primary = igt_output_get_plane_type(output, DRM_PLANE_TYPE_PRIMARY); > - igt_plane_set_fb(primary, NULL); > - > - igt_output_set_pipe(output, PIPE_ANY); > - igt_display_commit(display); > } > > static int wait_vblank(int fd, union drm_wait_vblank *vbl) > -- > 2.15.1 > > _______________________________________________ > Intel-gfx mailing list > [email protected] > https://lists.freedesktop.org/mailman/listinfo/intel-gfx -- Daniel Vetter Software Engineer, Intel Corporation http://blog.ffwll.ch _______________________________________________ Intel-gfx mailing list [email protected] https://lists.freedesktop.org/mailman/listinfo/intel-gfx
