Re: [igt-dev] [PATCH i-g-t 1/9] tests/kms_lease: Add invalid planes tests

[Boris Brezillon <boris.brezillon@collabora.com>]

On Wed, 20 Feb 2019 17:25:22 +0100
Daniel Vetter <daniel.vetter@ffwll.ch> wrote:

> Test gap because we're using universal planes by default. Fix that.
> 
> Cc: Keith Packard <keithp@keithp.com>
> Signed-off-by: Daniel Vetter <daniel.vetter@ffwll.ch>

Reviewed-by: Boris Brezillon <boris.brezillon@collabora.com>

> ---
>  tests/kms_lease.c | 15 +++++++++++++++
>  1 file changed, 15 insertions(+)
> 
> diff --git a/tests/kms_lease.c b/tests/kms_lease.c
> index cca424598ea7..70a96362d77a 100644
> --- a/tests/kms_lease.c
> +++ b/tests/kms_lease.c
> @@ -573,6 +573,20 @@ static void lease_invalid_crtc(data_t *data)
>  	igt_assert_eq(ret, -EINVAL);
>  }
>  
> +static void lease_invalid_plane(data_t *data)
> +{
> +	lease_t lease;
> +	uint32_t save_plane_id;
> +	int ret;
> +
> +	/* Create an invalid lease */
> +	save_plane_id = data->plane_id;
> +	data->plane_id = 0xbaadf00d;
> +	ret = make_lease(data, &lease);
> +	data->plane_id = save_plane_id;
> +	igt_assert_eq(ret, -EINVAL);
> +}
> +
>  
>  static void run_test(data_t *data, void (*testfunc)(data_t *))
>  {
> @@ -625,6 +639,7 @@ igt_main
>  		{ "lease_again", lease_again },
>  		{ "lease_invalid_connector", lease_invalid_connector },
>  		{ "lease_invalid_crtc", lease_invalid_crtc },
> +		{ "lease_invalid_plane", lease_invalid_plane },
>  		{ }
>  	}, *f;
>  

_______________________________________________
igt-dev mailing list
igt-dev@lists.freedesktop.org
https://lists.freedesktop.org/mailman/listinfo/igt-dev