[Mesa-dev] [PATCH v2 00/14] implement EGL_EXT_image_dma_buf_import_modifiers v2

[Ben Widawsky]

On 17-01-04 22:55:17, Varad Gautam wrote:
>Hi Ben,
>
>On Wed, Jan 4, 2017 at 3:25 AM, Ben Widawsky <ben at 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 <ben at bwidawsk.net>
>> 5-9,12-13: See below
>> 10: Not really my thing, and tied up with 9.
>> 14: Acked-by: Ben Widawsky <ben at 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