On Wed, Jun 29, 2011 at 6:02 PM, Chase Douglas<chase.douglas@canonical.com> wrote:>> On 06/29/2011 06:07 AM, djkurtz@chromium.org wrote:> > From: Daniel Kurtz <djkurtz@chromium.org>> >> > A Synaptics image sensor tracks 5 fingers, but can only report 2.> > This behavior is called "T5R2" = Track 5 Report 2> >> > Algorithm for choosing which 2 fingers to report in which packet:> > Touchpad maintains 5 slots, numbered 0 to 4.> > Initially all slots are empty.> > As new fingers are detected, they are assigned the lowest available> > slot.> > Touchpad always reports:> > SGM: lowest numbered non-empty slot> > AGM: highest numbered non-empty slot, if there is one.>> Hi Daniel,>> Thanks for the new patches! I will review them soon in detail, but I> would like to first confirm some details about the above.>> In the older "profile" devices, we got essentially the bounding box of> all touches. This allows us to detect pinch gestures between the two> most extreme touches when there are three touches on the touchpad.>> The above description makes it sound like we will no longer have a> bounding box of touches. According to the description, if I put four> fingers down in a square formation and then touch a fifth finger in the> middle of the square, I will only see the locations of one corner and> the middle of the square. This would make meaningful gesture detection> beyond two touches nearly impossible. Is this really how the touchpad> works, or is the above description not quite right?

However, at least from my experiments with one profile sensor, thereis no guarantee that this rectangle actually contains all touches onthe pad.In fact, I think it is trying really hard to return the position ofthe first two touches that it is tracking.When there are two fingers, it returns the finger with the largery-value (and what the trackpad believes to be its x-), and the SGMpacket, and the other finger in the AGM.When there are three or more fingers, it still reports the first two,irregardless of where the new fingers are added.

It would be very curious if different profile sensors behaved differently.

In any case, this patch should not change the behavior of profilesensors, since it only applies to image sensors.