Re: [Intel-gfx] [PATCH i-g-t] tests/kms_plane_lowres: Rework tests to work without fbcon.

2017-10-23 Thread Maarten Lankhorst
Op 23-10-17 om 12:05 schreef Mika Kahola: > Reviewed-by: Mika Kahola > > On Fri, 2017-10-13 at 16:58 +0200, Maarten Lankhorst wrote: >> kmstest_get_crtc was skipping because at that point the crtc was not >> active yet, instead we should only use igt_assert_plane_visible >>

Re: [Intel-gfx] [PATCH i-g-t] tests/kms_plane_lowres: Rework tests to work without fbcon.

2017-10-23 Thread Mika Kahola
Reviewed-by: Mika Kahola On Fri, 2017-10-13 at 16:58 +0200, Maarten Lankhorst wrote: > kmstest_get_crtc was skipping because at that point the crtc was not > active yet, instead we should only use igt_assert_plane_visible > directly. Unexport kmstest_get_crtc, since

[Intel-gfx] [PATCH i-g-t] tests/kms_plane_lowres: Rework tests to work without fbcon, v3.

2017-10-19 Thread Maarten Lankhorst
kmstest_get_crtc was skipping because at that point the crtc was not active yet, instead we should only use igt_assert_plane_visible directly. Unexport kmstest_get_crtc, since nothing here should need it. While at it fix a small leak in igt_assert_plane_visible, the only remaining user.

[Intel-gfx] [PATCH i-g-t] tests/kms_plane_lowres: Rework tests to work without fbcon, v2.

2017-10-17 Thread Maarten Lankhorst
kmstest_get_crtc was skipping because at that point the crtc was not active yet, instead we should only use igt_assert_plane_visible directly. Unexport kmstest_get_crtc, since nothing here should need it. While at it fix a small leak in igt_assert_plane_visible, the only remaining user.

[Intel-gfx] [PATCH i-g-t] tests/kms_plane_lowres: Rework tests to work without fbcon.

2017-10-13 Thread Maarten Lankhorst
kmstest_get_crtc was skipping because at that point the crtc was not active yet, instead we should only use igt_assert_plane_visible directly. Unexport kmstest_get_crtc, since nothing here should need it. While at it fix a small leak in igt_assert_plane_visible, the only remaining user.