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

Maarten Lankhorst maarten.lankhorst at linux.intel.com
Mon May 13 15:07:26 UTC 2019


Op 11-05-2019 om 17:17 schreef Juha-Pekka Heikkilä:
> 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(-)
>>
Thanks, pushed. :)

~Maarten



More information about the igt-dev mailing list