From patchwork Sun Jul 31 15:23:07 2016 Content-Type: text/plain; charset="utf-8" MIME-Version: 1.0 Content-Transfer-Encoding: 7bit X-Patchwork-Submitter: Hans de Goede X-Patchwork-Id: 9253407 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 5B9AF601C0 for ; Sun, 31 Jul 2016 15:25:41 +0000 (UTC) Received: from mail.wl.linuxfoundation.org (localhost [127.0.0.1]) by mail.wl.linuxfoundation.org (Postfix) with ESMTP id 4C5102848A for ; Sun, 31 Jul 2016 15:25:41 +0000 (UTC) Received: by mail.wl.linuxfoundation.org (Postfix, from userid 486) id 406152848D; Sun, 31 Jul 2016 15:25:41 +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=-4.2 required=2.0 tests=BAYES_00, RCVD_IN_DNSWL_MED autolearn=unavailable version=3.3.1 Received: from bombadil.infradead.org (bombadil.infradead.org [198.137.202.9]) (using TLSv1.2 with cipher AES128-GCM-SHA256 (128/128 bits)) (No client certificate requested) by mail.wl.linuxfoundation.org (Postfix) with ESMTPS id BE93F2848A for ; Sun, 31 Jul 2016 15:25:40 +0000 (UTC) Received: from localhost ([127.0.0.1] helo=bombadil.infradead.org) by bombadil.infradead.org with esmtp (Exim 4.85_2 #1 (Red Hat Linux)) id 1bTsap-0005dW-K5; Sun, 31 Jul 2016 15:23:55 +0000 Received: from mx1.redhat.com ([209.132.183.28]) by bombadil.infradead.org with esmtps (Exim 4.85_2 #1 (Red Hat Linux)) id 1bTsaX-0005Yg-UC for linux-arm-kernel@lists.infradead.org; Sun, 31 Jul 2016 15:23:39 +0000 Received: from int-mx13.intmail.prod.int.phx2.redhat.com (int-mx13.intmail.prod.int.phx2.redhat.com [10.5.11.26]) (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 2FF94C05AA4E; Sun, 31 Jul 2016 15:23:19 +0000 (UTC) Received: from shalem.localdomain.com (vpn1-6-127.ams2.redhat.com [10.36.6.127]) by int-mx13.intmail.prod.int.phx2.redhat.com (8.14.4/8.14.4) with ESMTP id u6VFNCw1006607; Sun, 31 Jul 2016 11:23:16 -0400 From: Hans de Goede To: Dmitry Torokhov , Rob Herring Subject: [PATCH 1/4] input: touchscreen: Add generic touchscreen softbutton handling code Date: Sun, 31 Jul 2016 17:23:07 +0200 Message-Id: <1469978590-14081-2-git-send-email-hdegoede@redhat.com> In-Reply-To: <1469978590-14081-1-git-send-email-hdegoede@redhat.com> References: <1469978590-14081-1-git-send-email-hdegoede@redhat.com> X-Scanned-By: MIMEDefang 2.68 on 10.5.11.26 X-Greylist: Sender IP whitelisted, not delayed by milter-greylist-4.5.16 (mx1.redhat.com [10.5.110.32]); Sun, 31 Jul 2016 15:23:19 +0000 (UTC) X-CRM114-Version: 20100106-BlameMichelson ( TRE 0.8.0 (BSD) ) MR-646709E3 X-CRM114-CacheID: sfid-20160731_082338_033272_EF47DE33 X-CRM114-Status: GOOD ( 25.63 ) X-BeenThere: linux-arm-kernel@lists.infradead.org X-Mailman-Version: 2.1.20 Precedence: list List-Id: List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , Cc: devicetree , Hans de Goede , Chen-Yu Tsai , linux-input@vger.kernel.org, Maxime Ripard , linux-arm-kernel@lists.infradead.org MIME-Version: 1.0 Sender: "linux-arm-kernel" Errors-To: linux-arm-kernel-bounces+patchwork-linux-arm=patchwork.kernel.org@lists.infradead.org X-Virus-Scanned: ClamAV using ClamSMTP Some touchscreens extend over the display they cover and have a number of capacative softbuttons outside of the display the cover. With some hardware these softbuttons simply report touches with coordinates outside of the normal coordinate space for touches on the display. This commit adds a devicetree binding for describing such buttons in devicetree and a bunch of helper functions to easily add support for these to existing touchscreen drivers. Signed-off-by: Hans de Goede --- .../bindings/input/touchscreen/softbuttons.txt | 58 +++++++++ drivers/input/touchscreen/Makefile | 2 +- drivers/input/touchscreen/softbuttons.c | 145 +++++++++++++++++++++ include/linux/input/touchscreen.h | 9 ++ 4 files changed, 213 insertions(+), 1 deletion(-) create mode 100644 Documentation/devicetree/bindings/input/touchscreen/softbuttons.txt create mode 100644 drivers/input/touchscreen/softbuttons.c diff --git a/Documentation/devicetree/bindings/input/touchscreen/softbuttons.txt b/Documentation/devicetree/bindings/input/touchscreen/softbuttons.txt new file mode 100644 index 0000000..3eb6f4c --- /dev/null +++ b/Documentation/devicetree/bindings/input/touchscreen/softbuttons.txt @@ -0,0 +1,58 @@ +General Touchscreen Softbutton Properties: + +Some touchscreens extend over the display they cover and have a number +of capacative softbuttons outside of the display the cover. + +Some of these softbuttons simply report touches with coordinates outside of +the normal coordinate space for touches on the display. This binding is for +describing such buttons in devicetree. + +Each softkey is represented as a sub-node of the touchscreen node. + +Required subnode-properties: + - label : Descriptive name of the key. + - linux,code : Keycode to emit. + - softbutton-min-x : X start of the area the softbutton area covers + - softbutton-max-x : X end of the area the softbutton area covers + - softbutton-min-y : Y start of the area the softbutton area covers + - softbutton-max-y : Y end of the area the softbutton area covers + +Example: + +#include + +&i2c2 { + ft5406ee8: touchscreen@38 { + compatible = "edt,edt-ft5406"; + reg = <0x38>; + touchscreen-size-x = <1024>; + touchscreen-size-y = <768>; + + button@0 { + label = "Esc"; + linux,code = ; + softbutton-min-x = <1084>; + softbutton-max-x = <1098>; + softbutton-min-y = <0>; + softbutton-max-y = <49>; + }; + + button@1 { + label = "Home"; + linux,code = ; + softbutton-min-x = <1084>; + softbutton-max-x = <1098>; + softbutton-min-y = <50>; + softbutton-max-y = <99>; + }; + + button@2 { + label = "Menu"; + linux,code = ; + softbutton-min-x = <1084>; + softbutton-max-x = <1098>; + softbutton-min-y = <100>; + softbutton-max-y = <149>; + }; + }; +}; diff --git a/drivers/input/touchscreen/Makefile b/drivers/input/touchscreen/Makefile index 4166747..a4b00d0 100644 --- a/drivers/input/touchscreen/Makefile +++ b/drivers/input/touchscreen/Makefile @@ -6,7 +6,7 @@ wm97xx-ts-y := wm97xx-core.o -obj-$(CONFIG_TOUCHSCREEN_PROPERTIES) += of_touchscreen.o +obj-$(CONFIG_TOUCHSCREEN_PROPERTIES) += of_touchscreen.o softbuttons.o obj-$(CONFIG_TOUCHSCREEN_88PM860X) += 88pm860x-ts.o obj-$(CONFIG_TOUCHSCREEN_AD7877) += ad7877.o obj-$(CONFIG_TOUCHSCREEN_AD7879) += ad7879.o diff --git a/drivers/input/touchscreen/softbuttons.c b/drivers/input/touchscreen/softbuttons.c new file mode 100644 index 0000000..47aea18 --- /dev/null +++ b/drivers/input/touchscreen/softbuttons.c @@ -0,0 +1,145 @@ +/* + * touchscreen softbutton helper functions + * + * Copyright (c) 2016 Hans de Goede + * + * This program is free software; you can redistribute it and/or modify + * it under the terms of the GNU General Public License as published by + * the Free Software Foundation; either version 2 of the License, or + * (at your option) any later version. + */ + +#include +#include +#include + +struct touchscreen_softbutton { + u32 min_x; + u32 max_x; + u32 min_y; + u32 max_y; + u32 keycode; +}; + +struct touchscreen_softbutton_info { + struct input_dev *input; + struct touchscreen_softbutton *buttons; + int button_count; +}; + +/** + * devm_touchscreen_alloc_softbuttons - allocate softbuttons + * @input: touchscreen input device for which softbuttons should be allocated + * + * This function parses touschcreen softbutton DT properties for touchscreens + * and allocates and fill a touchscreen_softbutton_info struct if any + * softbuttons are found. + * + * Returns prepared struct touchscreen_softbutton_info on success, + * %NULL if no softbuttons were found (this is not an error) or a ERR_PTR + * in case of an error. + * + * Note as this is a devm function the returned pointer does not need to + * be freed. + */ +struct touchscreen_softbutton_info *devm_touchscreen_alloc_softbuttons( + struct input_dev *input) +{ + struct device *dev = input->dev.parent; + struct device_node *np, *pp; + struct touchscreen_softbutton_info *info; + int i, err, button_count; + + np = dev->of_node; + if (!np) + return NULL; + + button_count = of_get_child_count(np); + if (button_count == 0) + return NULL; + + info = devm_kzalloc(dev, sizeof(*info), GFP_KERNEL); + if (!info) + return ERR_PTR(-ENOMEM); + + info->input = input; + info->button_count = button_count; + info->buttons = devm_kzalloc(dev, button_count * sizeof(*info->buttons), + GFP_KERNEL); + if (!info->buttons) + return ERR_PTR(-ENOMEM); + + for (pp = of_get_next_child(np, NULL), i = 0; + pp != NULL; + pp = of_get_next_child(np, pp), i++) { + struct touchscreen_softbutton *btn = &info->buttons[i]; + + err = of_property_read_u32(pp, "linux,code", &btn->keycode); + if (err) { + dev_err(dev, "%s: Inval linux,code prop\n", pp->name); + return ERR_PTR(-EINVAL); + } + + err = of_property_read_u32(pp, "softbutton-min-x", &btn->min_x); + if (err) { + dev_err(dev, "%s: Inval min-x prop\n", pp->name); + return ERR_PTR(-EINVAL); + } + + err = of_property_read_u32(pp, "softbutton-max-x", &btn->max_x); + if (err) { + dev_err(dev, "%s: Inval max-x prop\n", pp->name); + return ERR_PTR(-EINVAL); + } + + err = of_property_read_u32(pp, "softbutton-min-y", &btn->min_y); + if (err) { + dev_err(dev, "%s: Inval min-y prop\n", pp->name); + return ERR_PTR(-EINVAL); + } + + err = of_property_read_u32(pp, "softbutton-max-y", &btn->max_y); + if (err) { + dev_err(dev, "%s: Inval max-y prop\n", pp->name); + return ERR_PTR(-EINVAL); + } + } + + __set_bit(EV_KEY, input->evbit); + for (i = 0; i < info->button_count; i++) + __set_bit(info->buttons[i].keycode, input->keybit); + + return info; +} + +/** + * touchscreen_handle_softbuttons - check for softbutton press + * @info: softbutton info retured by devm_touchscreen_alloc_softbuttons. + * + * This function checks if the passed in coordinates match any softbutton, + * and when they do reports a key press / release for the softbutton. + * + * Returns true if the coordinates match a softbutton and a key press / release + * was reported, false otherwise. + */ +bool touchscreen_handle_softbuttons(struct touchscreen_softbutton_info *info, + unsigned int x, unsigned int y, bool down) +{ + int i; + + if (info == NULL) + return false; + + for (i = 0; i < info->button_count; i++) { + if (x >= info->buttons[i].min_x && + x <= info->buttons[i].max_x && + y >= info->buttons[i].min_y && + y <= info->buttons[i].max_y) { + input_report_key(info->input, + info->buttons[i].keycode, down); + return true; + } + } + + return false; +} diff --git a/include/linux/input/touchscreen.h b/include/linux/input/touchscreen.h index 09d22cc..0b9d4ee 100644 --- a/include/linux/input/touchscreen.h +++ b/include/linux/input/touchscreen.h @@ -9,8 +9,11 @@ #ifndef _TOUCHSCREEN_H #define _TOUCHSCREEN_H +#include + struct input_dev; struct input_mt_pos; +struct touchscreen_softbutton_info; struct touchscreen_properties { unsigned int max_x; @@ -32,4 +35,10 @@ void touchscreen_report_pos(struct input_dev *input, unsigned int x, unsigned int y, bool multitouch); +struct touchscreen_softbutton_info *devm_touchscreen_alloc_softbuttons( + struct input_dev *input); + +bool touchscreen_handle_softbuttons(struct touchscreen_softbutton_info *info, + unsigned int x, unsigned int y, bool down); + #endif