Re: Commit c64f6f934c7490faff76faf96217066a1b3570a0 breaks my MosArt touchscreen

2 messages, 2 authors, 2011-03-09 · open the first message on its own page

Re: Commit c64f6f934c7490faff76faf96217066a1b3570a0 breaks my MosArt touchscreen

From: Benjamin Tissoires <hidden>
Date: 2011-03-09 12:19:59

Hi James,

Ouch, sorry for that. In fact, hid-mosart is buggy since its creation
for the relative/absolute behavior you are observing.
A patch corrects this and has been included in 2.6.38, but it has not
been backported to 2.6.37.
The name of the patch is "HID: hid-mosart: ignore buttons report".

Jiri, Greg, do you think we can include this in the stable queue?

Cheers,
Benjamin


On Wed, Mar 9, 2011 at 11:28, James Sharam [off-list ref] wrote:
Dear Benjamin,
I've recently upgraded to kernel 2.6.37.2 from 2.6.37.1 and have run in to
problems with my MosArt touchscreen that I think is related to
commit c64f6f934c7490faff76faf96217066a1b3570a0 that you signed off. The
details are in this kernel bug report that I have just
posted: https://bugzilla.kernel.org/show_bug.cgi?id=30792 .
Would you mind having a quick look to see if this is easily resolvable?
Thanks,
- James Sharam
--
To unsubscribe from this list: send the line "unsubscribe linux-input" in
the body of a message to majordomo@vger.kernel.org
More majordomo info at  http://vger.kernel.org/majordomo-info.html

Re: Commit c64f6f934c7490faff76faf96217066a1b3570a0 breaks my MosArt touchscreen

From: Jiri Kosina <hidden>
Date: 2011-03-09 12:28:50

On Wed, 9 Mar 2011, Benjamin Tissoires wrote:
Ouch, sorry for that. In fact, hid-mosart is buggy since its creation
for the relative/absolute behavior you are observing.
A patch corrects this and has been included in 2.6.38, but it has not
been backported to 2.6.37.
The name of the patch is "HID: hid-mosart: ignore buttons report".

Jiri, Greg, do you think we can include this in the stable queue?
Absolutely. Adding stable@kernel.org to CC: and inlcuding the patch below.



[ upstream commit ad6d42670279d ]

From: Benjamin Tissoires <redacted>
Subject: [PATCH] HID: hid-mosart: ignore buttons report

This commit allows the device to be recognized as a touchscreen, and not a
touchpad by xf86-input-evdev.

The device has 2 modes. The first one is an emulation of a touchscreen by
sending left and right button, and the second mode is the one used in
dual-touch (sending trackingID, touch and else).

That's why there is a hid report containing left and right buttons
(9000001 and 9000002). The point is that xorg relies on these fields to
determine if it's a touchpad or a touchscreen.
Clearing the report (return -1) makes xorg detecting it out of the box
as a quite pleasant (dual)touchscreen.

Signed-off-by: Benjamin Tissoires <redacted>
Acked-by: Chase Douglas <redacted>
Signed-off-by: Jiri Kosina <redacted>
---
 drivers/hid/hid-mosart.c |    4 ++++
 1 files changed, 4 insertions(+), 0 deletions(-)
diff --git a/drivers/hid/hid-mosart.c b/drivers/hid/hid-mosart.c
index ac5421d..acd8a49 100644
--- a/drivers/hid/hid-mosart.c
+++ b/drivers/hid/hid-mosart.c
@@ -90,6 +90,10 @@ static int mosart_input_mapping(struct hid_device *hdev, struct hid_input *hi,
 	case 0xff000000:
 		/* ignore HID features */
 		return -1;
+
+	case HID_UP_BUTTON:
+		/* ignore buttons */
+		return -1;
 	}
 
 	return 0;
-- 
1.7.3.1
Keyboard shortcuts
hback out one level
jnext message in thread
kprevious message in thread
ldrill in
Escclose help / fold thread tree
?toggle this help