On 10.10.2017 18:30, Emil Velikov wrote:
On 6 October 2017 at 21:16, Nicolai Hähnle <[email protected]> wrote:
From: Nicolai Hähnle <[email protected]>

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 <[email protected]>

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

Thanks
Emil


_______________________________________________
mesa-dev mailing list
[email protected]
https://lists.freedesktop.org/mailman/listinfo/mesa-dev

Reply via email to