[Mesa-dev] [PATCH mesa] khronos/egl: remove dependency on Android NDK header
Eric Engestrom
eric.engestrom at imgtec.com
Thu Aug 24 14:22:49 UTC 2017
On Thursday, 2017-08-24 08:54:04 -0500, Rob Herring wrote:
> On Thu, Aug 24, 2017 at 7:49 AM, Eric Engestrom
> <eric.engestrom at imgtec.com> wrote:
> > Khronos: https://github.com/KhronosGroup/EGL-Registry/pull/22
> > Cc: Rob Herring <robh at kernel.org>
> > Cc: Emil Velikov <emil.l.velikov at gmail.com>
> > Signed-off-by: Eric Engestrom <eric.engestrom at imgtec.com>
> > ---
> > include/EGL/eglplatform.h | 3 +--
> > 1 file changed, 1 insertion(+), 2 deletions(-)
> >
> > diff --git a/include/EGL/eglplatform.h b/include/EGL/eglplatform.h
> > index f045d009c0..bf9ec0bf5f 100644
> > --- a/include/EGL/eglplatform.h
> > +++ b/include/EGL/eglplatform.h
> > @@ -97,8 +97,7 @@ typedef void *EGLNativeWindowType;
> >
> > #elif defined(__ANDROID__) || defined(ANDROID)
> >
> > -#include <android/native_window.h>
> > -
> > +struct ANativeWindow;
> > struct egl_native_pixmap_t;
>
> How does this work when we need to dereference the struct to call
> ANativeWindow::dequeueBuffer() and others?
Right, there are two things at play here:
- eglplatform.h doesn't need to know the struct, so it shouldn't include
a whole header but simply forward declare for the pointer.
- platform_android does need it, but wasn't including the proper
headers, so I missed it in my initial grep.
It seems these two issues are orthogonal after all. This khronos header
patch should land IMO, but platform_android needs a patch to avoid
breaking (incoming) and will need the proper libraries that your patch
2/2 provides for the O update.
Cheers,
Eric
More information about the mesa-dev
mailing list