Re: [PATCH v15 00/34] Atomic modesetting

2018-03-15 Thread Thierry Reding
s not meant for inclusion or review, > but is hopefully a useful aid to, well, debugging. > > Cheers, > Daniel FWIW, I tested this on Tegra with some patches to the kernel to enable the IN_FORMATS property and it all seems to work, so: Tested-by: Thierry Reding signature.asc Description: PGP signature ___ wayland-devel mailing list wayland-devel@lists.freedesktop.org https://lists.freedesktop.org/mailman/listinfo/wayland-devel

Re: [PATCH v15 20/34] compositor-drm: Support modifiers for drm_fb

2018-03-15 Thread Thierry Reding
On Mon, Feb 05, 2018 at 06:44:29PM +, Daniel Stone wrote: > Use the new drmModeAddFB2WithModifiers interface to import buffers with > modifiers. > > Signed-off-by: Daniel Stone > Reviewed-by: Pekka Paalanen > --- > configure.ac | 3 +++ > libweston/compositor-drm.c | 26 +

Re: FOSDEM15: Graphics DevRoom: call for speakers.

2015-01-13 Thread Thierry Reding
On Tue, Dec 09, 2014 at 03:39:26PM +0100, Luc Verhaegen wrote: > On Thu, Oct 02, 2014 at 07:44:57PM +0200, Luc Verhaegen wrote: > > Hi, > > > > At FOSDEM on the 31st of january and the 1st of February 2015, there > > will be another graphics DevRoom. URL: https://fosdem.org/2015/ > > > Slots wil

Re: [PATCH 2/2] compositor-drm: Add support for Tegra Jetson TK1

2014-08-08 Thread Thierry Reding
On Sat, Jun 21, 2014 at 01:37:16PM +0200, Rob Clark wrote: > On Fri, Jun 20, 2014 at 12:12 PM, James Thomas > wrote: > > Update configure.ac to add check for libdrm_tegra > > --- > > configure.ac | 18 ++ > > src/compositor-drm.c | 96 > > +

Sharing meta-data between DMA-BUF exporter and importer

2014-06-04 Thread Thierry Reding
Hi, I've recently added support for Tegra K1 to kmscube[0]. The changes add support for sharing buffers using DMA-BUF. On Tegra K1 the buffers are shared between the GK20A GPU (Nouveau) for rendering and Tegra DRM for display. One of the commits[1] adds a Tegra-specific hook that sets the tiling

Re: [PATCH 2/5] Remove wayland_protocoldir definition

2014-05-27 Thread Thierry Reding
On Tue, May 27, 2014 at 09:43:09AM +0300, Pekka Paalanen wrote: > On Mon, 19 May 2014 17:12:40 +0200 > Thierry Reding wrote: > > > From: Thierry Reding > > > > This is mostly useless and can be confusing in makefiles. The pattern > > rules defined in the makef

Re: [PATCH 1/5] Always build wayland-scanner

2014-05-27 Thread Thierry Reding
On Tue, May 27, 2014 at 09:22:30AM +0300, Pekka Paalanen wrote: > On Mon, 19 May 2014 17:12:39 +0200 > Thierry Reding wrote: > > > From: Thierry Reding > > > > When cross-compiling, wayland-scanner cannot be used to generate files > > because it is built for t

[PATCH] compositor-wayland: Fix compiler warning

2014-05-27 Thread Thierry Reding
From: Thierry Reding sizeof returns size_t, for which the correct printf specifier is %zu. Fixes the following warning when building for ARMv7. src/compositor-wayland.c: In function 'wayland_output_get_shm_buffer': src/compositor-wayland.c:260:3: warning: format &#x

[PATCH] build: Silence libtoolize warnings

2014-05-27 Thread Thierry Reding
From: Thierry Reding When running the autogen.sh script, libtoolize complains thusly: libtoolize: Consider adding `AC_CONFIG_MACRO_DIR([m4])' to configure.ac and libtoolize: rerunning libtoolize, to keep the correct libtool macros in-tree. libtoolize: Consider a

Re: [PATCH wayland-web] Added depencies and bug fixes to build instructions

2014-05-23 Thread Thierry Reding
On Thu, May 22, 2014 at 09:35:23AM -0700, Bill Spitzak wrote: > On 05/21/2014 03:09 PM, Thierry Reding wrote: > >On Wed, May 21, 2014 at 02:54:36PM -0700, Bill Spitzak wrote: > >>On 05/21/2014 02:16 PM, Thierry Reding wrote: > >> > >>>While I agree with t

Re: [PATCH wayland-web] Added depencies and bug fixes to build instructions

2014-05-22 Thread Thierry Reding
On Thu, May 22, 2014 at 09:25:40AM +0300, Pekka Paalanen wrote: > On Thu, 22 May 2014 00:18:23 +0200 > Thierry Reding wrote: > > > On Wed, May 21, 2014 at 12:30:18PM +0300, Pekka Paalanen wrote: > > > On Tue, 20 May 2014 13:12:32 -0700 Bill Spitzak wrote: > > [.

Re: [PATCH weston] comp-wayland: use safe foreach when destroying outputs

2014-05-21 Thread Thierry Reding
t_for_each_safe instead, which is designed to > handle this kind of pattern. > > Signed-off-by: U. Artie Eoff > --- > src/compositor-wayland.c | 4 ++-- > 1 file changed, 2 insertions(+), 2 deletions(-) Reviewed-by: Thierry Reding pgp6tjMVuBDQH.pgp Description: PGP signature

Re: [PATCH wayland-web] Added depencies and bug fixes to build instructions

2014-05-21 Thread Thierry Reding
On Wed, May 21, 2014 at 12:30:18PM +0300, Pekka Paalanen wrote: > On Tue, 20 May 2014 13:12:32 -0700 Bill Spitzak wrote: [...] > > I have to tell you that such one-line-at-a-time cut & paste is > > unbelievably tedious, and my biggest screwups when trying this on a > > second machine was when I

Re: [PATCH wayland-web] Added depencies and bug fixes to build instructions

2014-05-21 Thread Thierry Reding
On Wed, May 21, 2014 at 02:54:36PM -0700, Bill Spitzak wrote: > On 05/21/2014 02:16 PM, Thierry Reding wrote: > > >While I agree with the other points, I think it's perfectly consistent > >for --version to output the version of pkg-config itself. There's > >--m

Re: [PATCH wayland-web] Added depencies and bug fixes to build instructions

2014-05-21 Thread Thierry Reding
On Wed, May 21, 2014 at 12:17:29PM -0700, Bill Spitzak wrote: > On 05/21/2014 02:30 AM, Pekka Paalanen wrote: [...] > >Would that not cause the main build guide to become a mixture of > >apt-get, emerge, yum, pacman, aptitude etc. commands? Where the > >complete set of commands would never work on

Re: [PATCH wayland-web] Added depencies and bug fixes to build instructions

2014-05-21 Thread Thierry Reding
On Tue, May 20, 2014 at 01:12:32PM -0700, Bill Spitzak wrote: [...] > The biggest hint would be to print something with the word "pkg-config" in > it. Then I would have the secret password that would lead me to a man page > that would tell me what is going on. > > Please print the package name the

[PATCH 2/5] Remove wayland_protocoldir definition

2014-05-19 Thread Thierry Reding
From: Thierry Reding This is mostly useless and can be confusing in makefiles. The pattern rules defined in the makefile snippet are generic enough to allow the protocol sources to reside in subdirectories. Signed-off-by: Thierry Reding --- wayland-scanner.m4 | 2 -- wayland-scanner.mk | 6

[PATCH 5/5] Make wayland-scanner.pc sysroot aware

2014-05-19 Thread Thierry Reding
From: Thierry Reding The pkgdatadir variable is used to find data files installed by wayland. If wayland is installed in a sysroot, the path to this directory needs to be prefixed with the sysroot to make sure the proper files are found. ${pc_sysrootdir} is expanded to the value of the

[PATCH 3/5] Improve wayland-scanner m4 rules

2014-05-19 Thread Thierry Reding
From: Thierry Reding When cross-compiling always look up the wayland-scanner binary in the PATH. Using the value from the wayland-scanner.pc file will not work because it will point to a binary built for the host. While trying to find the path of the wayland-scanner makefile snippet, check that

[PATCH 1/5] Always build wayland-scanner

2014-05-19 Thread Thierry Reding
From: Thierry Reding When cross-compiling, wayland-scanner cannot be used to generate files because it is built for the host architecture. To overcome this, users currently need to instruct the build not to create wayland-scanner and provide a native variant in the PATH. This has the

[PATCH 4/5] Dogfood wayland-scanner makefile snippet

2014-05-19 Thread Thierry Reding
From: Thierry Reding To reduce code duplication and make sure it receives at minimum amount of testing, use the wayland-scanner makefile snippet in this package. Signed-off-by: Thierry Reding --- Makefile.am | 9 + 1 file changed, 1 insertion(+), 8 deletions(-) diff --git a

Re: [PATCH v3] configure.ac: Change in build system to use the path from pkg-config for wayland-scanner.

2014-05-19 Thread Thierry Reding
On Mon, May 19, 2014 at 01:39:24PM +0300, Pekka Paalanen wrote: > On Mon, 19 May 2014 11:56:39 +0200 Thierry Reding > wrote: > > On Mon, May 19, 2014 at 11:59:51AM +0300, Pekka Paalanen wrote: > > > On Sun, 18 May 2014 00:39:16 +0200 Thierry Reding > > > wrote:

Re: [PATCH v3] configure.ac: Change in build system to use the path from pkg-config for wayland-scanner.

2014-05-19 Thread Thierry Reding
On Mon, May 19, 2014 at 11:59:51AM +0300, Pekka Paalanen wrote: > On Sun, 18 May 2014 00:39:16 +0200 Thierry Reding > wrote: [...] > > One issue here is that if --disable-scanner is specified, then the > > wayland-scanner binary is neither built nor insta

Re: [PATCH v3] configure.ac: Change in build system to use the path from pkg-config for wayland-scanner.

2014-05-17 Thread Thierry Reding
On Fri, May 16, 2014 at 03:43:47PM +0530, Srivardhan Hebbar wrote: > This is a fix to the bug https://bugs.freedesktop.org/show_bug.cgi?id=78688. > > Signed-off-by: Srivardhan Hebbar > --- > configure.ac | 14 +- > 1 file changed, 9 insertions(+), 5 deletions(-) > > diff --git a/c

[PATCH] Do not distribute generated headers

2014-05-07 Thread Thierry Reding
From: Thierry Reding The wayland-server-protocol.h and wayland-client-protocol.h headers are currently being shipped in tarballs created using make dist. This causes out-of-tree builds to fail since make will detect that the headers exist by looking at the source directory (via VPATH) and not

Re: undefined symbol: wl_display_get_rgb_visual

2011-05-19 Thread Thierry Reding
* Christopher Friedt wrote: > Hi folks, > > I'm running into the following error when trying to launch demos. > > terminal: symbol lookup error: /usr/lib64/egl/egl_gallium.so: > undefined symbol: wl_display_get_rgb_visual > > The folllowing code snippet confirms that the symbol does not get > in

[PATCH] egl: Link wayland-drm.a into libEGL after egl_dri2

2011-05-18 Thread Thierry Reding
Fixes the following build error in wayland-demos: CCLD wayland-compositor /usr/lib/libEGL.so: undefined reference to `wayland_drm_buffer_get_buffer' /usr/lib/libEGL.so: undefined reference to `wayland_drm_uninit' /usr/lib/libEGL.so: undefined reference to `wayl