[PATCH] DirectFB Integration v1

Denis Oliver Kropp dfbdok at gmail.com
Tue Oct 8 21:02:11 CEST 2013


Hi,

here's a patch set for the master, from git format-patch on a local branch
I created.

I also used the auto indent of my editor, please check.

Thanks!


On Mon, Oct 7, 2013 at 9:20 AM, Pekka Paalanen <ppaalanen at gmail.com> wrote:

> On Mon, 7 Oct 2013 02:00:15 +0200
> Denis Oliver Kropp <dfbdok at gmail.com> wrote:
>
> > Hi Wayland!
> >
> > This is our first release of the integration of Weston with DirectFB ;)
> >
> > Please see
> http://www.directfb.org/docs/DirectFB_Foreseeing_2013-10-07.pdf
> > and http://www.directfb.org/docs/DirectFB_EGL_2013-10-07.pdf for details
> > and plans.
> >
> > With this version you can run Weston on any DirectFB enabled platform
> > including OpenGL ES support
> > via EGL where available.
> >
> > We're also introducing our abstract high-level EGL implementation called
> > "EGL United". This aims to
> > provide a complete EGL implementation with all extensions, e.g. WL
> > extensions or DIRECTFB EGLImage
> > support, while integrating different "low-level" (EGL) implementations in
> > one EGLDisplay.
> >
> > The Wayland extensions in EGL United are implemented in a generic module.
> >
> > No vendor support is needed for the Wayland extensions to work!
> >
> > The patch is for 1.2, but I will provide a patch against master next day!
>
> Hi,
>
> the EGL work sounds cool. :-) Where is it available?
>
> While you update your patch, could you also split it into logical
> commits and use git-send-email to make review more feasible, please?
> Right now there is not even a diffstat, not to mention commit messages,
> and the patch is huge.
>
> I hope the commit messages will explain why each change is done, e.g.
> why do you comment out asserts from simple-egl, what is the
> purpose of each added component, why you disable building of
> weston-editor, why you change the configure.ac defaults, why add
> weston_surface::compositor_state, etc.
>
> Also, is there a good reason to deviate from the prevalent Weston
> coding style? (spaces vs. tabs, overly long lines, mixed code and
> declarations... not just in the C++ code)
>
> Was this patch meant for review, an RFC, or just a heads-up or what
> you are preparing? I see quite many commented out lines of code, too.
>
>
> Thanks,
> pq
>
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://lists.freedesktop.org/archives/wayland-devel/attachments/20131008/aa8ee643/attachment-0001.html>
-------------- next part --------------
A non-text attachment was scrubbed...
Name: 0001-surface-Add-compositor_state-to-weston_surface-for-t.patch
Type: application/octet-stream
Size: 753 bytes
Desc: not available
URL: <http://lists.freedesktop.org/archives/wayland-devel/attachments/20131008/aa8ee643/attachment-0004.obj>
-------------- next part --------------
A non-text attachment was scrubbed...
Name: 0002-configure-Add-C-compiler-support-and-enable-c-0x.patch
Type: application/octet-stream
Size: 758 bytes
Desc: not available
URL: <http://lists.freedesktop.org/archives/wayland-devel/attachments/20131008/aa8ee643/attachment-0005.obj>
-------------- next part --------------
A non-text attachment was scrubbed...
Name: 0003-compositor-Add-DirectFB-compositor-with-EGL-and-Pixm.patch
Type: application/octet-stream
Size: 73243 bytes
Desc: not available
URL: <http://lists.freedesktop.org/archives/wayland-devel/attachments/20131008/aa8ee643/attachment-0006.obj>
-------------- next part --------------
A non-text attachment was scrubbed...
Name: 0004-DirectFB-Add-weston-dfb-adapter-to-integrate-native-.patch
Type: application/octet-stream
Size: 44937 bytes
Desc: not available
URL: <http://lists.freedesktop.org/archives/wayland-devel/attachments/20131008/aa8ee643/attachment-0007.obj>


More information about the wayland-devel mailing list