On 07/06/2011 01:20 PM, Henrik Rydberg wrote: >>> To me, it seems we do need a little bit of extra information to >>> determine this new type of device. >> >> I think we already have all we need (see above). > > I concur. So, to conclude: > > a) The improved synaptics behavior can be achieved by simply using > MT-B plus BTN_TOOL_*. > > b) Userspace should check BTN_TOOL_* for any discrepancies between the > maximum number of available slots (always two in this case) and the > maximum number of fingers reported (BTN_TOOL_TRIPLETAP etc). Extra > actions may then be taken to support more fingers than slots. > > c) The semi-mt flag is only used to signal that the two points sent > via MT-B are the corners of a bounding box. This isn't quite enough. If we don't set the semi-mt flag or any other new flag, then we'll have slots that become inconsistent when touches are added or removed. For example, start with two touches being tracked correctly. Now, add a touch. The second slot will now get the data of the third touch, which is in a different location. You haven't changed the tracking_id though, so it looks like the same touch. This is incorrect behavior. Or, you could change the tracking_id, but that implies that a touch was lifted and another was placed. This is also incorrect behavior. We need to tell userspace that this is a messed up device that can't accurately track touch locations across touch up/down boundaries. Once userspace sees this, it can act appropriately when it sees a transition from BTN_TOOL_DOUBLETAP to BTN_TOOL_TRIPLETAP, for example. This is how we handle transitions in the uTouch stack for semi-mt devices. Perhaps a clean implementation would be to keep semi-mt as a flag stating that there will only ever be two slots, the 0th will be the minimum (x,y) of the bounding box and the 1st will be the maximum. Then, we add a flag like NO_TOUCH_TRANSITION_TRACKING that would be set on both semi-mt and these new devices that denote the slot data may transition from one physical touchpoint to another when the number of touches changes. We could leave this up to userspace and have it detect a NO_TOUCH_TRANSITION_TRACKING device based on the fact that the max slots is less than the max fingers, but I would argue here that a more clear protocol is a better protocol. -- Chase -- To unsubscribe from this list: send the line "unsubscribe linux-input" in the body of a message to majordomo@xxxxxxxxxxxxxxx More majordomo info at http://vger.kernel.org/majordomo-info.html
