Re: [igt-dev] [PATCH i-g-t 0/4] kms_rotation_crc improvements and generalization

["Maíra Canal" <mcanal@igalia.com>]

Gentle ping. Thanks!

Best Regards,
- Maíra Canal

On 4/3/23 10:01, Maíra Canal wrote:
> This patchset introduces some improvements and generalizes the kms_rotation_crc
> test. Currently, I'm working on the rotation property of the vkms driver [1]
> and while working on it, I noticed that some subtests could be added and some
> subtests could be more generic.
> 
> The first and second patches make the tests more generic. Currently, some
> subtests use i915 modifiers, which make the subtest fail for other vendors
> that are not Intel. Therefore, check if the device is i915 before using the
> i915-specific modifiers.
> 
> The third patch expands the subtest "sprite-rotation-90-pos-100-0" by testing
> all 90, 180 and 270 degrees rotation with x and y offsets. This can helps us
> to assure that 90, 180 and 270 degrees rotation happens correctly even with
> offsets.
> 
> Finally, add generic tests for reflect-x and reflect-y. Currently, there are
> no tests for reflection values. So, add some generic reflection tests to
> ensure that these properties are working correctly. Note that these subtests
> must skip on i915 devices, as i915 uses reflect-x together with specific
> modifiers.
> 
> [1] https://gitlab.freedesktop.org/mairacanal/linux/-/commits/vkms/rotation
> 
> Best Regards,
> - Maíra Canal
> 
> Maíra Canal (4):
>    tests/kms_rotation_crc: make the subtest "bad-pixel-format" i915-specific
>    tests/kms_rotation_crc: use DRM_FORMAT_MOD_LINEAR as default modifier
>    tests/kms_rotation_crc: add tests for a rotated plane in a given position
>    tests/kms_rotation_crc: create generic reflection tests
> 
>   tests/kms_rotation_crc.c | 69 ++++++++++++++++++++++++++++++++++++----
>   1 file changed, 62 insertions(+), 7 deletions(-)
>