Re: [igt-dev] [i-g-t 4/7] tests/kms_cursor_legacy: Add XE support

[Karthik B S <karthik.b.s@intel.com>]

On 4/13/2023 9:57 PM, Bhanuprakash Modem wrote:
> Add XE driver support for kms tests.
>
> Signed-off-by: Bhanuprakash Modem <bhanuprakash.modem@intel.com>
> ---
>   tests/kms_cursor_legacy.c | 3 ++-
>   1 file changed, 2 insertions(+), 1 deletion(-)
>
> diff --git a/tests/kms_cursor_legacy.c b/tests/kms_cursor_legacy.c
> index 5f430090815..cd247cf0088 100644
> --- a/tests/kms_cursor_legacy.c
> +++ b/tests/kms_cursor_legacy.c
> @@ -308,7 +308,7 @@ enum flip_test {
>   static bool cursor_slowpath(igt_display_t *display, enum flip_test mode)
>   {
>   	/* Intel display 9 and newer will handle cursor movement as fastsets */
> -	if (is_i915_device(display->drm_fd) &&
> +	if (is_intel_device(display->drm_fd) &&
>   	    intel_display_ver(intel_get_drm_devid(display->drm_fd)) >= 9)
>   	    return true;
>   
> @@ -1708,6 +1708,7 @@ igt_main
>   	igt_describe("this test perform a busy bo update followed by a cursor update");
>   	igt_subtest_group {
>   		igt_fixture {
> +			igt_require_i915(display.drm_fd);

Hi,

Could you please remove the 'igt_require_intel' check inside 
'flip_vs_cursor_busy_crc', as with the above check that is redundant.

With this removed, the patch LGTM.

Reviewed-by: Karthik B S <karthik.b.s@intel.com>

Thanks,
Karthik.B.S
>   			igt_require_pipe_crc(display.drm_fd);
>   			igt_display_require_output(&display);
>   		}