On 10.10.2017 18:30, Emil Velikov wrote:
On 6 October 2017 at 21:16, Nicolai Hähnle <nhaeh...@gmail.com> wrote:
From: Nicolai Hähnle <nicolai.haeh...@amd.com>

Tested with dEQP-EGL tests.

+    .createImageWithModifiers     = NULL,
+    .createImageFromDmaBufs2      = NULL,
+    .queryDmaBufFormats           = NULL,
+    .queryDmaBufModifiers         = NULL,
+    .queryDmaBufFormatModifierAttribs = NULL,

One of the nice things about C99 initializers is that you don't need
these lines ;-)

I added them because it highlights things that are not supported. I'll admit that that's a matter of taste.

Cheers,
Nicolai

Eric's got a good point through - please mention the set of tests and
flip the order of 5 and 6.

With the small nitpick with in 4/6 patches 1-4 are
Reviewed-by: Emil Velikov <emil.veli...@collabora.com>

I'll send some patches for the i965/!Gallium drivers as a follow-up.

Thanks
Emil


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

Reply via email to