Messages in this thread | | | From | Stéphane Chatty <> | Subject | Re: [PATCH 2/3] hid: 3m: Convert to MT slots | Date | Mon, 20 Sep 2010 22:23:21 +0200 |
| |
Le 20 sept. 10 à 22:04, Henrik Rydberg a écrit : >> >> What about this: >> + value = clamp_val(value, 0, MAX_SLOTS - 1); >> if (md->valid) { >> md->curid = value; >> md->f[value].touch = md->touch; >> >> If you had tracking IDs 59, 60 and 61 in the same frame, this >> would not work >> properly would it? > > > That is the slot id being set. The microtouch controller is also > using slots > internally. >
Oh, right. Once again back to your definition of tracking ID that is not so intuitive to me :-) In the end we might end up dealing with three IDs: the device's tracking ID, the slot ID, and the Linux tracking ID. Here, the first and the second are the same but will they always be? I remember when Stantum used to provide us with 16 bit "device tracking IDs"...
Anyway, I have no more problem with this constant. Just with the vocabulary :-)
St.-- To unsubscribe from this list: send the line "unsubscribe linux-kernel" in the body of a message to majordomo@vger.kernel.org More majordomo info at http://vger.kernel.org/majordomo-info.html Please read the FAQ at http://www.tux.org/lkml/
| |