Commit f7bc8046 authored by Henrik Rydberg's avatar Henrik Rydberg

hid: egalax: Convert to MT slots

The firmware in the joojoo reports touches sequentially, one per
report, which confuses the current driver. A further complication is
the absense of any indication of a touch frame. This patch converts
the driver to the MT slots protocol, and outputs one full touch frame
per report. This way, proper handling for both firmwares is ensured.
Tested-by: default avatarPhilipp Merkel <mail@philmerk.de>
Cc: Stephane Chatty <chatty@enac.fr>
Signed-off-by: default avatarJiri Kosina <jkosina@suse.cz>
Signed-off-by: default avatarHenrik Rydberg <rydberg@euromail.se>
parent 17c76068
...@@ -2,6 +2,8 @@ ...@@ -2,6 +2,8 @@
* HID driver for eGalax dual-touch panels * HID driver for eGalax dual-touch panels
* *
* Copyright (c) 2010 Stephane Chatty <chatty@enac.fr> * Copyright (c) 2010 Stephane Chatty <chatty@enac.fr>
* Copyright (c) 2010 Henrik Rydberg <rydberg@euromail.se>
* Copyright (c) 2010 Canonical, Ltd.
* *
*/ */
...@@ -16,6 +18,7 @@ ...@@ -16,6 +18,7 @@
#include <linux/hid.h> #include <linux/hid.h>
#include <linux/module.h> #include <linux/module.h>
#include <linux/usb.h> #include <linux/usb.h>
#include <linux/input/mt.h>
#include <linux/slab.h> #include <linux/slab.h>
#include "usbhid/usbhid.h" #include "usbhid/usbhid.h"
...@@ -25,17 +28,17 @@ MODULE_LICENSE("GPL"); ...@@ -25,17 +28,17 @@ MODULE_LICENSE("GPL");
#include "hid-ids.h" #include "hid-ids.h"
#define MAX_SLOTS 2
/* estimated signal-to-noise ratios */ /* estimated signal-to-noise ratios */
#define SN_MOVE 4096 #define SN_MOVE 4096
#define SN_PRESSURE 32 #define SN_PRESSURE 32
struct egalax_data { struct egalax_data {
__u16 x, y, z; int valid;
__u8 id; int slot;
bool first; /* is this the first finger in the frame? */ int touch;
bool valid; /* valid finger data, or just placeholder? */ int x, y, z;
bool activity; /* at least one active finger previously? */
__u16 lastx, lasty, lastz; /* latest valid (x, y, z) in the frame */
}; };
static void set_abs(struct input_dev *input, unsigned int code, static void set_abs(struct input_dev *input, unsigned int code,
...@@ -89,9 +92,7 @@ static int egalax_input_mapping(struct hid_device *hdev, struct hid_input *hi, ...@@ -89,9 +92,7 @@ static int egalax_input_mapping(struct hid_device *hdev, struct hid_input *hi,
case HID_DG_CONTACTMAX: case HID_DG_CONTACTMAX:
return -1; return -1;
case HID_DG_CONTACTID: case HID_DG_CONTACTID:
hid_map_usage(hi, usage, bit, max, input_mt_init_slots(input, MAX_SLOTS);
EV_ABS, ABS_MT_TRACKING_ID);
set_abs(input, ABS_MT_TRACKING_ID, field, 0);
return 1; return 1;
case HID_DG_TIPPRESSURE: case HID_DG_TIPPRESSURE:
field->logical_minimum = 0; field->logical_minimum = 0;
...@@ -125,58 +126,16 @@ static int egalax_input_mapped(struct hid_device *hdev, struct hid_input *hi, ...@@ -125,58 +126,16 @@ static int egalax_input_mapped(struct hid_device *hdev, struct hid_input *hi,
*/ */
static void egalax_filter_event(struct egalax_data *td, struct input_dev *input) static void egalax_filter_event(struct egalax_data *td, struct input_dev *input)
{ {
td->first = !td->first; /* touchscreen emulation */ input_mt_slot(input, td->slot);
input_mt_report_slot_state(input, MT_TOOL_FINGER, td->touch);
if (td->valid) { if (td->touch) {
/* emit multitouch events */ input_event(input, EV_ABS, ABS_MT_POSITION_X, td->x);
input_event(input, EV_ABS, ABS_MT_TRACKING_ID, td->id); input_event(input, EV_ABS, ABS_MT_POSITION_Y, td->y);
input_event(input, EV_ABS, ABS_MT_POSITION_X, td->x >> 3);
input_event(input, EV_ABS, ABS_MT_POSITION_Y, td->y >> 3);
input_event(input, EV_ABS, ABS_MT_PRESSURE, td->z); input_event(input, EV_ABS, ABS_MT_PRESSURE, td->z);
input_mt_sync(input);
/*
* touchscreen emulation: store (x, y) as
* the last valid values in this frame
*/
td->lastx = td->x;
td->lasty = td->y;
td->lastz = td->z;
}
/*
* touchscreen emulation: if this is the second finger and at least
* one in this frame is valid, the latest valid in the frame is
* the oldest on the panel, the one we want for single touch
*/
if (!td->first && td->activity) {
input_event(input, EV_ABS, ABS_X, td->lastx >> 3);
input_event(input, EV_ABS, ABS_Y, td->lasty >> 3);
input_event(input, EV_ABS, ABS_PRESSURE, td->lastz);
}
if (!td->valid) {
/*
* touchscreen emulation: if the first finger is invalid
* and there previously was finger activity, this is a release
*/
if (td->first && td->activity) {
input_event(input, EV_KEY, BTN_TOUCH, 0);
td->activity = false;
}
return;
}
/* touchscreen emulation: if no previous activity, emit touch event */
if (!td->activity) {
input_event(input, EV_KEY, BTN_TOUCH, 1);
td->activity = true;
} }
input_mt_report_pointer_emulation(input, true);
} }
static int egalax_event(struct hid_device *hid, struct hid_field *field, static int egalax_event(struct hid_device *hid, struct hid_field *field,
struct hid_usage *usage, __s32 value) struct hid_usage *usage, __s32 value)
{ {
...@@ -186,25 +145,26 @@ static int egalax_event(struct hid_device *hid, struct hid_field *field, ...@@ -186,25 +145,26 @@ static int egalax_event(struct hid_device *hid, struct hid_field *field,
* uses a standard parallel multitouch protocol (product ID == * uses a standard parallel multitouch protocol (product ID ==
* 48xx). The second is capacitive and uses an unusual "serial" * 48xx). The second is capacitive and uses an unusual "serial"
* protocol with a different message for each multitouch finger * protocol with a different message for each multitouch finger
* (product ID == 72xx). We do not yet generate a correct event * (product ID == 72xx).
* sequence for the capacitive/serial protocol.
*/ */
if (hid->claimed & HID_CLAIMED_INPUT) { if (hid->claimed & HID_CLAIMED_INPUT) {
struct input_dev *input = field->hidinput->input; struct input_dev *input = field->hidinput->input;
switch (usage->hid) { switch (usage->hid) {
case HID_DG_INRANGE: case HID_DG_INRANGE:
td->valid = value;
break;
case HID_DG_CONFIDENCE: case HID_DG_CONFIDENCE:
/* avoid interference from generic hidinput handling */ /* avoid interference from generic hidinput handling */
break; break;
case HID_DG_TIPSWITCH: case HID_DG_TIPSWITCH:
td->valid = value; td->touch = value;
break; break;
case HID_DG_TIPPRESSURE: case HID_DG_TIPPRESSURE:
td->z = value; td->z = value;
break; break;
case HID_DG_CONTACTID: case HID_DG_CONTACTID:
td->id = value; td->slot = clamp_val(value, 0, MAX_SLOTS - 1);
break; break;
case HID_GD_X: case HID_GD_X:
td->x = value; td->x = value;
...@@ -212,11 +172,11 @@ static int egalax_event(struct hid_device *hid, struct hid_field *field, ...@@ -212,11 +172,11 @@ static int egalax_event(struct hid_device *hid, struct hid_field *field,
case HID_GD_Y: case HID_GD_Y:
td->y = value; td->y = value;
/* this is the last field in a finger */ /* this is the last field in a finger */
if (td->valid)
egalax_filter_event(td, input); egalax_filter_event(td, input);
break; break;
case HID_DG_CONTACTCOUNT: case HID_DG_CONTACTCOUNT:
/* touch emulation: this is the last field in a frame */ /* touch emulation: this is the last field in a frame */
td->first = false;
break; break;
default: default:
......
Markdown is supported
0%
or
You are about to add 0 people to the discussion. Proceed with caution.
Finish editing this message first!
Please register or to comment