public inbox for linux-kernel@vger.kernel.org
 help / color / mirror / Atom feed
From: Henrik Rydberg <rydberg@euromail.se>
To: Ping Cheng <pinglinux@gmail.com>
Cc: Dmitry Torokhov <dmitry.torokhov@gmail.com>,
	Jiri Kosina <jkosina@suse.cz>,
	linux-input@vger.kernel.org, linux-kernel@vger.kernel.org
Subject: Re: [PATCH] input: mt: Interface and MT_TOOL documentation updates
Date: Fri, 10 Dec 2010 21:01:34 +0100	[thread overview]
Message-ID: <4D02871E.6060204@euromail.se> (raw)
In-Reply-To: <AANLkTinctPSxGt36HCdyoWYH7TBznzY0CejLP3uALkV3@mail.gmail.com>

On 12/10/2010 08:00 PM, Ping Cheng wrote:

> On Fri, Dec 10, 2010 at 10:13 AM, Henrik Rydberg <rydberg@euromail.se> wrote:
>>
>>> Can we make MT_TOOL_ENVELOPE cover a bit more cases by:
>>>
>>> 1. Removing ", and is only used for legacy hardware";
>>> 2. Adding "Or the number of contacts inside the bounding rectangle is
>>> reported if hardware provides the number but not the real contact
>>> positions" to the end of the paragraph.
>>
>> You might disagree, but "old" is still somewhat apt in this situation.
> 
> It's ok if we say the new type was inspired by legacy hardware. But
> saying that it "is only used for legacy hardware" closes the door for
> future development. That's not what we are trying to do, right?


Well, in a sense we are. I would agree that data aiming to provide gestures as a
2D transformation matrix can be handled quite well with two tracked points and a
finger count. However, a multitouch interface where users manipulate different
objects on the screen simultaneously is a different story.

> 
>> How would you suggest we report the number of fingers?
> 
> I guess if we want to make it generic, we could have something like
> ABS_MT_NUM_CONTACTS to go with MT_TOOL_ENVELOPE. Clients, such as
> synaptics touchpads, that only care about the number of contacts
> inside the envelope don't need to process the contact positions even
> when they are reported. This also resolve the potential that
> BTN_TOOL_QUADTAP is not enough to tell us how many contacts are on the
> surface.


I really would like to avoid adding a new way to solve an old problem, in
particular given the statement above. Adding something like BTN_TOOL_QUINTAP
would hurt a little bit, but not nearly as much.

> 
> Maybe we should also tell the clients whether they are going to get
> the contact positions or not.


I may not understand what you mean here, but if you are referring to an up-front
declaration of what MT_TOOL types are to be expected, we did discuss this
before, without any conclusion. Perhaps it is relevant to outline why this would
be important.

Thanks,
Henrik

  reply	other threads:[~2010-12-10 20:01 UTC|newest]

Thread overview: 11+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2010-12-09 21:22 [PATCH] input: mt: Interface and MT_TOOL documentation updates Henrik Rydberg
2010-12-10  7:43 ` Dmitry Torokhov
2010-12-10 13:54   ` Henrik Rydberg
2010-12-10 17:55 ` Ping Cheng
2010-12-10 18:13   ` Henrik Rydberg
2010-12-10 18:52     ` Dmitry Torokhov
2010-12-10 19:00     ` Ping Cheng
2010-12-10 20:01       ` Henrik Rydberg [this message]
2010-12-10 21:17         ` Ping Cheng
2010-12-10 18:13   ` Dmitry Torokhov
2010-12-10 18:50     ` Dmitry Torokhov

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=4D02871E.6060204@euromail.se \
    --to=rydberg@euromail.se \
    --cc=dmitry.torokhov@gmail.com \
    --cc=jkosina@suse.cz \
    --cc=linux-input@vger.kernel.org \
    --cc=linux-kernel@vger.kernel.org \
    --cc=pinglinux@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