Re: [waffle] [PATCH 0/7] Use eglGetPlatformDisplay when possible (v2)
On Tue 25 Oct 2016, Emil Velikov wrote: > On 24 October 2016 at 22:57, Chad Versace wrote: > > Before EGL_EXT_platform_base and EGL 1.5, when using Mesa the best way > > to select the EGL platform was to set the EGL_PLATFORM environment > > variable. Now that a standard way exists, eglGetPlatformDisplay, let's > > use it when available. > > > > After this series, I have a series to add support for > > EGL_MESA_platform_surfaceless. > > > > This branch lives at > > > > https://github.com/chadversary/waffle/commits/review/eglGetPlatformDisplay-v02 > > > > v2: > > - Fix bugs found by Emil. > > - Don't set EGL_PLATFORM=android. > > - Last patch, add fallback to use eglGetPlatformDisplayEXT. > > > > Chad Versace (8): > > egl: Define EGL_PLATFORM_* enums > > egl: Update wegl_platform_init signature > > egl: Move each platform's setenv("EGL_PLATFORM") into core EGL code (v2) > > egl: Query client extensions string > > egl: Optionally query for eglGetPlatformDisplay (v2) > > egl: Update wegl_display_init signature > > egl: Use eglGetPlatformDisplay when possible (v2) > > egl: Use eglGetPlatformDisplayEXT as a fallback > > > Cannot see 8/8 in the list, so I've check it via the repo. > The lot looks great imho. > > Reviewed-by: Emil Velikov Thanks and pushed. ___ waffle mailing list waffle@lists.freedesktop.org https://lists.freedesktop.org/mailman/listinfo/waffle
Re: [waffle] [PATCH 0/7] Use eglGetPlatformDisplay when possible (v2)
On 24 October 2016 at 22:57, Chad Versace wrote: > Before EGL_EXT_platform_base and EGL 1.5, when using Mesa the best way > to select the EGL platform was to set the EGL_PLATFORM environment > variable. Now that a standard way exists, eglGetPlatformDisplay, let's > use it when available. > > After this series, I have a series to add support for > EGL_MESA_platform_surfaceless. > > This branch lives at > > https://github.com/chadversary/waffle/commits/review/eglGetPlatformDisplay-v02 > > v2: > - Fix bugs found by Emil. > - Don't set EGL_PLATFORM=android. > - Last patch, add fallback to use eglGetPlatformDisplayEXT. > > Chad Versace (8): > egl: Define EGL_PLATFORM_* enums > egl: Update wegl_platform_init signature > egl: Move each platform's setenv("EGL_PLATFORM") into core EGL code (v2) > egl: Query client extensions string > egl: Optionally query for eglGetPlatformDisplay (v2) > egl: Update wegl_display_init signature > egl: Use eglGetPlatformDisplay when possible (v2) > egl: Use eglGetPlatformDisplayEXT as a fallback > Cannot see 8/8 in the list, so I've check it via the repo. The lot looks great imho. Reviewed-by: Emil Velikov -Emil ___ waffle mailing list waffle@lists.freedesktop.org https://lists.freedesktop.org/mailman/listinfo/waffle
[waffle] [PATCH 0/7] Use eglGetPlatformDisplay when possible (v2)
Before EGL_EXT_platform_base and EGL 1.5, when using Mesa the best way to select the EGL platform was to set the EGL_PLATFORM environment variable. Now that a standard way exists, eglGetPlatformDisplay, let's use it when available. After this series, I have a series to add support for EGL_MESA_platform_surfaceless. This branch lives at https://github.com/chadversary/waffle/commits/review/eglGetPlatformDisplay-v02 v2: - Fix bugs found by Emil. - Don't set EGL_PLATFORM=android. - Last patch, add fallback to use eglGetPlatformDisplayEXT. Chad Versace (8): egl: Define EGL_PLATFORM_* enums egl: Update wegl_platform_init signature egl: Move each platform's setenv("EGL_PLATFORM") into core EGL code (v2) egl: Query client extensions string egl: Optionally query for eglGetPlatformDisplay (v2) egl: Update wegl_display_init signature egl: Use eglGetPlatformDisplay when possible (v2) egl: Use eglGetPlatformDisplayEXT as a fallback src/waffle/android/droid_display.c| 3 +- src/waffle/egl/wegl_display.c | 26 +-- src/waffle/egl/wegl_display.h | 2 +- src/waffle/egl/wegl_imports.h | 30 src/waffle/egl/wegl_platform.c| 128 +- src/waffle/egl/wegl_platform.h| 33 - src/waffle/gbm/wgbm_display.c | 2 +- src/waffle/gbm/wgbm_platform.c| 8 +-- src/waffle/wayland/wayland_display.c | 2 +- src/waffle/wayland/wayland_platform.c | 7 +- src/waffle/xegl/xegl_display.c| 2 +- src/waffle/xegl/xegl_platform.c | 8 +-- 12 files changed, 218 insertions(+), 33 deletions(-) -- 2.10.1 ___ waffle mailing list waffle@lists.freedesktop.org https://lists.freedesktop.org/mailman/listinfo/waffle
Re: [waffle] [PATCH 0/7] Use eglGetPlatformDisplay when possible
On Wed 19 Oct 2016, Emil Velikov wrote: > On 18 October 2016 at 17:58, Chad Versace wrote: > > Before EGL_EXT_platform_base and EGL 1.5, when using Mesa the best way > > to select the EGL platform was to set the EGL_PLATFORM environment > > variable. Now that a standard way exists, eglGetPlatformDisplay, let's > > use it when available. > > > > After this series, I have a series to add support for > > EGL_MESA_platform_surfaceless. > > > > This branch lives at > > > > https://github.com/chadversary/waffle/commits/review/eglGetPlatformDisplay-v01 > > > > Chad Versace (7): > > egl: Define EGL_PLATFORM_* enums > > egl: Update wegl_platform_init signature > > egl: Move each platform's setenv("EGL_PLATFORM") into core EGL code > > egl: Query client extensions string > > egl: Optionally dlsym eglGetPlatformDisplay > > egl: Update wegl_display_init signature > > egl: Use eglGetPlatformDisplay when possible > > > The questions in 5 and 7 are not blockers, by any means. Just please > split the Android change from 3. > > With that (and the extra Android change itself) are > Reviewed-by: Emil Velikov THANK YOU for reviewing this. You caught some bugs. I'll submit v2 of individual patches when I return to work on Monday. ___ waffle mailing list waffle@lists.freedesktop.org https://lists.freedesktop.org/mailman/listinfo/waffle
Re: [waffle] [PATCH 0/7] Use eglGetPlatformDisplay when possible
On 18 October 2016 at 17:58, Chad Versace wrote: > Before EGL_EXT_platform_base and EGL 1.5, when using Mesa the best way > to select the EGL platform was to set the EGL_PLATFORM environment > variable. Now that a standard way exists, eglGetPlatformDisplay, let's > use it when available. > > After this series, I have a series to add support for > EGL_MESA_platform_surfaceless. > > This branch lives at > > https://github.com/chadversary/waffle/commits/review/eglGetPlatformDisplay-v01 > > Chad Versace (7): > egl: Define EGL_PLATFORM_* enums > egl: Update wegl_platform_init signature > egl: Move each platform's setenv("EGL_PLATFORM") into core EGL code > egl: Query client extensions string > egl: Optionally dlsym eglGetPlatformDisplay > egl: Update wegl_display_init signature > egl: Use eglGetPlatformDisplay when possible > The questions in 5 and 7 are not blockers, by any means. Just please split the Android change from 3. With that (and the extra Android change itself) are Reviewed-by: Emil Velikov -Emil ___ waffle mailing list waffle@lists.freedesktop.org https://lists.freedesktop.org/mailman/listinfo/waffle
[waffle] [PATCH 0/7] Use eglGetPlatformDisplay when possible
Before EGL_EXT_platform_base and EGL 1.5, when using Mesa the best way to select the EGL platform was to set the EGL_PLATFORM environment variable. Now that a standard way exists, eglGetPlatformDisplay, let's use it when available. After this series, I have a series to add support for EGL_MESA_platform_surfaceless. This branch lives at https://github.com/chadversary/waffle/commits/review/eglGetPlatformDisplay-v01 Chad Versace (7): egl: Define EGL_PLATFORM_* enums egl: Update wegl_platform_init signature egl: Move each platform's setenv("EGL_PLATFORM") into core EGL code egl: Query client extensions string egl: Optionally dlsym eglGetPlatformDisplay egl: Update wegl_display_init signature egl: Use eglGetPlatformDisplay when possible src/waffle/android/droid_display.c| 3 +- src/waffle/egl/wegl_display.c | 24 +--- src/waffle/egl/wegl_display.h | 2 +- src/waffle/egl/wegl_imports.h | 30 ++ src/waffle/egl/wegl_platform.c| 73 ++- src/waffle/egl/wegl_platform.h| 22 ++- src/waffle/gbm/wgbm_display.c | 2 +- src/waffle/gbm/wgbm_platform.c| 8 +--- src/waffle/wayland/wayland_display.c | 2 +- src/waffle/wayland/wayland_platform.c | 7 +--- src/waffle/xegl/xegl_display.c| 2 +- src/waffle/xegl/xegl_platform.c | 8 +--- 12 files changed, 150 insertions(+), 33 deletions(-) -- 2.10.0 ___ waffle mailing list waffle@lists.freedesktop.org https://lists.freedesktop.org/mailman/listinfo/waffle