On Wed, Apr 20, 2016 at 10:00:22AM -0400, [email protected] wrote:
> From: Robert Foss <[email protected]>
> 
> This change mirrors the change in 8a5c4d567fafffbda57cbe5e4ed8c419193cada5
> of drm.

I don't have that commit in my linux.git ... Much better to also cite the
commit title, not just the sha1.

> The assert(major < 1) is only needed for the legacy intel driver.
> 
> Signed-off-by: Robert Foss <[email protected]>
> ---
>  tests/core_getversion.c | 4 +++-
>  1 file changed, 3 insertions(+), 1 deletion(-)
> 
> diff --git a/tests/core_getversion.c b/tests/core_getversion.c
> index b336e9f..db28c8d 100644
> --- a/tests/core_getversion.c
> +++ b/tests/core_getversion.c
> @@ -42,7 +42,9 @@ igt_simple_main
>       igt_assert_neq(strlen(v->name), 0);
>       igt_assert_neq(strlen(v->date), 0);
>       igt_assert_neq(strlen(v->desc), 0);
> -     igt_assert_lt(1, v->version_major);
> +     if (strcmp(v->name, "i915") == 0)

Tomeu has a nice helper to check for i915.
-Daniel

> +             igt_assert_lt(1, v->version_major);
> +
>       drmFree(v);
>       close(fd);
>  }
> -- 
> 2.5.0
> 
> _______________________________________________
> Intel-gfx mailing list
> [email protected]
> https://lists.freedesktop.org/mailman/listinfo/intel-gfx

-- 
Daniel Vetter
Software Engineer, Intel Corporation
http://blog.ffwll.ch
_______________________________________________
Intel-gfx mailing list
[email protected]
https://lists.freedesktop.org/mailman/listinfo/intel-gfx

Reply via email to