From mboxrd@z Thu Jan 1 00:00:00 1970 From: Peter Wu Subject: About "HID: logitech-hidpp: add support of the first Logitech Wireless Touchpad" Date: Thu, 11 Dec 2014 11:02:20 +0100 Message-ID: <1659058.m7A8Z3l7lH@al> Mime-Version: 1.0 Content-Type: text/plain; charset="us-ascii" Content-Transfer-Encoding: 7Bit Return-path: Received: from lekensteyn.nl ([178.21.112.251]:51055 "EHLO lekensteyn.nl" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S932698AbaLKKCZ (ORCPT ); Thu, 11 Dec 2014 05:02:25 -0500 Sender: linux-input-owner@vger.kernel.org List-Id: linux-input@vger.kernel.org To: Benjamin Tissoires Cc: linux-input@vger.kernel.org, linux-kernel@vger.kernel.org, Andrew de los Reyes , Jiri Kosina Hi Benjamin, In commit 57ac86cf52e903d9e3e0f12b34c814cce6b65550 ("HID: logitech-hidpp: add support of the first Logitech Wireless Touchpad") which is in jikos/hid, you made this change to wtp_raw_event: switch (data[0]) { case 0x02: - if (size < 21) - return 1; - return wtp_mouse_raw_xy_event(hidpp, &data[7]); + if (hidpp->quirks & HIDPP_QUIRK_WTP_PHYSICAL_BUTTONS) { + input_event(wd->input, EV_KEY, BTN_LEFT, + !!(data[1] & 0x01)); + input_event(wd->input, EV_KEY, BTN_RIGHT, + !!(data[1] & 0x02)); + input_sync(wd->input); + } else { + if (size < 21) + return 1; + return wtp_mouse_raw_xy_event(hidpp, &data[7]); + } case REPORT_ID_HIDPP_LONG: if ((report->fap.feature_index != wd->mt_feature_index) || (report->fap.funcindex_clientid != EVENT_TOUCHPAD_RAW_XY)) return 1; hidpp_touchpad_raw_xy_event(hidpp, data + 4, &raw); wtp_send_raw_xy_event(hidpp, &raw); return 0; } Report ID 2 is the mouse descriptor, so it seems correct in that it falls-through to the next case, but I wanted to check with you that this is indeed your intention. If so, could you explicitly mark it with a comment, /* fallthrough */ ? -- Kind regards, Peter https://lekensteyn.nl