Re: Additional pixel formats for wl_shm

2013-05-01 Thread Kristian Høgsberg
On Fri, Apr 26, 2013 at 04:08:01PM +0300, Pekka Paalanen wrote: > Hi Kristian, > > I'm working on Raspberry Pi, where the VideoCore can deal directly with > a large number of pixel formats. However, wl_shm only exposes ARGB and > XRGB 32-bit per pixel formats, which on such tiny devices are very >

Re: [DRM] Patch for Compositor Drm

2013-05-01 Thread Kristian Høgsberg
On Mon, Apr 22, 2013 at 05:12:58PM +0100, Christopher Michael wrote: > From acb79e4a5921525b35e07e48f7f903e42a08fb7c Mon Sep 17 00:00:00 2001 > From: Chris Michael > Date: Mon, 22 Apr 2013 15:22:48 +0100 > Subject: [PATCH] Fix not checking return value of drmIoctl function call to > map dumb buff

Re: [PATCH 1/4] Add a destroy_signal on weston_output

2013-05-01 Thread Kristian Høgsberg
On Wed, May 01, 2013 at 09:52:10PM +0100, Richard Hughes wrote: > --- > src/compositor.c | 3 +++ > src/compositor.h | 1 + > 2 files changed, 4 insertions(+) This and the other three patches look good, committed. Kristian > diff --git a/src/compositor.c b/src/compositor.c > index 693df2c..0214

Re: Clarifying scope and goals for weston

2013-05-01 Thread Sam Spilsbury
On Wed, May 1, 2013 at 2:01 PM, Kristian Høgsberg wrote: > On Fri, Apr 19, 2013 at 11:03:16PM -0700, Kenneth Graunke wrote: >> On 04/18/2013 12:04 AM, Sam Spilsbury wrote: >> >On Wed, Apr 17, 2013 at 1:34 PM, Kristian Høgsberg >> >wrote: [snip] > Toolkit type scenegraphs > (QML, Clutter) has a

Re: [patches] Add a color management framework to weston

2013-05-01 Thread Kristian Høgsberg
On Mon, Apr 22, 2013 at 01:30:28PM +0300, Pekka Paalanen wrote: > On Fri, 5 Apr 2013 14:23:50 -0500 > Thomas Daede wrote: > > > I am not sure that doing the color conversion in the compositor is the > > correct place. Some of it has to be there to support vcgt, but for > > more general conversion

Re: Clarifying scope and goals for weston

2013-05-01 Thread Kristian Høgsberg
On Fri, Apr 19, 2013 at 11:03:16PM -0700, Kenneth Graunke wrote: > On 04/18/2013 12:04 AM, Sam Spilsbury wrote: > >On Wed, Apr 17, 2013 at 1:34 PM, Kristian Høgsberg > >wrote: > [snip] > >2. Re-use of weston functionality > > > >Weston has a lot of really great functionality that would be useful

[PATCH 4/4] Include config.h in compositor-drm.c

2013-05-01 Thread Richard Hughes
--- src/compositor-drm.c | 4 1 file changed, 4 insertions(+) diff --git a/src/compositor-drm.c b/src/compositor-drm.c index 857aeed..db4db33 100644 --- a/src/compositor-drm.c +++ b/src/compositor-drm.c @@ -21,6 +21,10 @@ * CONNECTION WITH THE USE OR PERFORMANCE OF THIS SOFTWARE. */ +#

[PATCH 3/4] Add a set_gamma vfunc on weston_output

2013-05-01 Thread Richard Hughes
--- src/compositor-drm.c | 24 src/compositor.h | 7 +++ 2 files changed, 31 insertions(+) diff --git a/src/compositor-drm.c b/src/compositor-drm.c index c8016cd..857aeed 100644 --- a/src/compositor-drm.c +++ b/src/compositor-drm.c @@ -545,6 +545,27 @@ drm_output

[PATCH 2/4] Add a output_created_signal on weston_compositor

2013-05-01 Thread Richard Hughes
--- src/compositor.c | 2 ++ src/compositor.h | 1 + 2 files changed, 3 insertions(+) diff --git a/src/compositor.c b/src/compositor.c index 0214eed..c1acd50 100644 --- a/src/compositor.c +++ b/src/compositor.c @@ -3076,6 +3076,7 @@ weston_output_init(struct weston_output *output, struct weston_

[PATCH 1/4] Add a destroy_signal on weston_output

2013-05-01 Thread Richard Hughes
--- src/compositor.c | 3 +++ src/compositor.h | 1 + 2 files changed, 4 insertions(+) diff --git a/src/compositor.c b/src/compositor.c index 693df2c..0214eed 100644 --- a/src/compositor.c +++ b/src/compositor.c @@ -2906,6 +2906,8 @@ weston_output_destroy(struct weston_output *output) {

Re: [PATCH] Override modules list and don't always load desktop-shell.so

2013-05-01 Thread Kristian Høgsberg
On Fri, Apr 19, 2013 at 04:11:01PM -0700, Bill Spitzak wrote: > >2013/4/16 Kristian Høgsberg : > > >>I know the current behavior is a bit problematic for some use cases. > >>However it works well for the case where you load a plugin in addition > >>to the shell, like xwayland. I'm not sure what t

Re: [PATCH 3/3] Add a colord implementation of a CMS plugin for weston

2013-05-01 Thread Kristian Høgsberg
On Wed, Apr 24, 2013 at 02:58:04PM +0100, Richard Hughes wrote: > This allows users to change the assigned display profile in GNOME (using > gnome-control-center) or KDE (using colord-kde) and also allows the profiling > tools to correctly inhibit the calibration state whilst measuring the native >

Re: [PATCH 2/3] Add initial color management framework code

2013-05-01 Thread Kristian Høgsberg
On Wed, Apr 24, 2013 at 02:58:03PM +0100, Richard Hughes wrote: > ICC profiles can now be specified in weston.ini for each output, or a CMS > implementation can optionally loaded from a pluggable module. Hey Richard, This is cool stuff and I'm happy to see weston gain support for color management

Re: [PATCH] client: Add wl_display_prepare_read() API to thread model assumptions

2013-05-01 Thread Bill Spitzak
Kristian Høgsberg wrote: + * A correct sequence would be: + * + * while (wl_display_prepare_read(display) != 0) + * wl_display_dispatch_pending(display); + * wl_display_flush(display); + * poll(fds, nfds, -1); + * wl_display_read_events(display); + * wl_display_dispatch_pendi

Re: [PATCH 1/3] Extract and parse the EDID when outputs are added

2013-05-01 Thread Kristian Høgsberg
On Wed, Apr 24, 2013 at 02:58:02PM +0100, Richard Hughes wrote: > At the moment we're only extracting interesting strings. We have to be quite > careful parsing the EDID data, as vendors like to do insane things. > > The original EDID parsing code was written by me for gnome-color-manager. > --- >

Re: [PATCH] weston.ini: document background-type

2013-05-01 Thread Kristian Høgsberg
On Mon, Apr 22, 2013 at 11:00:07AM +0200, Emilio Pozuelo Monfort wrote: > --- > man/weston.ini.man |4 > weston.ini |1 + > 2 files changed, 5 insertions(+) Thanks, committed. Kristian > diff --git a/man/weston.ini.man b/man/weston.ini.man > index 8dde82c..2287730 100644 >

Re: [PATCH] X server uses xwayland.conf instead of xorg.conf

2013-05-01 Thread Kristian Høgsberg
On Sun, Apr 21, 2013 at 10:24:58PM -0700, spit...@gmail.com wrote: > From: spitzak > > This allows X applications to be run on wayland without having to > delete conf files needed to run the legacy X server on the same machine. > > This is the only method I have found that will allow the built-i

Re: [PATCH] Fixed simple-egl tear-down order to prevent a crash on exit time.

2013-05-01 Thread Kristian Høgsberg
On Fri, Apr 19, 2013 at 05:49:12PM +, Yeh, Sinclair wrote: > wl_egl_window_destory() distroys the window handle that > dri2_destroy_surface() later uses when eglTerminate() is called. > > Reordering the tear down order prevents such case from occuring. That's a good fix, thanks. Committed.

Re: [PATCH] client: Add wl_display_prepare_read() API to thread model assumptions

2013-05-01 Thread Uli Schlachter
On 01.05.2013 17:52, Kristian Høgsberg wrote: > On Tue, Apr 30, 2013 at 11:45:15PM +0200, Uli Schlachter wrote: >> On 30.04.2013 22:06, Kristian Høgsberg wrote: >> [...] >>> diff --git a/src/connection.c b/src/connection.c >>> index dca134b..b26402f 100644 >>> --- a/src/connection.c >>> +++ b/src/c

Re: [PATCH] client: Add wl_display_prepare_read() API to thread model assumptions

2013-05-01 Thread Kristian Høgsberg
On Tue, Apr 30, 2013 at 11:45:15PM +0200, Uli Schlachter wrote: > Evening, > > On 30.04.2013 22:06, Kristian Høgsberg wrote: > [...] > > diff --git a/src/connection.c b/src/connection.c > > index dca134b..b26402f 100644 > > --- a/src/connection.c > > +++ b/src/connection.c > > @@ -324,7 +324,7 @@