From patchwork Wed Sep 28 08:35:42 2016 Content-Type: text/plain; charset="utf-8" MIME-Version: 1.0 Content-Transfer-Encoding: 7bit X-Patchwork-Submitter: Benjamin Tissoires X-Patchwork-Id: 9353309 Return-Path: Received: from mail.wl.linuxfoundation.org (pdx-wl-mail.web.codeaurora.org [172.30.200.125]) by pdx-korg-patchwork.web.codeaurora.org (Postfix) with ESMTP id 1AFD660756 for ; Wed, 28 Sep 2016 08:38:04 +0000 (UTC) Received: from mail.wl.linuxfoundation.org (localhost [127.0.0.1]) by mail.wl.linuxfoundation.org (Postfix) with ESMTP id 09F2B2901D for ; Wed, 28 Sep 2016 08:38:04 +0000 (UTC) Received: by mail.wl.linuxfoundation.org (Postfix, from userid 486) id F2ACC2943D; Wed, 28 Sep 2016 08:38:03 +0000 (UTC) X-Spam-Checker-Version: SpamAssassin 3.3.1 (2010-03-16) on pdx-wl-mail.web.codeaurora.org X-Spam-Level: X-Spam-Status: No, score=-6.9 required=2.0 tests=BAYES_00,RCVD_IN_DNSWL_HI autolearn=ham version=3.3.1 Received: from vger.kernel.org (vger.kernel.org [209.132.180.67]) by mail.wl.linuxfoundation.org (Postfix) with ESMTP id 63FAB2901D for ; Wed, 28 Sep 2016 08:38:03 +0000 (UTC) Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S1752139AbcI1IiA (ORCPT ); Wed, 28 Sep 2016 04:38:00 -0400 Received: from mx1.redhat.com ([209.132.183.28]:42744 "EHLO mx1.redhat.com" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S1752150AbcI1IgI (ORCPT ); Wed, 28 Sep 2016 04:36:08 -0400 Received: from int-mx11.intmail.prod.int.phx2.redhat.com (int-mx11.intmail.prod.int.phx2.redhat.com [10.5.11.24]) (using TLSv1.2 with cipher ECDHE-RSA-AES256-GCM-SHA384 (256/256 bits)) (No client certificate requested) by mx1.redhat.com (Postfix) with ESMTPS id AA94E43A42; Wed, 28 Sep 2016 08:36:06 +0000 (UTC) Received: from plouf.banquise.eu.com (ovpn-116-65.ams2.redhat.com [10.36.116.65]) by int-mx11.intmail.prod.int.phx2.redhat.com (8.14.4/8.14.4) with ESMTP id u8S8Zm4n013918; Wed, 28 Sep 2016 04:36:05 -0400 From: Benjamin Tissoires To: Dmitry Torokhov , Lyude Paul , Andrew Duggan , Christopher Heiny Cc: Peter Hutterer , linux-kernel@vger.kernel.org, linux-input@vger.kernel.org Subject: [PATCH v2 07/12] Input: synaptics-rmi4 - f30/f03: Forward mechanical buttons on buttonpads to PS/2 guest Date: Wed, 28 Sep 2016 10:35:42 +0200 Message-Id: <1475051747-25988-8-git-send-email-benjamin.tissoires@redhat.com> In-Reply-To: <1475051747-25988-1-git-send-email-benjamin.tissoires@redhat.com> References: <1475051747-25988-1-git-send-email-benjamin.tissoires@redhat.com> X-Scanned-By: MIMEDefang 2.68 on 10.5.11.24 X-Greylist: Sender IP whitelisted, not delayed by milter-greylist-4.5.16 (mx1.redhat.com [10.5.110.30]); Wed, 28 Sep 2016 08:36:06 +0000 (UTC) Sender: linux-input-owner@vger.kernel.org Precedence: bulk List-ID: X-Mailing-List: linux-input@vger.kernel.org X-Virus-Scanned: ClamAV using ClamSMTP On the latest series of ThinkPads, the button events for the TrackPoint are reported through the touchpad itself as opposed to the TrackPoint device. In order to report these buttons properly, we need to forward them to the TrackPoint device and send the button presses/releases through there instead. Signed-off-by: Lyude Paul Signed-off-by: Benjamin Tissoires --- changes in v2: - make sure we don't break other devices using F30 --- drivers/input/rmi4/rmi_driver.h | 13 ++++++++ drivers/input/rmi4/rmi_f03.c | 28 ++++++++++++++++ drivers/input/rmi4/rmi_f30.c | 72 ++++++++++++++++++++++++++++++++--------- 3 files changed, 98 insertions(+), 15 deletions(-) diff --git a/drivers/input/rmi4/rmi_driver.h b/drivers/input/rmi4/rmi_driver.h index 9da1ee4..898cadc 100644 --- a/drivers/input/rmi4/rmi_driver.h +++ b/drivers/input/rmi4/rmi_driver.h @@ -99,6 +99,19 @@ struct rmi_function *rmi_find_function(struct rmi_device *rmi_dev, u8 number); char *rmi_f01_get_product_ID(struct rmi_function *fn); +#ifdef CONFIG_RMI4_F03 +int rmi_f03_overwrite_button(struct rmi_function *fn, unsigned int button, + int value); +void rmi_f03_commit_buttons(struct rmi_function *fn); +#else +static inline int rmi_f03_overwrite_button(struct rmi_function *fn, + unsigned int button, int value) +{ + return 0; +} +static inline void rmi_f03_commit_buttons(struct rmi_function *fn) {} +#endif + extern struct rmi_function_handler rmi_f01_handler; extern struct rmi_function_handler rmi_f03_handler; extern struct rmi_function_handler rmi_f11_handler; diff --git a/drivers/input/rmi4/rmi_f03.c b/drivers/input/rmi4/rmi_f03.c index a124b33..f309b5f 100644 --- a/drivers/input/rmi4/rmi_f03.c +++ b/drivers/input/rmi4/rmi_f03.c @@ -37,6 +37,34 @@ struct f03_data { u8 rx_queue_length; }; +int rmi_f03_overwrite_button(struct rmi_function *fn, unsigned int button, + int value) +{ + struct f03_data *f03 = dev_get_drvdata(&fn->dev); + unsigned int bit = BIT(button); + + if (button > 2) + return -EINVAL; + + if (value) + f03->overwrite_buttons |= bit; + else + f03->overwrite_buttons &= ~bit; + + return 0; +} + +void rmi_f03_commit_buttons(struct rmi_function *fn) +{ + struct f03_data *f03 = dev_get_drvdata(&fn->dev); + int i; + + f03->serio->extra_byte = f03->overwrite_buttons; + + for (i = 0; i < 3; i++) + serio_interrupt(f03->serio, 0x00, 0x00); +} + static int rmi_f03_pt_write(struct serio *id, unsigned char val) { struct f03_data *f03 = id->port_data; diff --git a/drivers/input/rmi4/rmi_f30.c b/drivers/input/rmi4/rmi_f30.c index c8885d8..865cb6c 100644 --- a/drivers/input/rmi4/rmi_f30.c +++ b/drivers/input/rmi4/rmi_f30.c @@ -74,8 +74,11 @@ struct f30_data { u8 data_regs[RMI_F30_CTRL_MAX_BYTES]; u16 *gpioled_key_map; + u16 *gpio_passthrough_key_map; struct input_dev *input; + bool trackstick_buttons; + struct rmi_function *f03; }; static int rmi_f30_read_control_parameters(struct rmi_function *fn, @@ -108,6 +111,13 @@ static int rmi_f30_attention(struct rmi_function *fn, unsigned long *irq_bits) if (!f30->input) return 0; + if (f30->trackstick_buttons && !f30->f03) { + f30->f03 = rmi_find_function(rmi_dev, 3); + + if (!f30->f03) + return -EBUSY; + } + /* Read the gpi led data. */ if (rmi_dev->xport->attn_data) { memcpy(f30->data_regs, rmi_dev->xport->attn_data, @@ -128,23 +138,29 @@ static int rmi_f30_attention(struct rmi_function *fn, unsigned long *irq_bits) for (reg_num = 0; reg_num < f30->register_count; ++reg_num) { for (i = 0; gpiled < f30->gpioled_count && i < 8; ++i, ++gpiled) { - if (f30->gpioled_key_map[gpiled] != 0) { - /* buttons have pull up resistors */ - value = (((f30->data_regs[reg_num] >> i) & 0x01) - == 0); + /* buttons have pull up resistors */ + value = (((f30->data_regs[reg_num] >> i) & 0x01) == 0); + if (f30->gpioled_key_map[gpiled] != 0) { rmi_dbg(RMI_DEBUG_FN, &fn->dev, "%s: call input report key (0x%04x) value (0x%02x)", __func__, f30->gpioled_key_map[gpiled], value); + input_report_key(f30->input, f30->gpioled_key_map[gpiled], value); + } else if (f30->gpio_passthrough_key_map[gpiled]) { + rmi_f03_overwrite_button(f30->f03, + f30->gpio_passthrough_key_map[gpiled] - BTN_LEFT, + value); } - } } + if (f30->trackstick_buttons) + rmi_f03_commit_buttons(f30->f03); + return 0; } @@ -242,10 +258,11 @@ static inline int rmi_f30_initialize(struct rmi_function *fn) int retval = 0; int control_address; int i; - int button; + int button, extra_button; u8 buf[RMI_F30_QUERY_SIZE]; u8 *ctrl_reg; - u8 *map_memory; + u8 *map_memory, *pt_memory; + bool buttonpad; f30 = devm_kzalloc(&fn->dev, sizeof(struct f30_data), GFP_KERNEL); @@ -343,29 +360,54 @@ static inline int rmi_f30_initialize(struct rmi_function *fn) map_memory = devm_kzalloc(&fn->dev, (f30->gpioled_count * (sizeof(u16))), GFP_KERNEL); - if (!map_memory) { + pt_memory = devm_kzalloc(&fn->dev, + (f30->gpioled_count * (sizeof(u16))), + GFP_KERNEL); + if (!map_memory || !pt_memory) { dev_err(&fn->dev, "Failed to allocate gpioled map memory.\n"); return -ENOMEM; } f30->gpioled_key_map = (u16 *)map_memory; + f30->gpio_passthrough_key_map = (u16 *)pt_memory; pdata = rmi_get_platform_data(rmi_dev); if (f30->has_gpio) { + /* + * buttonpad might be given by f30->has_mech_mouse_btns, + * but I am not sure, so use only the pdata info + */ + buttonpad = pdata->f30_data.buttonpad; + f30->trackstick_buttons = pdata->f30_data.trackstick_buttons; + + /* + * For touchpads the buttons are mapped as: + * - bit 0 = Left, bit 1 = right, bit 2 = middle / clickbutton + * - 3, 4, 5 are extended buttons and + * - 6 and 7 are other sorts of GPIOs + */ button = BTN_LEFT; - for (i = 0; i < f30->gpioled_count; i++) { + extra_button = BTN_LEFT; + for (i = 0; i < f30->gpioled_count && i < 3; i++) { if (rmi_f30_is_valid_button(i, f30->ctrl)) { f30->gpioled_key_map[i] = button++; - /* - * buttonpad might be given by - * f30->has_mech_mouse_btns, but I am - * not sure, so use only the pdata info - */ - if (pdata->f30_data.buttonpad) + if (buttonpad) break; } } + + if (f30->trackstick_buttons) { + for (i = 3; i < f30->gpioled_count && i < 6; i++) { + if (rmi_f30_is_valid_button(i, f30->ctrl)) + f30->gpio_passthrough_key_map[i] = extra_button++; + } + } else if (!buttonpad) { + for (i = 3; i < f30->gpioled_count; i++) { + if (rmi_f30_is_valid_button(i, f30->ctrl)) + f30->gpioled_key_map[i] = button++; + } + } } return 0;