From: Ping Cheng <pinglinux@gmail.com>
To: Henrik Rydberg <rydberg@euromail.se>
Cc: Dmitry Torokhov <dmitry.torokhov@gmail.com>,
Chris Bagwell <chris@cnpbagwell.com>,
linux-input@vger.kernel.org
Subject: Re: [PATCH 4/5] input: wacom: Add support for the Bamboo Touch trackpad (rev4)
Date: Wed, 13 Oct 2010 13:15:54 -0700 [thread overview]
Message-ID: <AANLkTi=RZ5OmBBOV2=kYJ+wE2GD1N9ZKMNyKGVvFqwiR@mail.gmail.com> (raw)
In-Reply-To: <4CB5F8B3.6010504@euromail.se>
On Wed, Oct 13, 2010 at 11:21 AM, Henrik Rydberg <rydberg@euromail.se> wrote:
>
> In the language of ABS_MT, there is a distinction between slot and id. The slot
> is the handle used to communicate a unique touch. The id is the identifier of
> that touch. The device tells us how many slots we have. The range of ids is in
> principle infinite. In practice, it is set to a large number.
>
> To answer a possible follow-up question: from the tracking id, you can tell if a
> contact is present, if it is new, and if it is older than another contact.
I see. Somehow I remember tracking ID is the ID from the hardware/firmware.
>>> +
>>> + input_report_key(input, BTN_LEFT, (data[1] & 0x08) != 0);
>>> + input_report_key(input, BTN_FORWARD, (data[1] & 0x04) != 0);
>>> + input_report_key(input, BTN_BACK, (data[1] & 0x02) != 0);
>>> + input_report_key(input, BTN_RIGHT, (data[1] & 0x01) != 0);
>>
>> This implementation impairs the value of those buttons since I know a
>> lot of users want them configurable. If we can not or do not pass a
>> generic set to the userland, we will need to make it configurable in
>> the kernel driver as Dmitry suggested (if I remember it correctly).
>> Which approach, in the userland or the kernel, do you like, Henrik?
>
> Perhaps I should not remind you that the current code was your suggestion. ;-)
Yes, I remember that. TBH, I did not want to open another can of worms
then. And I did not have time to care about where that train went :).
> In my mind, these are buttons just like the buttons on any mouse or trackpad.
For this particular model, it might be fine. My concern is for the other models.
> Regarding remapping, I think it matters if buttons have strong or weak semantics.
Supporting button remapping is not a question. It is a decision. The
question is how. I hate to open this can of worms even now. But, we do
have to face the reality.
Currently, I use BTN_TOOL_FINGER to group those buttons together and
to pass them to the userland. This has never been an issue before MT
became such a hot topic. Now, we need to EITHER choose a new
BTN_TOOL_something to pass the poor buttons to the userland OR use the
kernel approach that Dmitry suggested. I am open to all suggestions
although I feel passing the buttons to the userland is a
better/cleaner solution.
My goal is to get a consensus so no one can blame me or the maintainer
for the decision :). Fair?
Ping
--
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:[~2010-10-13 20:15 UTC|newest]
Thread overview: 19+ messages / expand[flat|nested] mbox.gz Atom feed top
2010-09-04 13:42 [PATCH 0/5] input: wacom: Initial support for Bamboo (rev3) Henrik Rydberg
2010-09-04 13:42 ` [PATCH 1/5] input: wacom: Add fuzz parameters to features Henrik Rydberg
2010-09-04 13:43 ` [PATCH 2/5] input: wacom: Parse the Bamboo device family Henrik Rydberg
2010-09-04 13:43 ` [PATCH 3/5] input: wacom: Collect device quirks into single function (rev2) Henrik Rydberg
2010-09-04 13:43 ` [PATCH 4/5] input: wacom: Add support for the Bamboo Touch trackpad (rev4) Henrik Rydberg
2010-09-05 10:04 ` Henrik Rydberg
2010-09-05 20:03 ` Dmitry Torokhov
2010-10-13 16:31 ` Ping Cheng
2010-10-13 18:21 ` Henrik Rydberg
2010-10-13 20:15 ` Ping Cheng [this message]
2010-10-13 20:50 ` Chris Bagwell
2010-10-13 21:19 ` Ping Cheng
2010-10-29 20:45 ` Ping Cheng
2010-10-30 0:55 ` Chris Bagwell
2010-10-30 11:52 ` Ping Cheng
2010-11-01 14:43 ` Henrik Rydberg
2010-09-04 13:43 ` [PATCH 5/5] input: wacom: Add a quirk for lowres Bamboo devices (rev2) Henrik Rydberg
2010-09-04 21:24 ` [PATCH 0/5] input: wacom: Initial support for Bamboo (rev3) Ping Cheng
2010-09-05 14:28 ` Chris Bagwell
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='AANLkTi=RZ5OmBBOV2=kYJ+wE2GD1N9ZKMNyKGVvFqwiR@mail.gmail.com' \
--to=pinglinux@gmail.com \
--cc=chris@cnpbagwell.com \
--cc=dmitry.torokhov@gmail.com \
--cc=linux-input@vger.kernel.org \
--cc=rydberg@euromail.se \
/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).