Re: [PATCH] compositor-headless: Report a more realistic physical size

2018-07-03 Thread Pekka Paalanen
On Mon, 02 Jul 2018 10:29:48 -0400 Simon Ser wrote: > > On Thu, 12 Apr 2018 09:31:48 +0200 > > Johan Klokkhammer Helsing wrote: > > > > > Some clients rely on the physical size to determine the physical DPI. > > > With the > > > previous implementation, we would report 1px==1mm, which is a

Re: Users of libweston?

2018-07-03 Thread Pekka Paalanen
On Mon, 2 Jul 2018 09:34:40 -0500 Matt Hoosier wrote: > On Mon, Jul 2, 2018 at 4:10 AM Pekka Paalanen wrote: > > > On Fri, 29 Jun 2018 10:05:58 -0500 > > Matt Hoosier wrote: > > > > > Hi all, > > > > > > Pekka's recent comments about wanting to enable set-top boxes built with > > >

RE: Wayland content-protection extension

2018-07-03 Thread C, Ramalingam
> -Original Message- > From: Pekka Paalanen [mailto:ppaala...@gmail.com] > Sent: Tuesday, July 3, 2018 1:14 PM > To: C, Ramalingam > Cc: Arnaud Vrac ; Nautiyal, Ankit K > ; wayland-devel@lists.freedesktop.org; Daniel > Stone ; Sharma, Shashank > > Subject: Re: Wayland content-protection

Re: Wayland content-protection extension

2018-07-03 Thread Pekka Paalanen
On Mon, 2 Jul 2018 20:37:23 +0530 Ramalingam C wrote: > On Monday 02 July 2018 07:26 PM, Pekka Paalanen wrote: > > Ok, so it is acceptable to transmit protected content to a potentially > > unprotected display until the app reacts to the event. That makes > > things very simple. > > Yes. >

Re: [PATCH weston] simple-dmabuf-drm: fix build with --disable-egl

2018-07-03 Thread Pekka Paalanen
On Mon, 2 Jul 2018 17:22:30 +0200 Emilio Pozuelo Monfort wrote: > Signed-off-by: Emilio Pozuelo Monfort > --- > I tried a build with --disable-egl as I didn't have the headers > installed, and it broke here. The EGL usage here seemed optional so I > did that, but I didn't run-test the result.

Re: [PATCH weston] parse_modeline: Ignore case of {h,v}sync flags

2018-07-03 Thread Pekka Paalanen
On Tue, 26 Jun 2018 20:40:08 +0200 Guido Günther wrote: > Some modeline generators put out e.g. +HSync instead of +hsync. Accept > that too since it's not ambigous. > > Signed-off-by: Guido Günther > --- > libweston/compositor-drm.c | 8 > 1 file changed, 4 insertions(+), 4

Re: [PATCH v5] compositor-drm: Add aspect-ratio parsing support

2018-07-03 Thread Sharma, Shashank
Hello Pekka, Thanks for your attention and code review of the patch series. Ankit will respond on rest of the review comments, I would take one related to mode with implicit aspect ratio information. Please find my comment inline. Regards Shashank On 6/28/2018 7:09 PM, Pekka Paalanen wrote:

Re: [PATCH weston] parse_modeline: Ignore case of {h,v}sync flags

2018-07-03 Thread Guido Günther
Hi, On Tue, Jul 03, 2018 at 12:04:37PM +0300, Pekka Paalanen wrote: > On Tue, 26 Jun 2018 20:40:08 +0200 > Guido Günther wrote: > > > Some modeline generators put out e.g. +HSync instead of +hsync. Accept > > that too since it's not ambigous. > > > > Signed-off-by: Guido Günther > > --- > >

Re: [PATCH v5] compositor-drm: Add aspect-ratio parsing support

2018-07-03 Thread Pekka Paalanen
On Tue, 3 Jul 2018 14:42:06 +0530 "Sharma, Shashank" wrote: > Hello Pekka, > > Thanks for your attention and code review of the patch series. > Ankit will respond on rest of the review comments, I would take one > related to mode with implicit aspect ratio information. > Please find my comment

Re: [PATCH v5] compositor-drm: Add aspect-ratio parsing support

2018-07-03 Thread Nautiyal, Ankit K
Hi Pekka, Thanks for the review and the comments. I agree with all of the suggestions, will send a patch in a day or two. Discussion about the CEA with aspect ratio and and Non- CEA modes is currently in progress in another thread, lets close it in the other thread itself. Please find my

Re: [PATCH v5] compositor-drm: Add aspect-ratio parsing support

2018-07-03 Thread Sharma, Shashank
Regards Shashank On 7/3/2018 3:26 PM, Pekka Paalanen wrote: On Tue, 3 Jul 2018 14:42:06 +0530 "Sharma, Shashank" wrote: Hello Pekka, Thanks for your attention and code review of the patch series. Ankit will respond on rest of the review comments, I would take one related to mode with

Re: [PATCH weston] simple-dmabuf-drm: fix build with --disable-egl

2018-07-03 Thread Emilio Pozuelo Monfort
On 03/07/18 11:00, Pekka Paalanen wrote: > On Mon, 2 Jul 2018 17:22:30 +0200 > Emilio Pozuelo Monfort wrote: > >> Signed-off-by: Emilio Pozuelo Monfort >> --- >> I tried a build with --disable-egl as I didn't have the headers >> installed, and it broke here. The EGL usage here seemed optional

Re: [PATCH wayland 2/8] fixup! scanner: use c99 initializers for the wl_interface * array

2018-07-03 Thread Pekka Paalanen
On Thu, 14 Jun 2018 16:49:39 +0100 Emil Velikov wrote: > --- > tests/data/example-code.c | 190 > > tests/data/small-code-core.c| 16 ++-- > tests/data/small-code.c | 16 ++-- > tests/data/small-private-code.c | 16 ++-- > 4 files

Re: [PATCH wayland 4/8] fixup! scanner: use c99 initializers for the request/events arrays

2018-07-03 Thread Pekka Paalanen
On Thu, 14 Jun 2018 16:49:41 +0100 Emil Velikov wrote: > --- > tests/data/example-code.c | 238 > > tests/data/small-code-core.c| 8 +- > tests/data/small-code.c | 8 +- > tests/data/small-private-code.c | 8 +- > 4 files

Re: [PATCH v5] compositor-drm: Add aspect-ratio parsing support

2018-07-03 Thread Pekka Paalanen
On Tue, 3 Jul 2018 15:56:10 +0530 "Nautiyal, Ankit K" wrote: > Hi Pekka, > > Thanks for the review and the comments. > > I agree with all of the suggestions, will send a patch in a day or two. > > Discussion about the CEA with aspect ratio and and Non- CEA modes is > currently in progress in

Re: [PATCH weston] simple-dmabuf-drm: fix build with --disable-egl

2018-07-03 Thread Pekka Paalanen
On Tue, 3 Jul 2018 12:35:30 +0200 Emilio Pozuelo Monfort wrote: > On 03/07/18 11:00, Pekka Paalanen wrote: > > On Mon, 2 Jul 2018 17:22:30 +0200 > > Emilio Pozuelo Monfort wrote: > > > >> Signed-off-by: Emilio Pozuelo Monfort > >> --- > >> I tried a build with --disable-egl as I didn't

[PATCH] protocol: allow to send a zero physical output size

2018-07-03 Thread Simon Ser
Physical size doesn't always make sense for all outputs. In case it's not available or not relevant, allow compositors to send zero. --- In practice this doesn't seem to break any client toolkit. We've been doing that for a long time in wlroots. protocol/wayland.xml | 3 +++ 1 file changed, 3

Re: [PATCH wayland 8/8] fixup! scanner: initialize .{method, event}_count via ARRAY_SIZE

2018-07-03 Thread Simon Ser
On July 3, 2018 12:45 PM, Pekka Paalanen wrote: > On Thu, 14 Jun 2018 16:49:45 +0100 > Emil Velikov wrote: > > > --- > > tests/data/example-code.c | 73 > > + > > tests/data/small-code-core.c| 5 +-- > > tests/data/small-code.c | 5

[PATCH v2 wayland 2/4] contributing: review rules for bugs

2018-07-03 Thread Pekka Paalanen
From: Pekka Paalanen Half of the ideas came from Daniel but most of them are reworded, the rest are my thoughts. Mention compiler warnings specifically, and be more explicit on what kind of code or bugs or bug fixes are acceptable or not. Clarify commit scope. v2: move the "In a patch series"

[PATCH wayland 1/4] contributing: how to read the review rules

2018-07-03 Thread Pekka Paalanen
From: Pekka Paalanen This is to avoid fighting around the letter of the guidelines. This is not a protocol spec. Signed-off-by: Pekka Paalanen --- CONTRIBUTING.md | 5 + 1 file changed, 5 insertions(+) diff --git a/CONTRIBUTING.md b/CONTRIBUTING.md index 70d0eca..538613f 100644 ---

[PATCH wayland 4/4] contributing: document the release cycle freezes

2018-07-03 Thread Pekka Paalanen
From: Pekka Paalanen These should be the conventions we have been using since 1.0, written down more accurately. Signed-off-by: Pekka Paalanen --- CONTRIBUTING.md | 35 +++ 1 file changed, 35 insertions(+) diff --git a/CONTRIBUTING.md b/CONTRIBUTING.md index

Re: [PATCH wayland 8/8] fixup! scanner: initialize .{method, event}_count via ARRAY_SIZE

2018-07-03 Thread Pekka Paalanen
On Thu, 14 Jun 2018 16:49:45 +0100 Emil Velikov wrote: > --- > tests/data/example-code.c | 73 > + > tests/data/small-code-core.c| 5 +-- > tests/data/small-code.c | 5 +-- > tests/data/small-private-code.c | 5 +-- > 4 files

Re: [PATCH wayland 6/8] fixup! scanner: use c99 initializers for the interface symbols

2018-07-03 Thread Pekka Paalanen
On Thu, 14 Jun 2018 16:49:43 +0100 Emil Velikov wrote: > --- > tests/data/example-code.c | 124 > +++- > tests/data/small-code-core.c| 6 +- > tests/data/small-code.c | 6 +- > tests/data/small-private-code.c | 6 +- > 4 files

Re: [PATCH v5] compositor-drm: Add aspect-ratio parsing support

2018-07-03 Thread Nautiyal, Ankit K
Hi Shashank, Thanks for the detailed explanation about the relationship between the aspect-ratio, CEA modes and VIC codes. I will capture these and try to add this as part of man pages for a better understanding. Regards, Ankit On 7/3/2018 4:05 PM, Sharma, Shashank wrote: Regards

Re: [RFC PATCH libinput] tools: run list-quirks from the git data when running from the builddir

2018-07-03 Thread Peter Hutterer
On Mon, Jun 25, 2018 at 11:06:36AM +0300, Pekka Paalanen wrote: > On Fri, 22 Jun 2018 12:48:53 +1000 > Peter Hutterer wrote: > > > I'd like some comment on this from someone who knows this stuff better than > > I do. The main goal is to have the same binary default to different search > > paths,

[PATCH v2 libinput 1/5] tools: if the execdir is the builddir, add it to the path

2018-07-03 Thread Peter Hutterer
When running libinput tools from the builddir, look up the subtools in the builddir as well. Otherwise, add the install prefix to the list of lookup locations. This ensures that a) we're running builddir stuff against builddir stuff, but also b) that we're not running builddir stuff against

Upcoming release reminder

2018-07-03 Thread Derek Foreman
Hi all, Just a quick reminder that we're due for alpha shortly, with the following intended release schedule: Alpha - July 10th Beta - July 24th RC1 - August 7th First possible release August 14th. Thanks, Derek ___ wayland-devel mailing list

Re: [PATCH wayland 8/8] fixup! scanner: initialize .{method, event}_count via ARRAY_SIZE

2018-07-03 Thread Emil Velikov
On 3 July 2018 at 12:45, Pekka Paalanen wrote: > On Thu, 14 Jun 2018 16:49:45 +0100 > Emil Velikov wrote: > >> --- >> tests/data/example-code.c | 73 >> + >> tests/data/small-code-core.c| 5 +-- >> tests/data/small-code.c | 5 +-- >>

[PATCH] protocol: warn clients about some wl_output properties

2018-07-03 Thread Simon Ser
All wl_output properties don't always make sense for all compositors. Some compositors might not implement a "global compositor space", (e.g. 3D compositors) in which case properties like x and y don't make sense. Some compositors might expose virtual outputs, in which case modes, make and model

Re: [PATCH wayland 4/8] fixup! scanner: use c99 initializers for the request/events arrays

2018-07-03 Thread Emil Velikov
On 3 July 2018 at 12:28, Pekka Paalanen wrote: > On Thu, 14 Jun 2018 16:49:41 +0100 > Emil Velikov wrote: > >> --- >> tests/data/example-code.c | 238 >> >> tests/data/small-code-core.c| 8 +- >> tests/data/small-code.c | 8 +- >>

Re: [PATCH] protocol: warn clients about some wl_output properties

2018-07-03 Thread Philipp Kerling
Hi, one small comment below. 2018-07-03 (火) の 09:01 -0400 に Simon Ser さんは書きました: > All wl_output properties don't always make sense for all > compositors. > > Some compositors might not implement a "global compositor space", > (e.g. 3D compositors) in which case properties like x and y don't >

Re: [PATCH weston] simple-dmabuf-drm: fix build with --disable-egl

2018-07-03 Thread Emil Velikov
Hi Emilio, On 2 July 2018 at 16:22, Emilio Pozuelo Monfort wrote: > Signed-off-by: Emilio Pozuelo Monfort > --- > I tried a build with --disable-egl as I didn't have the headers > installed, and it broke here. The EGL usage here seemed optional so I > did that, but I didn't run-test the result.