2009-03-09 01:03:39

by Łukasz Stelmach

[permalink] [raw]
Subject: [PATCH] use multitouch touchpad events as buttons mousedev

-----BEGIN PGP SIGNED MESSAGE-----
Hash: SHA1

- --- drivers/input/mousedev.c~ 2009-03-08 15:11:11.000000000 +0100
+++ drivers/input/mousedev.c 2009-03-09 00:27:34.000000000 +0100
@@ -316,7 +316,7 @@
wake_up_interruptible(&mousedev->wait);
}

- -static void mousedev_touchpad_touch(struct mousedev *mousedev, int value)
+static void mousedev_touchpad_touch(struct mousedev *mousedev, unsigned int code, int value)
{
if (!value) {
if (mousedev->touch &&
@@ -327,13 +327,23 @@
* We rely on the fact that mousedev_mix always has 0
* motion packet so we won't mess current position.
*/
- - set_bit(0, &mousedev->packet.buttons);
- - set_bit(0, &mousedev_mix->packet.buttons);
+ int index;
+
+ switch (code) {
+ case BTN_TOOL_DOUBLETAP: index = 1; break;
+
+ case BTN_TOOL_TRIPLETAP: index = 2; break;
+
+ default: index = 0;
+ }
+
+ set_bit(index, &mousedev->packet.buttons);
+ set_bit(index, &mousedev_mix->packet.buttons);
mousedev_notify_readers(mousedev, &mousedev_mix->packet);
mousedev_notify_readers(mousedev_mix,
&mousedev_mix->packet);
- - clear_bit(0, &mousedev->packet.buttons);
- - clear_bit(0, &mousedev_mix->packet.buttons);
+ clear_bit(index, &mousedev->packet.buttons);
+ clear_bit(index, &mousedev_mix->packet.buttons);
}
mousedev->touch = mousedev->pkt_count = 0;
mousedev->frac_dx = 0;
@@ -371,7 +381,7 @@
if (value != 2) {
if (code == BTN_TOUCH &&
test_bit(BTN_TOOL_FINGER, handle->dev->keybit))
- - mousedev_touchpad_touch(mousedev, value);
+ mousedev_touchpad_touch(mousedev, code, value);
else
mousedev_key_event(mousedev, code, value);
}

Some touchpads can recognize multi-finger taps. They can be used as
additional buttons (left and middle).

Signed-off-by: Łukasz Stelmach <[email protected]>


Please CC any comments.
- --
Było mi bardzo miło. Czwarta pospolita klęska, [...]
>Łukasz< Już nie katolicka lecz złodziejska. (c)PP
-----BEGIN PGP SIGNATURE-----
Version: GnuPG v2.0.9 (GNU/Linux)
Comment: Using GnuPG with Mozilla - http://enigmail.mozdev.org

iEYEARECAAYFAkm0atoACgkQNdzY8sm9K9z8rQCfQkfJ4xHSEeao6yy9wzz/6zkT
t0kAni6046EHiTmVc7McixxEsrSfJZDS
=w8xZ
-----END PGP SIGNATURE-----


----------------------------------------------------------------------
Udar sloneczny prezesa Kaczynskiego... >>> http://link.interia.pl/f2083


2009-04-16 02:11:42

by Dmitry Torokhov

[permalink] [raw]
Subject: Re: [PATCH] use multitouch touchpad events as buttons mousedev

Hi Lukasz,

On Mon, Mar 09, 2009 at 02:03:22AM +0100, Lukasz Stelmach wrote:
>
> Some touchpads can recognize multi-finger taps. They can be used as
> additional buttons (left and middle).
>

mousedev is a legacy interface and I don't think this change is needed
since evdev + Synaptics X driver combination works much better with
touchpads anyway.

Thanks.

--
Dmitry

2009-04-18 17:53:43

by Łukasz Stelmach

[permalink] [raw]
Subject: Re: [PATCH] use multitouch touchpad events as buttons mousedev

-----BEGIN PGP SIGNED MESSAGE-----
Hash: SHA1

Dmitry Torokhov pisze:
> Hi Lukasz,
>
> On Mon, Mar 09, 2009 at 02:03:22AM +0100, Lukasz Stelmach wrote:
>> Some touchpads can recognize multi-finger taps. They can be used as
>> additional buttons (left and middle).
>>
>
> mousedev is a legacy interface and I don't think this change is needed
> since evdev + Synaptics X driver combination works much better with
> touchpads anyway.

That is true, however, I happen to use plain old virtual terminal with
gpm which uses mousedev. Having that multitouch buttons is, I assure you,
awesome.

- --
Było mi bardzo miło. Czwarta pospolita klęska, [...]
>Łukasz< Już nie katolicka lecz złodziejska. (c)PP
-----BEGIN PGP SIGNATURE-----
Version: GnuPG v2.0.10 (GNU/Linux)
Comment: Using GnuPG with Mozilla - http://enigmail.mozdev.org

iEYEARECAAYFAknqE5EACgkQNdzY8sm9K9xsHwCeJhgMydNi4jw9RfNmu571aya+
lwMAoJJIflw/4kVnJ33G4Xte60MHUqGE
=/lR+
-----END PGP SIGNATURE-----


----------------------------------------------------------------------
Najlepsze dzwonki MP3 na telefon komorkowy!
Sprawdz >> http://link.interia.pl/f2128


Attachments:
stlman.vcf (177.00 B)

2009-04-18 22:13:29

by Dmitry Torokhov

[permalink] [raw]
Subject: Re: [PATCH] use multitouch touchpad events as buttons mousedev

On Sat, Apr 18, 2009 at 07:53:22PM +0200, Lukasz Stelmach wrote:
> -----BEGIN PGP SIGNED MESSAGE-----
> Hash: SHA1
>
> Dmitry Torokhov pisze:
> > Hi Lukasz,
> >
> > On Mon, Mar 09, 2009 at 02:03:22AM +0100, Lukasz Stelmach wrote:
> >> Some touchpads can recognize multi-finger taps. They can be used as
> >> additional buttons (left and middle).
> >>
> >
> > mousedev is a legacy interface and I don't think this change is needed
> > since evdev + Synaptics X driver combination works much better with
> > touchpads anyway.
>
> That is true, however, I happen to use plain old virtual terminal with
> gpm which uses mousedev. Having that multitouch buttons is, I assure you,
> awesome.
>

I see. In this case I suggest switching GPM to use evdev protocol. Since
I added what I cosider proper evdev support for GPM a few years ago it
should work OK without need to patch mousedev.

--
Dmitry