[igt-dev] [PATCH i-g-t 0/4] kms_cursor_crc test improvements and cleanups.

Juha-Pekka Heikkilä juhapekka.heikkila at gmail.com
Sat May 11 15:17:29 UTC 2019


This set look all good to me, I didn't notice anything I'd comment about 
other than:

Reviewed-by: Juha-Pekka Heikkila <juhapekka.heikkila at gmail.com>

Maarten Lankhorst kirjoitti 29.4.2019 klo 19.05:
> The tests were running on each output on each pipe, and did some cleanup
> that should be done in igt_display_reset(). Fix the test to use igt_kms
> calls all the time, and remove all hand-rolled cleanup.
> 
> As a bonus, we can now run the same test on an overlay plane with a
> s/CURSOR/OVERLAY/ as a sanity check that the test themselves are correct.
> 
> Maarten Lankhorst (4):
>    tests/kms_cursor_crc: Run tests once per pipe
>    tests/kms_cursor_crc: Handle display cleanup in init
>    lib/igt_kms: Update cursor when fb width/height changes too.
>    tests/kms_cursor_crc: Use igt_kms calls for everything.
> 
>   lib/igt_kms.c          |   4 +-
>   tests/kms_cursor_crc.c | 238 +++++++++++++++++++----------------------
>   2 files changed, 114 insertions(+), 128 deletions(-)
> 


More information about the igt-dev mailing list