Re: [PATCH weston v2 2/4] weston: Allow relative paths for modules

2013-06-07 Thread Bill Spitzak
Pekka Paalanen wrote: Hi Quentin, do we expect to load modules from subdirs under MODULEDIR ever? If not, I think this could be simplified to just scan the name for a '/', and if there is one use the name as is, and if there is not then prepend MODULEDIR. That's exactly what the earlier vers

Re: [evdev] device not configured are unhandled instead back in error

2013-06-07 Thread Marc Chalain
I updated my repository and the back to error doesn't crash Weston. Then It seems not more usefull to manage unhandled device. But to show problems that we can find in embedded solution, I continue some explanations. On board we have input device just for the on/off button, another for gpio_keypad

Re: [evdev] device not configured are unhandled instead back in error

2013-06-07 Thread Daniel Stone
Hi, On 7 June 2013 11:16, Marc Chalain wrote: > I tried and at now, it seems to add a lot of complexity. > I solution is to read the weston.ini file inside input.c... but is it > regular ? This is the usual solution for now, yes. > 2013/6/7 Daniel Stone >> >> Hi, >> >> On 7 June 2013 06:33, Ma

Re: [fbdev_backend] initialization of the first frame buffer

2013-06-07 Thread Marc Chalain
>From 6553ed3ad9845113f3b2280e6fc94e2b326a7eca Mon Sep 17 00:00:00 2001 From: mchalain Date: Fri, 7 Jun 2013 17:18:23 +0200 Subject: [PATCH 2/2] [fbdev_backend] initialization of the first frame buffer --- src/compositor-fbdev.c |8 +++- 1 file changed, 7 insertions(+), 1 deletion(-) di

Re: [evdev] device not configured are unhandled instead back in error

2013-06-07 Thread Marc Chalain
I tried and at now, it seems to add a lot of complexity. I solution is to read the weston.ini file inside input.c... but is it regular ? 2013/6/7 Daniel Stone > Hi, > > On 7 June 2013 06:33, Marc Chalain wrote: > > A input device could not be not configurated. That should not be an error > > a

Re: [fbdev_backend]: support of DIRECTCOLOR Frame buffer

2013-06-07 Thread Marc Chalain
>From e30486e0288e7be526fc358a5318b344522e407b Mon Sep 17 00:00:00 2001 From: mchalain Date: Fri, 7 Jun 2013 17:12:31 +0200 Subject: [PATCH] [fbdev_backend]: support of DIRECTCOLOR Frame buffer --- src/compositor-fbdev.c | 49 +--- 1 file changed, 46

RE: [PATCH 1/1] evdev: Fix a high CPU utilization bug

2013-06-07 Thread Singh, Satyeshwar
This happened to me in a VMWare image. Do you think that would make a difference? -Original Message- From: Kristian Høgsberg [mailto:hoegsb...@gmail.com] Sent: Thursday, June 06, 2013 10:17 PM To: Singh, Satyeshwar Cc: wayland-devel@lists.freedesktop.org Subject: Re: [PATCH 1/1] evdev: F

Re: [fbdev_backend]: support of DIRECTCOLOR Frame buffer

2013-06-07 Thread Pekka Paalanen
On Fri, 7 Jun 2013 16:03:18 +0200 Marc Chalain wrote: > > You should execute the ioctl in fbdev_set_screen_info. > fbdev_set_screen_info is used only when the fb device is reenabled. The > colors map has to be initialized at the beginning in all cases. > If we use this function we will call it in

Re: [PATCH weston v2 4/4] weston.pc: Provide moduledir

2013-06-07 Thread Pekka Paalanen
On Fri, 7 Jun 2013 10:46:44 +0200 Quentin Glidic wrote: > From: Quentin Glidic > > This allows modules to use the correct directory for the targeted Weston > to install themselves. > > Signed-off-by: Quentin Glidic > --- > configure.ac | 4 > src/Makefile.am | 1 -

Re: [fbdev_backend]: support of DIRECTCOLOR Frame buffer

2013-06-07 Thread Marc Chalain
> You should execute the ioctl in fbdev_set_screen_info. fbdev_set_screen_info is used only when the fb device is reenabled. The colors map has to be initialized at the beginning in all cases. If we use this function we will call it in all cases and it's not required. >I don't see the codepath wh

Re: [PATCH] weston.pc: Added libexecdir and pkglibexecdir variables

2013-06-07 Thread Pekka Paalanen
On Mon, 3 Jun 2013 12:24:09 -0300 "Eduardo Lima (Etrunko)" wrote: > From: "Eduardo Lima (Etrunko)" > > These can be used by external clients to check the installation path > > Signed-off-by: Eduardo Lima (Etrunko) > --- > src/weston.pc.in | 2 ++ > 1 file changed, 2 insertions(+) > > diff

[PATCH weston 3/3] gl-renderer: Split gl_renderer_attach() into smaller functions

2013-06-07 Thread Ander Conselvan de Oliveira
--- src/gl-renderer.c | 187 ++--- 1 file changed, 105 insertions(+), 82 deletions(-) diff --git a/src/gl-renderer.c b/src/gl-renderer.c index c5d683c..b853b7f 100644 --- a/src/gl-renderer.c +++ b/src/gl-renderer.c @@ -1201,14 +1201,115 @@ ensure_t

[PATCH weston 2/3] gl-renderer: Fix initial upload of SHM buffer as texture

2013-06-07 Thread Ander Conselvan de Oliveira
The fix to not call glTexImage2D() on every attach does not properly set the texture damage region appropriately when the surface has a buffer transform with 90 or 270 degrees rotation, since it would simply multiply the buffer dimensions by the buffer scale, but in this case width and height are i

[PATCH weston 1/3] gl-renderer: Track the buffer type in gl_surface_state

2013-06-07 Thread Ander Conselvan de Oliveira
Checking for gs->num_images for determining the previous buffer type when attaching is not reliable. The number of images is never cleared in the SHM path, so after a switch from an EGL buffer to SHM, every following attach of an SHM buffer will happen with gs->num_images > 0, and the code will ass

Re: [PATCH weston v2 1/4] tests: Add .weston extension to clients tests

2013-06-07 Thread Pekka Paalanen
On Fri, 07 Jun 2013 15:27:17 +0200 Quentin Glidic wrote: > On 07/06/2013 15:20, Pekka Paalanen wrote: > > Seems to work ok to me. Would be nice to have a "non-weston" test here > > to show off. One extra blank line above. > > Thanks for the review. > > I have a test in my option parser patch, a

Re: [PATCH weston v2 2/4] weston: Allow relative paths for modules

2013-06-07 Thread Pekka Paalanen
On Fri, 7 Jun 2013 10:46:42 +0200 Quentin Glidic wrote: > From: Quentin Glidic > > This is to be used by tests or when developping to load modules directly > from the build tree > > Signed-off-by: Quentin Glidic > --- > man/weston.man | 5 +++-- > src/compositor.c | 9 ++--- > 2 files

Re: [evdev] device not configured are unhandled instead back in error

2013-06-07 Thread Daniel Stone
Hi, On 7 June 2013 06:33, Marc Chalain wrote: > A input device could not be not configurated. That should not be an error > and just an unhandled device. > On embedded configuration there is different input devices which are not > used. The intention of this codepath is to avoid the situation on

Re: [PATCH weston v2 1/4] tests: Add .weston extension to clients tests

2013-06-07 Thread Quentin Glidic
On 07/06/2013 15:20, Pekka Paalanen wrote: Seems to work ok to me. Would be nice to have a "non-weston" test here to show off. One extra blank line above. Thanks for the review. I have a test in my option parser patch, and we should move the config-parser test here too, imo. -- Quentin “Sa

Re: [PATCH weston v2 1/4] tests: Add .weston extension to clients tests

2013-06-07 Thread Pekka Paalanen
On Fri, 7 Jun 2013 10:46:41 +0200 Quentin Glidic wrote: > From: Quentin Glidic > > We can then add tests which do not use Weston in the test suite. > > Signed-off-by: Quentin Glidic > --- > tests/.gitignore | 7 +-- > tests/Makefile.am | 41 ++--- >

Re: [fbdev_backend]: support of DIRECTCOLOR Frame buffer

2013-06-07 Thread John Kåre Alsaker
You should execute the ioctl in fbdev_set_screen_info. I don't see the codepath which results in cmap.transp being not set in the test for it. You don't initialize cmap.red so the test for it gives an undefined result. Don't define macros in the middle of code with a literal that's only used onc

[compositor] idle-time configuration inside weston.ini

2013-06-07 Thread Marc Chalain
this patch use the "idle-time" entry inside the "core" section of weston.ini to set the named value. weston-compositor-idle-time-configuration.patch Description: Binary data ___ wayland-devel mailing list wayland-devel@lists.freedesktop.org http://lists

[evdev] device not configured are unhandled instead back in error

2013-06-07 Thread Marc Chalain
A input device could not be not configurated. That should not be an error and just an unhandled device. On embedded configuration there is different input devices which are not used. weston-evdev-unconfigure-device-unhandled.patch Description: Binary data _

[fbdev_backend] initialization of the first frame buffer

2013-06-07 Thread Marc Chalain
Often the frame buffer device allows the double buffering, In this case the displayed buffer could not be initialized. This is done by yoffset of the varinfo. weston-fbdev-first-buffer-init.patch Description: Binary data ___ wayland-devel mailing list w

[fbdev_backend]: support of DIRECTCOLOR Frame buffer

2013-06-07 Thread Marc Chalain
this patch allow to use direct color and set the colors map with default value. weston-fbdev-directcolor.patch Description: Binary data ___ wayland-devel mailing list wayland-devel@lists.freedesktop.org http://lists.freedesktop.org/mailman/listinfo/wayl

[PATCH]: mtdev could be disabled at configuration

2013-06-07 Thread Marc Chalain
mtdev is not necessary on some device (desktop or set-top-box). For embedded solution the minimum of dependencies is required. "configure" accepts the --disable-mtdev option. weston-mtdev-disabled.patch Description: Binary data ___ wayland-devel mailing

Re: new contribution

2013-06-07 Thread Pekka Paalanen
On Fri, 7 Jun 2013 11:17:48 +0200 Marc Chalain wrote: > Hello, > I work on a ST-E ARM board (snowball ux500 for information) and I modified > weston to launch it on the platform. > To do that I made differents patch on the fbdev-backend and the main > compositor. > I would like to know who are th

new contribution

2013-06-07 Thread Marc Chalain
Hello, I work on a ST-E ARM board (snowball ux500 for information) and I modified weston to launch it on the platform. To do that I made differents patch on the fbdev-backend and the main compositor. I would like to know who are the main maintainers of this parts, to send and communicate. - fbdev_

[PATCH weston v2 4/4] weston.pc: Provide moduledir

2013-06-07 Thread Quentin Glidic
From: Quentin Glidic This allows modules to use the correct directory for the targeted Weston to install themselves. Signed-off-by: Quentin Glidic --- configure.ac | 4 src/Makefile.am | 1 - src/weston.pc.in | 2 ++ src/xwayland/Makefile.am | 1 - 4 files cha

[PATCH weston v2 3/4] weston.pc: Add dependencies for pixman-1 and xkbcommon

2013-06-07 Thread Quentin Glidic
From: Quentin Glidic Weston headers include pixman and libxkbcommon headers Signed-off-by: Quentin Glidic --- src/weston.pc.in | 1 + 1 file changed, 1 insertion(+) diff --git a/src/weston.pc.in b/src/weston.pc.in index 828cb1f..7a09938 100644 --- a/src/weston.pc.in +++ b/src/weston.pc.in @@

[PATCH weston v2 1/4] tests: Add .weston extension to clients tests

2013-06-07 Thread Quentin Glidic
From: Quentin Glidic We can then add tests which do not use Weston in the test suite. Signed-off-by: Quentin Glidic --- tests/.gitignore | 7 +-- tests/Makefile.am | 41 ++--- 2 files changed, 23 insertions(+), 25 deletions(-) diff --git a/tests/.giti

[PATCH weston v2 0/4] Small patches

2013-06-07 Thread Quentin Glidic
From: Quentin Glidic Rebased and commented patches Quentin Glidic (4): tests: Add .weston extension to clients tests weston: Allow relative paths for modules weston.pc: Add dependencies for pixman-1 and xkbcommon weston.pc: Provide moduledir configure.ac | 4 man/west

[PATCH weston v2 2/4] weston: Allow relative paths for modules

2013-06-07 Thread Quentin Glidic
From: Quentin Glidic This is to be used by tests or when developping to load modules directly from the build tree Signed-off-by: Quentin Glidic --- man/weston.man | 5 +++-- src/compositor.c | 9 ++--- 2 files changed, 9 insertions(+), 5 deletions(-) diff --git a/man/weston.man b/man/we