Hi Varad,
On 17 November 2016 at 11:55, Varad Gautam <varadgau...@gmail.com> wrote:
> attribs[atti++] = EGL_NONE;
> @@ -1924,9 +1941,10 @@ gl_renderer_import_dmabuf(struct weston_compositor *ec,
> assert(gr->has_dmabuf_import);
>
> for (i = 0; i < dmabuf->attributes.n_planes; i++) {
> - /* EGL import does not have modifiers */
> + /* return if EGL doesn't support import modifiers */
> if (dmabuf->attributes.modifier[i] != 0)
> - return false;
> + if (!gr->has_dmabuf_import_modifiers)
> + return false;
Given the discussion which only kicked off on dri-devel@ when this
extension landed, please also fail the import if modifier[i] !=
modifier[0], i.e. ensure that the modifier passed is the same for all
planes.
Cheers,
Daniel
_______________________________________________
wayland-devel mailing list
wayland-devel@lists.freedesktop.org
https://lists.freedesktop.org/mailman/listinfo/wayland-devel
