On 17-01-04 22:55:17, Varad Gautam wrote:
Hi Ben,

On Wed, Jan 4, 2017 at 3:25 AM, Ben Widawsky <b...@bwidawsk.net> wrote:
On 16-11-24 20:50:37, Varad Gautam wrote:

This is the second revision to the EGL_EXT_image_dma_buf_import_modifiers
[1]
series at [2], addressing the comments received. This diverges from v1 due
to
some reordered/squashed changes, hence the resend.


1-4 are Reviewed-by: Ben Widawsky <b...@bwidawsk.net>
5-9,12-13: See below
10: Not really my thing, and tied up with 9.
14: Acked-by: Ben Widawsky <b...@bwidawsk.net>

I guess we need to decide whose patches we're going to use. Many of the
patches
overlap with the work I did for GBM support (v2 here:
https://patchwork.freedesktop.org/series/16244/). While yours went to the
list
first, I had been working on mine for quite a while. I really would prefer
to
keep my patches only because the i965 hardware has proven really finicky
over
this stuff, and I want to avoid breakage, but I'll be as accommodating as
possible.

sure, we can have createImageWithModifiers/gbm support go in first. I've
already got a rebase of dmabuf eglext patches on your series, and it's easy
enough for me to deal with the clashes (just a couple of version bumps /
slight tweaks).

Thanks,
Varad


You're my new best friend. I think I can push most of the first ones since Eric
Engestrom reviewed those. I'll see about getting review on the others.

Thanks!!


Thanks.
Ben

[snip]

--
Ben Widawsky, Intel Open Source Technology Center
_______________________________________________
mesa-dev mailing list
mesa-dev@lists.freedesktop.org
https://lists.freedesktop.org/mailman/listinfo/mesa-dev

Reply via email to