From: Dmitry Tunin <hanipouspilot@gmail.com>
To: Benjamin Tissoires <benjamin.tissoires@gmail.com>
Cc: linux-input <linux-input@vger.kernel.org>,
Mathias Gottschlag <mgottschlag@gmail.com>
Subject: Re: Unusual Focaltech driver behavior
Date: Fri, 17 Apr 2015 17:56:50 +0300 [thread overview]
Message-ID: <55311F32.90300@gmail.com> (raw)
In-Reply-To: <55311D85.2030706@gmail.com>
17.04.2015 17:49, Dmitry Tunin пишет:
> 17.04.2015 16:57, Benjamin Tissoires пишет:
>> On Fri, Apr 17, 2015 at 9:39 AM, Dmitry Tunin <hanipouspilot@gmail.com> wrote:
>>>> Hi Dmitry,
>>>>
>>>> On Fri, Apr 17, 2015 at 5:56 AM, Dmitry Tunin <hanipouspilot@gmail.com> wrote:
>>>>>> I noticed that myself and got some complaints like this.
>>>>>> https://github.com/hanipouspilot/ubuntu-fixes/issues/2
>>>>>>
>>>>>> General issue is that when one finger is on the touchpad, movement of a second finger is ignored, if the first finger does not move.
>>>>>> Usually with other touchpads, when you have one finger on the touchpad and move the other, it is recognized as two-finger scrolling.
>>>>>> The device itself sends relative packages as normal in that case, but linux driver ignores them, until first finger is moved.
>>>>>>
>>>>>> I guess, Windows driver behaves same way. I can't test it, since I do not have Windows installed on that laptop.
>>>>>>
>>>>>> As I understood, Windows driver ignores that one finger is on button area and recognizes movement of the other as one-finger movement.
>>>>>>
>>>>>> It is clear that we do not know the full protocol or parameters of all touchpad models to have that button area always correct.
>>>>>> But it looks like button area is set when 3rd byte in abs package is 00. There is a good chance that it is common for all touchpad models.
>>>>>>
>>>>>> Do you have ideas how to fix it the easiest way?
>>>>>>
>>>>>> Regards,
>>>>>>
>>>>>> Dmitry
>>>>>>
>>>>>
>>>>> I looked at it some more and noticed that if I put one finger on touchpad, then another, then move the second one, rel packets are ignored.
>>>>> But if keeping both fingers on touchpad, I move the first one, rel packets work OK.
>>>>> This is wrong. I can't get what's wrong with the code at the moment.
>>>>
>>>>
>>>> It looks like your touchpad is in the mouse emulation mode, not the
>>>> raw touch mode. What you get is fed by the touchpad FW ans there is
>>>> nothing we can do in userspace to fix that.
>>>> That being said, there has been a lot of work with the focaltech
>>>> drivers in the previous kernel releases, and maybe trying a v4.0 will
>>>> switch your touchpad in the raw mode.
>>>> Once it is in raw mode, the software buttons, scrolling and gestures
>>>> are all treated in userspace and you will get the expected behavior.
>>>>
>>>> Cheers,
>>>> Benjamin
>>>>
>>>
>>> No, the mouse is in proprietary mode. Multitouch is supported. In emulation mode it is not supported at all.
>>> I am testing actually the driver from kernel 4.0. Everything works great except this strange problem, when some relative packets are ignored.
>>> I mentioned above the test case. Now I added some debug and trying to figure it out. But no success so far.
>>>
>>>
>>
>> If you are in absolute (raw) mode, then can you share the evemu-record
>> of the gesture you are trying to support?
>> We should be able to see what is going on and be able to pinpoint if
>> this is a kernel problem or a user space problem.
>>
>> Cheers,
>> Benjamin
>>
>
> Here is the record. I put one finger on the touchpad and moved the other from top to bottom. This gesture is ignored.
> But if I move first finger or both, it is recognized as scrolling
>
> # EVEMU 1.2
> # Input device name: "PS/2 FocalTech FocalTech Touchpad"
> # Input device ID: bus 0x11 vendor 0x02 product 0x01 version 0000
> # Supported events:
> # Event type 0 (EV_SYN)
> # Event code 0 (SYN_REPORT)
> # Event code 1 (SYN_CONFIG)
> # Event code 3 (SYN_MAX)
> # Event type 1 (EV_KEY)
> # Event code 272 (BTN_LEFT)
> # Event code 325 (BTN_TOOL_FINGER)
> # Event code 328 (BTN_TOOL_QUINTTAP)
> # Event code 330 (BTN_TOUCH)
> # Event code 333 (BTN_TOOL_DOUBLETAP)
> # Event code 334 (BTN_TOOL_TRIPLETAP)
> # Event code 335 (BTN_TOOL_QUADTAP)
> # Event type 3 (EV_ABS)
> # Event code 0 (ABS_X)
> # Value 1051
> # Min 0
> # Max 2944
> # Fuzz 0
> # Flat 0
> # Resolution 0
> # Event code 1 (ABS_Y)
> # Value 920
> # Min 0
> # Max 1664
> # Fuzz 0
> # Flat 0
> # Resolution 0
> # Event code 47 (ABS_MT_SLOT)
> # Value 0
> # Min 0
> # Max 4
> # Fuzz 0
> # Flat 0
> # Resolution 0
> # Event code 53 (ABS_MT_POSITION_X)
> # Value 0
> # Min 0
> # Max 2944
> # Fuzz 0
> # Flat 0
> # Resolution 0
> # Event code 54 (ABS_MT_POSITION_Y)
> # Value 0
> # Min 0
> # Max 1664
> # Fuzz 0
> # Flat 0
> # Resolution 0
> # Event code 57 (ABS_MT_TRACKING_ID)
> # Value 0
> # Min 0
> # Max 65535
> # Fuzz 0
> # Flat 0
> # Resolution 0
> # Properties:
> # Property type 0 (INPUT_PROP_POINTER)
> # Property type 2 (INPUT_PROP_BUTTONPAD)
> N: PS/2 FocalTech FocalTech Touchpad
> I: 0011 0002 0001 0000
> P: 05 00 00 00 00 00 00 00
> B: 00 0b 00 00 00 00 00 00 00
> B: 01 00 00 00 00 00 00 00 00
> B: 01 00 00 00 00 00 00 00 00
> B: 01 00 00 00 00 00 00 00 00
> B: 01 00 00 00 00 00 00 00 00
> B: 01 00 00 01 00 00 00 00 00
> B: 01 20 e5 00 00 00 00 00 00
> B: 01 00 00 00 00 00 00 00 00
> B: 01 00 00 00 00 00 00 00 00
> B: 01 00 00 00 00 00 00 00 00
> B: 01 00 00 00 00 00 00 00 00
> B: 01 00 00 00 00 00 00 00 00
> B: 01 00 00 00 00 00 00 00 00
> B: 02 00 00 00 00 00 00 00 00
> B: 03 03 00 00 00 00 80 60 02
> B: 04 00 00 00 00 00 00 00 00
> B: 05 00 00 00 00 00 00 00 00
> B: 11 00 00 00 00 00 00 00 00
> B: 12 00 00 00 00 00 00 00 00
> B: 15 00 00 00 00 00 00 00 00
> B: 15 00 00 00 00 00 00 00 00
> A: 00 0 2944 0 0 0
> A: 01 0 1664 0 0 0
> A: 2f 0 4 0 0 0
> A: 35 0 2944 0 0 0
> A: 36 0 1664 0 0 0
> A: 39 0 65535 0 0 0
> ################################
> # Waiting for events #
> ################################
> E: 0.000000 0003 0039 0279 # EV_ABS / ABS_MT_TRACKING_ID 279
> E: 0.000000 0003 0035 0750 # EV_ABS / ABS_MT_POSITION_X 750
> E: 0.000000 0003 0036 0680 # EV_ABS / ABS_MT_POSITION_Y 680
> E: 0.000000 0001 014a 0001 # EV_KEY / BTN_TOUCH 1
> E: 0.000000 0001 0145 0001 # EV_KEY / BTN_TOOL_FINGER 1
> E: 0.000000 0003 0000 0750 # EV_ABS / ABS_X 750
> E: 0.000000 0003 0001 0680 # EV_ABS / ABS_Y 680
> E: 0.000000 0000 0000 0000 # ------------ SYN_REPORT (0) ----------
> E: 0.160846 0003 0036 0678 # EV_ABS / ABS_MT_POSITION_Y 678
> E: 0.160846 0003 0001 0678 # EV_ABS / ABS_Y 678
> E: 0.160846 0000 0000 0000 # ------------ SYN_REPORT (0) ----------
> E: 0.853482 0003 002f 0001 # EV_ABS / ABS_MT_SLOT 1
> E: 0.853482 0003 0039 0280 # EV_ABS / ABS_MT_TRACKING_ID 280
> E: 0.853482 0003 0035 2238 # EV_ABS / ABS_MT_POSITION_X 2238
> E: 0.853482 0003 0036 0046 # EV_ABS / ABS_MT_POSITION_Y 46
> E: 0.853482 0001 0145 0000 # EV_KEY / BTN_TOOL_FINGER 0
> E: 0.853482 0001 014d 0001 # EV_KEY / BTN_TOOL_DOUBLETAP 1
> E: 0.853482 0000 0000 0000 # ------------ SYN_REPORT (0) ----------
> E: 1.053646 0003 0035 2237 # EV_ABS / ABS_MT_POSITION_X 2237
> E: 1.053646 0003 0036 0049 # EV_ABS / ABS_MT_POSITION_Y 49
> E: 1.053646 0000 0000 0000 # ------------ SYN_REPORT (0) ----------
> E: 1.069850 0003 0036 0053 # EV_ABS / ABS_MT_POSITION_Y 53
> E: 1.069850 0000 0000 0000 # ------------ SYN_REPORT (0) ----------
> E: 1.086012 0003 0035 2240 # EV_ABS / ABS_MT_POSITION_X 2240
> E: 1.086012 0003 0036 0068 # EV_ABS / ABS_MT_POSITION_Y 68
> E: 1.086012 0000 0000 0000 # ------------ SYN_REPORT (0) ----------
> E: 1.102159 0003 0035 2242 # EV_ABS / ABS_MT_POSITION_X 2242
> E: 1.102159 0003 0036 0092 # EV_ABS / ABS_MT_POSITION_Y 92
> E: 1.102159 0000 0000 0000 # ------------ SYN_REPORT (0) ----------
> E: 1.118366 0003 0036 0108 # EV_ABS / ABS_MT_POSITION_Y 108
> E: 1.118366 0000 0000 0000 # ------------ SYN_REPORT (0) ----------
> E: 1.134448 0003 0036 0135 # EV_ABS / ABS_MT_POSITION_Y 135
> E: 1.134448 0000 0000 0000 # ------------ SYN_REPORT (0) ----------
> E: 1.150600 0003 0035 2240 # EV_ABS / ABS_MT_POSITION_X 2240
> E: 1.150600 0003 0036 0200 # EV_ABS / ABS_MT_POSITION_Y 200
> E: 1.150600 0000 0000 0000 # ------------ SYN_REPORT (0) ----------
> E: 1.166755 0003 0035 2236 # EV_ABS / ABS_MT_POSITION_X 2236
> E: 1.166755 0003 0036 0236 # EV_ABS / ABS_MT_POSITION_Y 236
> E: 1.166755 0000 0000 0000 # ------------ SYN_REPORT (0) ----------
> E: 1.182997 0003 0035 2234 # EV_ABS / ABS_MT_POSITION_X 2234
> E: 1.182997 0003 0036 0278 # EV_ABS / ABS_MT_POSITION_Y 278
> E: 1.182997 0000 0000 0000 # ------------ SYN_REPORT (0) ----------
> E: 1.203908 0003 0035 2230 # EV_ABS / ABS_MT_POSITION_X 2230
> E: 1.203908 0003 0036 0326 # EV_ABS / ABS_MT_POSITION_Y 326
> E: 1.203908 0000 0000 0000 # ------------ SYN_REPORT (0) ----------
> E: 1.220070 0003 0035 2228 # EV_ABS / ABS_MT_POSITION_X 2228
> E: 1.220070 0003 0036 0382 # EV_ABS / ABS_MT_POSITION_Y 382
> E: 1.220070 0000 0000 0000 # ------------ SYN_REPORT (0) ----------
> E: 1.236227 0003 0036 0410 # EV_ABS / ABS_MT_POSITION_Y 410
> E: 1.236227 0000 0000 0000 # ------------ SYN_REPORT (0) ----------
> E: 1.252415 0003 0036 0459 # EV_ABS / ABS_MT_POSITION_Y 459
> E: 1.252415 0000 0000 0000 # ------------ SYN_REPORT (0) ----------
> E: 1.268505 0003 0035 2226 # EV_ABS / ABS_MT_POSITION_X 2226
> E: 1.268505 0003 0036 0584 # EV_ABS / ABS_MT_POSITION_Y 584
> E: 1.268505 0000 0000 0000 # ------------ SYN_REPORT (0) ----------
> E: 1.285702 0003 0035 2224 # EV_ABS / ABS_MT_POSITION_X 2224
> E: 1.285702 0003 0036 0666 # EV_ABS / ABS_MT_POSITION_Y 666
> E: 1.285702 0000 0000 0000 # ------------ SYN_REPORT (0) ----------
> E: 1.304459 0003 0035 2222 # EV_ABS / ABS_MT_POSITION_X 2222
> E: 1.304459 0003 0036 0738 # EV_ABS / ABS_MT_POSITION_Y 738
> E: 1.304459 0000 0000 0000 # ------------ SYN_REPORT (0) ----------
> E: 1.322557 0003 002f 0000 # EV_ABS / ABS_MT_SLOT 0
> E: 1.322557 0003 0035 0751 # EV_ABS / ABS_MT_POSITION_X 751
> E: 1.322557 0003 0036 0676 # EV_ABS / ABS_MT_POSITION_Y 676
> E: 1.322557 0003 002f 0001 # EV_ABS / ABS_MT_SLOT 1
> E: 1.322557 0003 0035 2220 # EV_ABS / ABS_MT_POSITION_X 2220
> E: 1.322557 0003 0036 0804 # EV_ABS / ABS_MT_POSITION_Y 804
> E: 1.322557 0003 0000 0751 # EV_ABS / ABS_X 751
> E: 1.322557 0003 0001 0676 # EV_ABS / ABS_Y 676
> E: 1.322557 0000 0000 0000 # ------------ SYN_REPORT (0) ----------
> E: 1.339614 0003 0036 0835 # EV_ABS / ABS_MT_POSITION_Y 835
> E: 1.339614 0000 0000 0000 # ------------ SYN_REPORT (0) ----------
> E: 1.356413 0003 0035 2218 # EV_ABS / ABS_MT_POSITION_X 2218
> E: 1.356413 0003 0036 0940 # EV_ABS / ABS_MT_POSITION_Y 940
> E: 1.356413 0000 0000 0000 # ------------ SYN_REPORT (0) ----------
> E: 1.374034 0003 002f 0000 # EV_ABS / ABS_MT_SLOT 0
> E: 1.374034 0003 0036 0674 # EV_ABS / ABS_MT_POSITION_Y 674
> E: 1.374034 0003 002f 0001 # EV_ABS / ABS_MT_SLOT 1
> E: 1.374034 0003 0036 0979 # EV_ABS / ABS_MT_POSITION_Y 979
> E: 1.374034 0003 0001 0674 # EV_ABS / ABS_Y 674
> E: 1.374034 0000 0000 0000 # ------------ SYN_REPORT (0) ----------
> E: 1.391422 0003 0035 2214 # EV_ABS / ABS_MT_POSITION_X 2214
> E: 1.391422 0003 0036 1098 # EV_ABS / ABS_MT_POSITION_Y 1098
> E: 1.391422 0000 0000 0000 # ------------ SYN_REPORT (0) ----------
> E: 1.408860 0003 0035 2210 # EV_ABS / ABS_MT_POSITION_X 2210
> E: 1.408860 0003 0036 1168 # EV_ABS / ABS_MT_POSITION_Y 1168
> E: 1.408860 0000 0000 0000 # ------------ SYN_REPORT (0) ----------
> E: 1.425074 0003 0035 2206 # EV_ABS / ABS_MT_POSITION_X 2206
> E: 1.425074 0003 0036 1238 # EV_ABS / ABS_MT_POSITION_Y 1238
> E: 1.425074 0000 0000 0000 # ------------ SYN_REPORT (0) ----------
> E: 1.441161 0003 0035 2204 # EV_ABS / ABS_MT_POSITION_X 2204
> E: 1.441161 0003 0036 1312 # EV_ABS / ABS_MT_POSITION_Y 1312
> E: 1.441161 0000 0000 0000 # ------------ SYN_REPORT (0) ----------
> E: 1.457252 0003 0035 2200 # EV_ABS / ABS_MT_POSITION_X 2200
> E: 1.457252 0003 0036 1384 # EV_ABS / ABS_MT_POSITION_Y 1384
> E: 1.457252 0000 0000 0000 # ------------ SYN_REPORT (0) ----------
> E: 1.473426 0003 0035 2198 # EV_ABS / ABS_MT_POSITION_X 2198
> E: 1.473426 0003 0036 1450 # EV_ABS / ABS_MT_POSITION_Y 1450
> E: 1.473426 0000 0000 0000 # ------------ SYN_REPORT (0) ----------
> E: 1.489499 0003 0036 1484 # EV_ABS / ABS_MT_POSITION_Y 1484
> E: 1.489499 0000 0000 0000 # ------------ SYN_REPORT (0) ----------
> E: 1.505633 0003 0036 1532 # EV_ABS / ABS_MT_POSITION_Y 1532
> E: 1.505633 0000 0000 0000 # ------------ SYN_REPORT (0) ----------
> E: 1.521753 0003 0036 1580 # EV_ABS / ABS_MT_POSITION_Y 1580
> E: 1.521753 0000 0000 0000 # ------------ SYN_REPORT (0) ----------
> E: 1.537900 0003 0039 -001 # EV_ABS / ABS_MT_TRACKING_ID -1
> E: 1.537900 0001 0145 0001 # EV_KEY / BTN_TOOL_FINGER 1
> E: 1.537900 0001 014d 0000 # EV_KEY / BTN_TOOL_DOUBLETAP 0
> E: 1.537900 0000 0000 0000 # ------------ SYN_REPORT (0) ----------
> E: 2.342684 0003 002f 0000 # EV_ABS / ABS_MT_SLOT 0
> E: 2.342684 0003 0039 -001 # EV_ABS / ABS_MT_TRACKING_ID -1
> E: 2.342684 0001 014a 0000 # EV_KEY / BTN_TOUCH 0
> E: 2.342684 0001 0145 0000 # EV_KEY / BTN_TOOL_FINGER 0
> E: 2.342684 0000 0000 0000 # ------------ SYN_REPORT (0) ----------
>
In addition. I grab packages through serio_raw and see that in both cases packages are sent normally form the device.
This must be something with state->finger[i].active or valid. But I added debug to the code and can't catch any problem there either.
--
To unsubscribe from this list: send the line "unsubscribe linux-input" in
the body of a message to majordomo@vger.kernel.org
More majordomo info at http://vger.kernel.org/majordomo-info.html
next prev parent reply other threads:[~2015-04-17 14:56 UTC|newest]
Thread overview: 10+ messages / expand[flat|nested] mbox.gz Atom feed top
2015-04-17 8:44 Unusual Focaltech driver behavior Dmitry Tunin
2015-04-17 9:56 ` Dmitry Tunin
2015-04-17 13:33 ` Benjamin Tissoires
2015-04-17 13:39 ` Dmitry Tunin
2015-04-17 13:57 ` Benjamin Tissoires
2015-04-17 14:49 ` Dmitry Tunin
2015-04-17 14:56 ` Dmitry Tunin [this message]
2015-04-17 15:43 ` Benjamin Tissoires
2015-04-17 15:54 ` Dmitry Tunin
2015-04-21 5:35 ` Peter Hutterer
Reply instructions:
You may reply publicly to this message via plain-text email
using any one of the following methods:
* Save the following mbox file, import it into your mail client,
and reply-to-all from there: mbox
Avoid top-posting and favor interleaved quoting:
https://en.wikipedia.org/wiki/Posting_style#Interleaved_style
* Reply using the --to, --cc, and --in-reply-to
switches of git-send-email(1):
git send-email \
--in-reply-to=55311F32.90300@gmail.com \
--to=hanipouspilot@gmail.com \
--cc=benjamin.tissoires@gmail.com \
--cc=linux-input@vger.kernel.org \
--cc=mgottschlag@gmail.com \
/path/to/YOUR_REPLY
https://kernel.org/pub/software/scm/git/docs/git-send-email.html
* If your mail client supports setting the In-Reply-To header
via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line
before the message body.
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox;
as well as URLs for NNTP newsgroup(s).