For the last month or so I've been spending some time on a helper library
for evdev devices. The motivation is two-fold:
* users of evdev currently issue ioctls directly, checking bit masks
manually. all this is a source for error, especially as there are some
inconsistencies in the ioctl
On Thu, Jun 27, 2013 at 11:15:03AM -0300, Wander Lairson Costa wrote:
2013/6/27 Peter Hutterer peter.hutte...@who-t.net:
For the last month or so I've been spending some time on a helper library
for evdev devices. The motivation is two-fold:
* users of evdev currently issue ioctls
On Thu, Jun 27, 2013 at 01:06:24PM -0400, Todd Showalter wrote:
On Thu, Jun 27, 2013 at 12:52 AM, Peter Hutterer
peter.hutte...@who-t.net wrote:
For the last month or so I've been spending some time on a helper library
for evdev devices.
Looks nice!
I assume this doesn't
[Note: bcc'd linux-input, wayland-devel and xorg-devel since those are the
expected interested parties. Apologies if you get this email multiple times,
I won't do this in the future. Future announcements will only go to
input-to...@lists.freedesktop.org]
This is the first release for libevdev, a
The current code works if pw-pw_shell is bash because:
If the shell is started with the effective user (group) id not equal to
the real user (group) id, and the -p option is not supplied, these actions
are taken and the effective user id is set to the real user id.
Thus, for bash, weston's
initgroups requires CAP_SETGID and may fail. If we're not actually
changing anything anyway we can just skip this and (most likely) fail later
when we don't have the required permissions to open something.
---
Should arguably be part of the previous patch but since it changes behaviour
from before
---
src/tty.c | 2 +-
1 file changed, 1 insertion(+), 1 deletion(-)
diff --git a/src/tty.c b/src/tty.c
index 38079a4..0ddf6e0 100644
--- a/src/tty.c
+++ b/src/tty.c
@@ -175,7 +175,7 @@ tty_create(struct weston_compositor *compositor,
tty_vt_func_t vt_func,
if (tty-starting_vt != tty-vt)
---
src/evdev.c | 1 +
1 file changed, 1 insertion(+)
diff --git a/src/evdev.c b/src/evdev.c
index d7e416a..96f3085 100644
--- a/src/evdev.c
+++ b/src/evdev.c
@@ -65,6 +65,7 @@ evdev_led_update(struct evdev_device *device, enum weston_led
leds)
static inline void
evdev_process_key(struct
---
src/evdev.c | 3 +--
1 file changed, 1 insertion(+), 2 deletions(-)
diff --git a/src/evdev.c b/src/evdev.c
index 96f3085..4f3f8cf 100644
--- a/src/evdev.c
+++ b/src/evdev.c
@@ -553,10 +553,9 @@ evdev_device_create(struct weston_seat *seat, const char
*path, int device_fd)
struct
mtdev as currently used in weston is a noop. mtdev's purpose is to convert
Protocol A devices (without ABS_MT_SLOT) to Protocol B devices (slots).
For Protocol B devices mtdev merely routes the events, so checking for
slots and then using mtdev based on that adds no functionality.
Check for
For Protocol B devices, mtdev merely routes the events and is not needed.
For Protocol A devices, mtdev is needed, so fail for those devices now if we
mtdev fails.
---
src/evdev.c | 20 +---
1 file changed, 13 insertions(+), 7 deletions(-)
diff --git a/src/evdev.c b/src/evdev.c
If touches are already present on the device, absinfo has the currently
active touch slot. There's a race condition where the slot may change before
we enable the fd and we thus miss out on the ABS_MT_SLOT event. It's still
slightly more correct than assuming whatever comes next is slot 0.
---
For touchpads, device-dispatch is set up when exiting
evdev_handle_device() and a potential source for a memleak.
This can't actually happen at the moment, as evdev_handle_device() won't
fail for touchpads after setting up the dispatch but prevent this from
happening in the future.
---
Avoid keeping keeping what needs to be freed in sync in multiple places,
make evdev_device_destroy do the right thing instead.
---
src/evdev.c | 30 ++
1 file changed, 10 insertions(+), 20 deletions(-)
diff --git a/src/evdev.c b/src/evdev.c
index bd3b025..7bacbd9
wayland-scanner without arguments prints out usage. With help or --help it
waits for stdin to supply something which isn't quite as informative as
printing out the help.
This patch also moves the strcmp for args up to have all of them in one
location.
---
src/scanner.c | 29
---
src/evdev.c | 5 -
1 file changed, 4 insertions(+), 1 deletion(-)
diff --git a/src/evdev.c b/src/evdev.c
index 7bacbd9..ab35582 100644
--- a/src/evdev.c
+++ b/src/evdev.c
@@ -495,8 +495,11 @@ evdev_handle_device(struct evdev_device *device)
key_bits);
Signed-off-by: Peter Hutterer peter.hutte...@who-t.net
---
src/scanner.c | 4
1 file changed, 4 insertions(+)
diff --git a/src/scanner.c b/src/scanner.c
index ace6633..5124000 100644
--- a/src/scanner.c
+++ b/src/scanner.c
@@ -34,6 +34,10 @@ static int
usage(int ret)
{
fprintf
If wayland-scanner.pc can't be found the variables end up being set
irrespectively, leaving the user with odd compiler errors about missing
headers, etc.
---
wayland-scanner.m4 | 2 ++
1 file changed, 2 insertions(+)
diff --git a/wayland-scanner.m4 b/wayland-scanner.m4
index 2b87c5f..4e4222a
On Wed, Aug 07, 2013 at 07:46:43AM +0200, Daiki Ueno wrote:
Hi,
Peter Hutterer peter.hutte...@who-t.net writes:
AC_DEFUN([WAYLAND_SCANNER_RULES], [
PKG_PROG_PKG_CONFIG
+PKG_CHECK_MODULES([WAYLAND_SCANNER], [wayland-scanner])
+
With this change, configure will abort
Same as calloc(1, len).
---
Feel free to squash this with the follow-up, I just kept it separately to
make changes easier, the second one is just search/replace.
shared/zalloc.h | 40
src/Makefile.am | 1 +
src/compositor.h | 1 +
3 files changed, 42
And for clients using the xmalloc helper, use xzalloc.
Signed-off-by: Peter Hutterer peter.hutte...@who-t.net
---
clients/clickdot.c| 4 +---
clients/cliptest.c| 4 +---
clients/desktop-shell.c | 16 +---
clients/dnd.c | 4
mtdev as currently used in weston is a noop. mtdev's purpose is to convert
Protocol A devices (without ABS_MT_SLOT) to Protocol B devices (slots).
For Protocol B devices mtdev merely routes the events, so checking for
slots and then using mtdev based on that adds no functionality.
Check for
---
src/tty.c | 2 +-
1 file changed, 1 insertion(+), 1 deletion(-)
diff --git a/src/tty.c b/src/tty.c
index 07a9405..212deb0 100644
--- a/src/tty.c
+++ b/src/tty.c
@@ -174,7 +174,7 @@ tty_create(struct weston_compositor *compositor,
tty_vt_func_t vt_func,
if (tty-starting_vt != tty-vt)
_GNU_SOURCE is always true as of c228e23b055f54d29f2df, so
program_invocation_short_name is available.
---
clients/window.c | 2 +-
1 file changed, 1 insertion(+), 1 deletion(-)
diff --git a/clients/window.c b/clients/window.c
index 9ec9786..cbf8bf5 100644
--- a/clients/window.c
+++
On Thu, Aug 08, 2013 at 02:17:45PM -0700, Kristian Høgsberg wrote:
On Tue, Aug 06, 2013 at 03:09:10PM +1000, Peter Hutterer wrote:
initgroups requires CAP_SETGID and may fail. If we're not actually
changing anything anyway we can just skip this and (most likely) fail later
when we don't
---
That's a slightly nicer error message than breaking during configure.
configure.ac | 2 ++
1 file changed, 2 insertions(+)
diff --git a/configure.ac b/configure.ac
index f9f1c53..3ad1995 100644
--- a/configure.ac
+++ b/configure.ac
@@ -396,6 +396,8 @@ if test x$have_lcms = xyes; then
fi
Setting ACLOCAL only means autoreconf will succeed, but any re-build
through the 'missing' script won't pick up the macro paths. Set ACLOCAL_PATH
too, so that touching configure.ac or Makefile.am won't fail for missing
macros.
---
building.html | 5 +++--
1 file changed, 3 insertions(+), 2
On Mon, Aug 12, 2013 at 06:15:43PM +0200, David Herrmann wrote:
Hi
On Mon, Aug 5, 2013 at 12:34 PM, Alexander E. Patrakov
patra...@gmail.com wrote:
This patch series adds support to weston for a special type of touchpads
found in some laptops. These touchpads contain one physical button
When generating HTML, don't split once we're into subjections. This
generates a single page for each protocol interface instead of the previous
separate pages for requests, events and enums.
No effect on the rest of the HTML configuration.
---
doc/publican/publican.cfg | 1 +
1 file changed, 1
On Thu, Aug 15, 2013 at 02:48:48PM -0700, Kristian Høgsberg wrote:
On Thu, Aug 15, 2013 at 12:10:30PM +0100, Daniel Stone wrote:
Hi,
On 15 August 2013 11:52, Peter Hutterer peter.hutte...@who-t.net wrote:
one of the things that should be done is to figure out _where_ features
check_PROGRAMS and friends are only built during make check. Which is a
great way of introducing compiler errors in tests. Always build them, TESTS
defines what's being run during make check.
---
tests/Makefile.am | 12
1 file changed, 4 insertions(+), 8 deletions(-)
diff --git
Slightly more readable and makes it easier to switch to use $2 for something
in the future (if that's ever needed).
---
tests/weston-tests-env | 17 -
1 file changed, 12 insertions(+), 5 deletions(-)
diff --git a/tests/weston-tests-env b/tests/weston-tests-env
index
---
tests/weston-test-runner.c | 1 +
1 file changed, 1 insertion(+)
diff --git a/tests/weston-test-runner.c b/tests/weston-test-runner.c
index 27ea9e4..7cc1cbe 100644
--- a/tests/weston-test-runner.c
+++ b/tests/weston-test-runner.c
@@ -20,6 +20,7 @@
* OF THIS SOFTWARE.
*/
+#include
Including listing the tests available in that binary
---
tests/weston-test-runner.c | 18 ++
1 file changed, 18 insertions(+)
diff --git a/tests/weston-test-runner.c b/tests/weston-test-runner.c
index 7cc1cbe..fefb93b 100644
--- a/tests/weston-test-runner.c
+++
---
tests/weston-test-runner.c | 1 +
1 file changed, 1 insertion(+)
diff --git a/tests/weston-test-runner.c b/tests/weston-test-runner.c
index fefb93b..ed5baf0 100644
--- a/tests/weston-test-runner.c
+++ b/tests/weston-test-runner.c
@@ -82,6 +82,7 @@ int main(int argc, char *argv[])
to run them.
Sam (and I) argue that if you always build them you're increasing the
likelyhood of someone actually running them.
On Wed, Sep 11, 2013 at 4:53 PM, Peter Hutterer wrote:
from my experience, every project I've worked on that has a test
suite needed this patch eventually, there's
---
clients/Makefile.am | 21 +
1 file changed, 9 insertions(+), 12 deletions(-)
diff --git a/clients/Makefile.am b/clients/Makefile.am
index 87b3a0e..24c6489 100644
--- a/clients/Makefile.am
+++ b/clients/Makefile.am
@@ -2,20 +2,17 @@ bin_PROGRAMS =
---
clients/simple-touch.c | 2 --
1 file changed, 2 deletions(-)
diff --git a/clients/simple-touch.c b/clients/simple-touch.c
index 69e460c..b5a84d7 100644
--- a/clients/simple-touch.c
+++ b/clients/simple-touch.c
@@ -301,8 +301,6 @@ touch_create(int width, int height)
exit(1);
Instead of NULL, a client that doesn't implement all callbacks of a listener
can use named defines to increase readability of the code. For example:
const struct wl_pointer_listener pointer_listener = {
WL_POINTER_ENTER_HANDLER_NONE,
WL_POINTER_LEAVE_HANDLER_NONE,
The scanner is not very forgiving if the protocol doesn't match it's
expectations and crashes without much of a notice. Thus, validate the protocol
against a DTD.
Move the protocol subdir forward so we validate first before trying anything
else, and install the DTD so we can validate weston's
I've been working on a protocol extension to support graphics tablets such
as the Wacom set of tablets, and I'm now at the stage where I'd like a few
comments. I was hoping that I'd get a full implementation before XDC but
unfortunately that didn't happen, so for now I'll just show the protocol.
On Wed, Oct 02, 2013 at 05:44:29PM +0200, David Herrmann wrote:
Hi Peter
On Fri, Sep 20, 2013 at 12:35 PM, Peter Hutterer
peter.hutte...@who-t.net wrote:
I've been working on a protocol extension to support graphics tablets such
as the Wacom set of tablets, and I'm now at the stage where
The scanner is not very forgiving if the protocol doesn't match it's
expectations and crashes without much of a notice. Thus, validate the protocol
against a DTD.
Move the protocol subdir forward so we validate first before trying anything
else, and install the DTD so we can validate weston's
---
configure.ac | 11 +++
protocol/Makefile.am | 17 -
2 files changed, 27 insertions(+), 1 deletion(-)
diff --git a/configure.ac b/configure.ac
index cfd4540..2d56444 100644
--- a/configure.ac
+++ b/configure.ac
@@ -474,6 +474,17 @@ if test x$wayland_scanner =
Hi Jonas,
On Tue, Nov 12, 2013 at 10:50:56PM +0100, Jonas Ådahl wrote:
Wayland compositors are expected to deal with input device processing
themselves providing input events according to the Wayland protocol to
the clients. So far only weston has had more than basic support for
processing
On Wed, Nov 13, 2013 at 01:22:26PM +0100, Jonas Ådahl wrote:
On Wed, Nov 13, 2013 at 7:22 AM, Peter Hutterer
peter.hutte...@who-t.net wrote:
Hi Jonas,
On Tue, Nov 12, 2013 at 10:50:56PM +0100, Jonas Ådahl wrote:
Wayland compositors are expected to deal with input device processing
On Wed, Nov 27, 2013 at 02:16:55PM -0800, Kristian Høgsberg wrote:
On Tue, Nov 26, 2013 at 02:25:42PM +0100, Jonny Lamb wrote:
Hi.
I have recently been working on some Wayland protocol stuff and found
referring to the XML tiresome. Telepathy's D-Bus protocol has a parser
which
Signed-off-by: Peter Hutterer peter.hutte...@who-t.net
---
clients/terminal.c |7 ++-
1 files changed, 6 insertions(+), 1 deletions(-)
diff --git a/clients/terminal.c b/clients/terminal.c
index e5baa0e..2cb1e58 100644
--- a/clients/terminal.c
+++ b/clients/terminal.c
@@ -2370,6 +2370,7
On Tue, Feb 21, 2012 at 04:25:39PM -0500, Kristian Høgsberg wrote:
2012/2/21 Chase Douglas chase.doug...@canonical.com:
On 02/21/2012 09:16 PM, Kristian Høgsberg wrote:
2012/2/20 Chase Douglas chase.doug...@canonical.com:
On 02/17/2012 06:01 PM, Kristian Høgsberg wrote:
- input protocol
Signed-off-by: Peter Hutterer peter.hutte...@who-t.net
---
spec/main.tex |6 +++---
1 files changed, 3 insertions(+), 3 deletions(-)
diff --git a/spec/main.tex b/spec/main.tex
index 59d4fe0..b6fcc4e 100644
--- a/spec/main.tex
+++ b/spec/main.tex
@@ -125,9 +125,9 @@ the type of event. Events
scanner.c: In function ‘desc_dump’:
scanner.c:142:42: warning: unused variable ‘len’ [-Wunused-variable]
Signed-off-by: Peter Hutterer peter.hutte...@who-t.net
---
src/scanner.c |2 +-
1 files changed, 1 insertions(+), 1 deletions(-)
diff --git a/src/scanner.c b/src/scanner.c
index 699782f
Signed-off-by: Peter Hutterer peter.hutte...@who-t.net
---
protocol/wayland.xml |2 +-
1 files changed, 1 insertions(+), 1 deletions(-)
diff --git a/protocol/wayland.xml b/protocol/wayland.xml
index a40e4b0..b274354 100644
--- a/protocol/wayland.xml
+++ b/protocol/wayland.xml
@@ -34,7 +34,7
Includes rudimentary styling only.
Signed-off-by: Peter Hutterer peter.hutte...@who-t.net
---
A few things to note:
- I'm not a designer
- Having a html version of the protocol makes it a lot easier to read, and
it certainly reveals missing bits of documentation in the protocol
- the .css file
On 23/02/12 19:57 , Tiago Vignatti wrote:
On 02/23/2012 07:58 AM, Peter Hutterer wrote:
Includes rudimentary styling only.
Signed-off-by: Peter Huttererpeter.hutte...@who-t.net
---
A few things to note:
- I'm not a designer
- Having a html version of the protocol makes it a lot easier to read
On Thu, Feb 23, 2012 at 04:26:23PM -0500, Kristian Hoegsberg wrote:
On Thu, Feb 23, 2012 at 03:58:37PM +1000, Peter Hutterer wrote:
Includes rudimentary styling only.
Signed-off-by: Peter Hutterer peter.hutte...@who-t.net
---
A few things to note:
- I'm not a designer
- Having
On 24/02/12 20:15 , Michael Hasselmann wrote:
On Fri, 2012-02-24 at 17:42 +1000, Peter Hutterer wrote:
I've played around a bit today and the result is this:
http://people.freedesktop.org/~whot/Wayland/tmp/en-US/html/
The blue text for type names etc. is a bit confusing, in the sense
On 25/02/12 01:04 , Kristian Høgsberg wrote:
On Fri, Feb 24, 2012 at 2:42 AM, Peter Hutterer
peter.hutte...@who-t.net wrote:
On Thu, Feb 23, 2012 at 04:26:23PM -0500, Kristian Hoegsberg wrote:
On Thu, Feb 23, 2012 at 03:58:37PM +1000, Peter Hutterer wrote:
Includes rudimentary styling only
Distribute all source files that we need for buildling.
Plus, remove the html file on make clean.
Signed-off-by: Peter Hutterer peter.hutte...@who-t.net
---
protocol/Makefile.am |7 ---
1 files changed, 4 insertions(+), 3 deletions(-)
diff --git a/protocol/Makefile.am b/protocol
I'll leave them in for now as a template for how things looked originally,
this can be removed later.
Signed-off-by: Peter Hutterer peter.hutte...@who-t.net
---
doc/Wayland/en-US/Wayland.xml |5 +
1 files changed, 5 insertions(+), 0 deletions(-)
diff --git a/doc/Wayland/en-US
automake doesn't seem to provide a sensible method to install a directory of
stuff in $(docdir). Do it manually then.
CC: Gaetan Nadon gaetan.na...@videotron.ca
Signed-off-by: Peter Hutterer peter.hutte...@who-t.net
---
Gaetan - can you check this please, it seems less-than-ideal, especially
Convert the wayland.xml protocol description to a docbook-compatible format
and hook it up to the publican sources.
Signed-off-by: Peter Hutterer peter.hutte...@who-t.net
---
doc/Wayland/.gitignore |1 +
doc/Wayland/Makefile.am |9 ++-
doc/Wayland/en-US
-only source tree, so we
can't just link).
CC: Gaetan Nadon gaetan.na...@videotron.ca
Signed-off-by: Peter Hutterer peter.hutte...@who-t.net
---
Gaetan, if you have any better suggestions I'd love to hear them.
doc/Wayland/Makefile.am | 22 ++
1 files changed, 18 insertions
On Thu, Mar 01, 2012 at 04:17:16PM -0500, Kristian Høgsberg wrote:
On Wed, Feb 29, 2012 at 1:46 PM, Kristian Hoegsberg hoegsb...@gmail.com
wrote:
On Wed, Feb 29, 2012 at 12:57:28PM +1000, Peter Hutterer wrote:
Branch available from
git://people.freedesktop.org/~whot/wayland.git
On Thu, Mar 01, 2012 at 10:32:28AM -0500, Gaetan Nadon wrote:
On 12-02-29 07:40 PM, Peter Hutterer wrote:
+# Copy the en_US source files into en-US destination
+# This is required for out-of-source-tree build as publican does not
allow us
+# to specify the location of the source code
Signed-off-by: Tiago Vignatti tiago.vigna...@intel.com
Reviewed-by: Peter Hutterer peter.hutte...@who-t.net
---
src/evdev.c |2 +-
1 files changed, 1 insertions(+), 1 deletions(-)
diff --git a/src/evdev.c b/src/evdev.c
index 7bd5059..5a3c748 100644
--- a/src/evdev.c
+++ b/src/evdev.c
directly depends on such library.
Signed-off-by: Tiago Vignatti tiago.vigna...@intel.com
---
FYI, simple-touch is having some delay effects for painting the fingers on
screen after the wait-for-input-when-repaint work was committed.
Reviewed-by: Peter Hutterer peter.hutte...@who-t.net
This moves the documentation into the docbook sources, removing the need for
the .tex file. Because the current docbook also includes some parts copied
from the website, the first three chapters are somewhat redundant.
Cheers,
Peter
___
wayland-devel
The former is the actual specification, the latter notes about the protocol.
Signed-off-by: Peter Hutterer peter.hutte...@who-t.net
---
doc/Wayland/Makefile.am | 12 +-
doc/Wayland/en_US/Protocol.xml| 684 +
doc/Wayland/en_US/Wayland.xml
2 spaces is enough for xml, otherwise we end up with too little room for the
actual text.
Signed-off-by: Peter Hutterer peter.hutte...@who-t.net
---
doc/Wayland/en_US/Architecture.xml | 616 ++--
doc/Wayland/en_US/Author_Group.xml | 16 +-
doc/Wayland/en_US
On Thu, Mar 29, 2012 at 01:34:40PM +0200, Andreas Ericsson wrote:
On 03/29/2012 02:50 AM, Peter Hutterer wrote:
- Implementation section skipped, this seems in need of an update anyway
and may be better documented elsewhere (wiki?)
Why not just use doxygen to document the api
On Thu, May 10, 2012 at 07:13:09PM +0100, Daniel Stone wrote:
Hi,
On 10 May 2012 17:45, Christopher James Halse Rogers
christopher.halse.rog...@canonical.com wrote:
Is Weston going to essentially fold in all the interesting bits from all
the input DDXs to the core (and require this to be
On Thu, May 10, 2012 at 10:59:17PM +0100, Daniel Stone wrote:
Hi,
On 10 May 2012 22:11, Kristian Høgsberg hoegsb...@gmail.com wrote:
I agree that we need to get evdev a little more under control, but I
was thinking that we'd be able to decide up front that a device is a
touchpad, a mouse
On Sat, May 12, 2012 at 01:16:38PM +0300, Pekka Paalanen wrote:
[snip]
Another drawback of Wayland is, that the compositor is also
responsible for reading and dispatching input events. If there's
a new class of input device _all_ compositors must be adjusted.
You could of course invent
Jonas posted some touchpad support patches earlier this month and
there were some calls for a input handling module. I've hacked around for a
few days to get the synaptics driver ready for wayland.
This is not a wayland touchpad driver! It's the X11 driver, abstracted
enough that a wayland
On 23/05/12 17:16 , Juan Zhao wrote:
On 05/23/2012 01:31 PM, Peter Hutterer wrote:
Jonas posted some touchpad support patches earlier this month and
there were some calls for a input handling module. I've hacked around
for a
few days to get the synaptics driver ready for wayland
On Wed, May 30, 2012 at 04:31:48PM +0100, Daniel Stone wrote:
Does what it says on the box: lists whether or not the device supports
key, absolute, relative or touch classes.
Signed-off-by: Daniel Stone dan...@fooishbar.org
---
src/evdev-private.h |9 +
src/evdev.c |
-by: Peter Hutterer peter.hutte...@who-t.net
---
test/litest.c | 5 +
test/litest.h | 18 +-
2 files changed, 14 insertions(+), 9 deletions(-)
diff --git a/test/litest.c b/test/litest.c
index 00db464..0341e9a 100644
--- a/test/litest.c
+++ b/test/litest.c
@@ -157,6 +157,9
No functional changes, just prep work for an upcoming patch
Signed-off-by: Peter Hutterer peter.hutte...@who-t.net
---
src/evdev.c | 13 +++--
src/evdev.h | 2 ++
2 files changed, 13 insertions(+), 2 deletions(-)
diff --git a/src/evdev.c b/src/evdev.c
index b9f635d..8b154a3 100644
Signed-off-by: Peter Hutterer peter.hutte...@who-t.net
---
test/device.c | 149 ++
1 file changed, 149 insertions(+)
diff --git a/test/device.c b/test/device.c
index 25e119e..2367fd6 100644
--- a/test/device.c
+++ b/test/device.c
@@ -30,6
Signed-off-by: Peter Hutterer peter.hutte...@who-t.net
---
test/Makefile.am | 7 +-
test/device.c| 250 +++
2 files changed, 256 insertions(+), 1 deletion(-)
create mode 100644 test/device.c
diff --git a/test/Makefile.am b/test
We only called this function before device removal, so failing to update the
button state didn't matter. To make this function generic for the upcoming
device suspend/resume, we need to keep track of the button/key count properly.
Signed-off-by: Peter Hutterer peter.hutte...@who-t.net
---
src
the release
event through the normal channels.
Don't assert if we're already on a press-count of 0, simply ignore it and
discard the event.
Signed-off-by: Peter Hutterer peter.hutte...@who-t.net
---
I'm not 100% sure on this one. It's the easy solution, but until we know the
general direction I didn't want
Does what it says on the box, preventing events from the device without actually
dropping the device from the context.
Signed-off-by: Peter Hutterer peter.hutte...@who-t.net
---
See my notes in the coverletter. For the T440 case I specifically did not
mention that _suspend() closes the file
was removed and libinput_dispatch() was
called, we can short-cut the syspath comparison by setting it to NULL.
Signed-off-by: Peter Hutterer peter.hutte...@who-t.net
---
src/evdev.c | 48 +++-
src/evdev.h | 4 +++-
src/path.c | 9
On Thu, Aug 21, 2014 at 10:16:49PM +0200, Jonas Ådahl wrote:
On Thu, Aug 21, 2014 at 04:18:41PM +1000, Peter Hutterer wrote:
replying to myself, now that I've had a bit of a think about this all.
On Wed, Aug 20, 2014 at 01:18:48PM +1000, Peter Hutterer wrote:
This patchset adds two new
On Thu, Aug 21, 2014 at 09:36:57PM +0200, Jonas Ådahl wrote:
On Wed, Aug 20, 2014 at 01:18:51PM +1000, Peter Hutterer wrote:
Does what it says on the box, preventing events from the device without
actually
dropping the device from the context.
Signed-off-by: Peter Hutterer peter.hutte
On Thu, Aug 21, 2014 at 06:35:57PM -0700, Bill Spitzak wrote:
Seems to me you only need two states: what you call disabled and what you
are calling smart disable.
The fully disabled is so a button the hardware does not know about can
disable the device. Or a wayland compositor could use a
:54 PM, Peter Hutterer wrote:
On Thu, Aug 21, 2014 at 06:35:57PM -0700, Bill Spitzak wrote:
Seems to me you only need two states: what you call disabled and what
you
are calling smart disable.
The fully disabled is so a button the hardware does not know about can
disable the device. Or a wayland
On Fri, Aug 22, 2014 at 06:50:04PM -0700, Bill Spitzak wrote:
On 08/22/2014 06:16 PM, Peter Hutterer wrote:
My main point is I don't think you need to distinguish enabled from
smart enabled. The smartness is in the device driver and could be
considered part of how it generates events
On Mon, Aug 25, 2014 at 10:23:09AM -0700, Bill Spitzak wrote:
On 08/24/2014 05:44 PM, Peter Hutterer wrote:
this is *not* about palm detection. palm detection is already always enabled
where possible and with the parameters sensible for that specific touchpad.
This option
On Fri, Aug 22, 2014 at 07:08:59PM +0300, Pekka Paalanen wrote:
Hi,
here is the alpha release on our road to 1.6.0:
0b0513706eee5a43d33f5e4bc9cafa5d78037db5 wayland-1.5.91.tar.xz
6d0f298665e0f6b11a18ab6b6ccc49ba990b4b3e wayland 1.5.91 tag
59b3f1fa9456ed9cf6b21201647ce3a96c91e03b
We have a wrapper, use it.
Signed-off-by: Peter Hutterer peter.hutte...@who-t.net
---
src/udev-seat.c | 27 ++-
1 file changed, 14 insertions(+), 13 deletions(-)
diff --git a/src/udev-seat.c b/src/udev-seat.c
index 8d19894..2947e71 100644
--- a/src/udev-seat.c
+++ b/src
WL_CALIBRATION, introduced in weston-1.1, requires the translation component
of the calibration matrix to be in screen coordinates. libinput does not have
access to this and it's not a very generic way to do this anyway. So with
the libinput backend, WL_CALIBRATION support is currently broken
Signed-off-by: Peter Hutterer peter.hutte...@who-t.net
---
src/evdev.c | 3 ++-
src/evdev.h | 3 ++-
2 files changed, 4 insertions(+), 2 deletions(-)
diff --git a/src/evdev.c b/src/evdev.c
index b09bb98..9c53e32 100644
--- a/src/evdev.c
+++ b/src/evdev.c
@@ -976,7 +976,8
more explicit.
Signed-off-by: Peter Hutterer peter.hutte...@who-t.net
---
Not sure if we want to do this. I'm somewhat tempted to push all WL_*
property parsing to the compositor and _not_ have a back-channel for
configuration in libinput.
As our brief stint with HAL in Xorg has shown, shoving
for
users to figure it out.
This changes the definition of the WL_CALIBRATION property (which is currently
broken).
Signed-off-by: Peter Hutterer peter.hutte...@who-t.net
---
src/evdev.c| 71 +++---
src/evdev.h| 2 +-
src/libinput.h | 26
litest_wait_for_event() returns if any event is available.
litest_wait_for_event_of_type(... type, type, type, -1) returns if any of the
given event types is availble. All other events are discarded.
Signed-off-by: Peter Hutterer peter.hutte...@who-t.net
---
test/litest.c | 46
Signed-off-by: Peter Hutterer peter.hutte...@who-t.net
---
See patch 08/11, not sure we really want that configuration back-channel.
src/evdev.c | 19 ---
src/evdev.h | 4
src/udev-seat.c | 2 +-
3 files changed, 17 insertions(+), 8 deletions(-)
diff --git a/src
With a non-zero absmin for both axes and different ranges for x/y, just to
detect those errors.
Signed-off-by: Peter Hutterer peter.hutte...@who-t.net
---
test/Makefile.am | 1 +
test/litest-generic-singletouch.c | 88 +++
test/litest.c
This series fixes calibration support in libinput.
The two simple changes: calibration is now applied to multitouch events as
well, and instead of libinput_device_calibrate() there is the quartett of
configuration hooks:
libinput_device_config_calibration_has/set/get/get_default_matrix()
401 - 500 of 3195 matches
Mail list logo