[Mesa-dev] [PATCH] Introduce platform displays internally
Chia-I Wu
olvaffe at gmail.com
Tue Jun 22 23:55:33 PDT 2010
2010/6/22 Kristian Høgsberg <krh at bitplanet.net>:
> I think the egl-drm-display branch looks good. The platform display
> patch is definitely good to commit, and I don't see a problem in
> merging the entire egl-drm-display branch either. As I said before, I
> was practically sold on the multi-egl idea, until I realized that the
> libEGL-drm.so dependency is pulled into higher level libraries that
> use EGL in a platform-independent way. What you implemented in
> egl-drm-display is exactly what I would've done myself.
Great. I will merge the branch.
> I'm still on the fence about -DMESA_EGL_NO_X11_HEADERS vs #include
> <egldrm.h>, that is, I think adding a new egldrm.h header file that
> defines the eglGetDRMDirsplayMESA() (and potentially DRM specific
> surface create functions) is a pretty clean approach as well. In
> fact, I think that would make sense for X11 and the other existing
> platforms too. Ideally, including egl.h wouldn't define any native
> types or display/surface create functions.
I am fine with -DMESA_EGL_NO_X11_HEADERS or egldrm.h. One idea I have is to
install egl-drm.pc. Applications may check egl-drm.pc to define the macro
automatically, or know the existence of egldrm.h.
> But for now, egl-drm-display gets us what we want. Thanks for taking this on.
--
olv at LunarG.com
More information about the mesa-dev
mailing list