On 10/18/2016 08:26 AM, Jiri Kosina wrote: > On Fri, 7 Oct 2016, Jason Gerecke wrote: > >> @@ -1711,7 +1736,7 @@ static int wacom_wac_finger_event(struct hid_device *hdev, >> >> >> if (usage->usage_index + 1 == field->report_count) { >> - if (usage->hid == wacom_wac->features.last_slot_field) >> + if (equivalent_usage == wacom_wac->features.last_slot_field) > > What tree is this based on please? In all the codebase I have, > last_slot_field is a member field of wacom_wac's hid_data, not a member of > struct wacom_features. > This was based on your for-4.9/wacom tree, whose tip (for me) was commit 1924e05 ("HID: wacom - add touch_arbitration parameter to wacom module"). It looks like my tree only has commit 601a22f ("HID: wacom: Report input events for each finger on generic devices") but not commit 003f50a ("HID: wacom: Update last_slot_field during pre_report phase"). Not sure why though. What would you like me to do? Jason --- Now instead of four in the eights place / you’ve got three, ‘Cause you added one / (That is to say, eight) to the two, / But you can’t take seven from three, / So you look at the sixty-fours.... -- 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