[Precise SRU][PATCH 3/3] Input: elantech - v4 is a clickpad, with only one button

joseph.salisbury at canonical.com joseph.salisbury at canonical.com
Thu Sep 13 18:00:58 UTC 2012

From: JJ Ding <jj_ding at emc.com.tw>

BugLink: http://bugs.launchpad.net/bugs/1033568

Add pointer and buttonpad properties for v4 hardware.

Also, Jachiet reported that on Asus UX31, right button has no effect.
It turns out v4 has only one button, the right-button effect is
implemented with software when Windows driver is installed, or in
firmware when touchpad is in relative mode. So remove BTN_RIGHT
while at it.

Reported-by: Jachiet Louis <louis at jachiet.com>
Signed-off-by: JJ Ding <jj_ding at emc.com.tw>
Reviewed-by: Chase Douglas <chase.douglas at canonical.com>
Signed-off-by: Dmitry Torokhov <dtor at mail.ru>
(cherry picked from commit e3dde4fba94e0ba5e1fd79ea9e5389eea1f0cfec)

Signed-off-by: Joseph Salisbury <joseph.salisbury at canonical.com>
 drivers/input/mouse/elantech.c |    6 ++++--
 1 file changed, 4 insertions(+), 2 deletions(-)

diff --git a/drivers/input/mouse/elantech.c b/drivers/input/mouse/elantech.c
index 7da63e5..d804146 100644
--- a/drivers/input/mouse/elantech.c
+++ b/drivers/input/mouse/elantech.c
@@ -486,7 +486,6 @@ static void elantech_input_sync_v4(struct psmouse *psmouse)
 	unsigned char *packet = psmouse->packet;
 	input_report_key(dev, BTN_LEFT, packet[0] & 0x01);
-	input_report_key(dev, BTN_RIGHT, packet[0] & 0x02);
 	input_mt_report_pointer_emulation(dev, true);
@@ -967,6 +966,7 @@ static int elantech_set_input_params(struct psmouse *psmouse)
 	if (elantech_set_range(psmouse, &x_min, &y_min, &x_max, &y_max, &width))
 		return -1;
+	__set_bit(INPUT_PROP_POINTER, dev->propbit);
 	__set_bit(EV_KEY, dev->evbit);
 	__set_bit(EV_ABS, dev->evbit);
 	__clear_bit(EV_REL, dev->evbit);
@@ -1017,7 +1017,9 @@ static int elantech_set_input_params(struct psmouse *psmouse)
 			psmouse_warn(psmouse, "couldn't query resolution data.\n");
+		/* v4 is clickpad, with only one button. */
+		__set_bit(INPUT_PROP_BUTTONPAD, dev->propbit);
+		__clear_bit(BTN_RIGHT, dev->keybit);
 		__set_bit(BTN_TOOL_QUADTAP, dev->keybit);
 		/* For X to recognize me as touchpad. */
 		input_set_abs_params(dev, ABS_X, x_min, x_max, 0, 0);

More information about the kernel-team mailing list