Re: [patch] Problem with mousedev.c

[Zephaniah E\. Hull]

--wq9mPyueHGvFACwf
Content-Type: multipart/mixed; boundary="bp/iNruPH9dso1Pn"
Content-Disposition: inline

--bp/iNruPH9dso1Pn
Content-Type: text/plain; charset=us-ascii
Content-Disposition: inline
Content-Transfer-Encoding: quoted-printable

On Wed, Oct 30, 2002 at 04:59:22PM +0100, Vojtech Pavlik wrote:
> On Wed, Oct 30, 2002 at 10:32:57AM -0500, Zephaniah E. Hull wrote:
> > Sadly, if PS/2 mice are any indication, mouse makers /will/ manage to
> > fuck things up on enough popular mice under USB as well, and there needs
> > to be a place to shove the dirty hacks needed to make things Just Work
> > for users..
>=20
> That place would be hid-input.c and psmouse.c. NOT mousedev.c.

Agreed, at the moment my patch for dealing with the A4 mouse hits three
files in the HID layer, and is not the cleanest, however it is attached
for reference.

Now if I could only find (better) documentation on the various PS2
protocols to find one that has in the spec itself handling for multiple
wheels..

--=20
	1024D/E65A7801 Zephaniah E. Hull <[e-mail]>
	   92ED 94E4 B1E6 3624 226D  5727 4453 008B E65A 7801
	    CCs of replies from mailing lists are requested.

"I would rather spend 10 hours reading someone else's source code than
10 minutes listening to Musak waiting for technical support which
isn't."
(By Dr. Greg Wettstein, Roger Maris Cancer Center)

--bp/iNruPH9dso1Pn
Content-Type: text/plain; charset=us-ascii
Content-Disposition: attachment; filename="hid_hack.diff"
Content-Transfer-Encoding: quoted-printable

diff -ur linux.orig/drivers/usb/hid-core.c linux/drivers/usb/hid-core.c
--- linux.orig/drivers/usb/hid-core.c	2002-10-26 20:31:21.000000000 -0400
+++ linux/drivers/usb/hid-core.c	2002-10-27 01:45:04.000000000 -0400
@@ -1086,6 +1086,8 @@
 #define USB_DEVICE_ID_ATEN_2PORTKVM	0x2204
 #define USB_DEVICE_ID_ATEN_4PORTKVM	0x2205
=20
+#define USB_VENDOR_ID_A4TECH		0x09DA
+#define USB_DEVICE_ID_A4TECH_WCP32PU	0x0006
 struct hid_blacklist {
 	__u16 idVendor;
 	__u16 idProduct;
@@ -1115,6 +1117,7 @@
 	{ USB_VENDOR_ID_ATEN, USB_DEVICE_ID_ATEN_CS124U, HID_QUIRK_NOGET },
 	{ USB_VENDOR_ID_ATEN, USB_DEVICE_ID_ATEN_2PORTKVM, HID_QUIRK_NOGET },
 	{ USB_VENDOR_ID_ATEN, USB_DEVICE_ID_ATEN_4PORTKVM, HID_QUIRK_NOGET },
+	{ USB_VENDOR_ID_A4TECH, USB_DEVICE_ID_A4TECH_WCP32PU, HID_QUIRK_2WHEEL_MO=
USE_HACK },
 	{ 0, 0 }
 };
=20
diff -ur linux.orig/drivers/usb/hid-input.c linux/drivers/usb/hid-input.c
--- linux.orig/drivers/usb/hid-input.c	2001-11-11 13:09:37.000000000 -0500
+++ linux/drivers/usb/hid-input.c	2002-10-27 03:36:52.000000000 -0500
@@ -269,6 +269,11 @@
 	}
=20
 	set_bit(usage->type, input->evbit);
+	if ((usage->type =3D=3D EV_REL)
+			&& (device->quirks & HID_QUIRK_2WHEEL_MOUSE_HACK)
+			&& (usage->code =3D=3D REL_WHEEL)) {
+		set_bit(REL_HWHEEL, bit);
+	}
=20
 	while (usage->code <=3D max && test_and_set_bit(usage->code, bit)) {
 		usage->code =3D find_next_zero_bit(bit, max + 1, usage->code);
@@ -303,6 +308,20 @@
 	struct input_dev *input =3D &hid->input;
 	int *quirks =3D &hid->quirks;
=20
+	if ((hid->quirks & HID_QUIRK_2WHEEL_MOUSE_HACK)
+			&& (usage->code =3D=3D BTN_BACK)) {
+		if (value)
+			hid->quirks |=3D HID_QUIRK_2WHEEL_MOUSE_HACK_ON;
+		else
+			hid->quirks &=3D ~HID_QUIRK_2WHEEL_MOUSE_HACK_ON;
+		return;
+	}
+	if ((hid->quirks & HID_QUIRK_2WHEEL_MOUSE_HACK_ON)
+			&& (usage->code =3D=3D REL_WHEEL)) {
+		input_event(input, usage->type, REL_HWHEEL, value);
+		return;
+	}
+
 	if (usage->hat_min !=3D usage->hat_max) {
 		value =3D (value - usage->hat_min) * 8 / (usage->hat_max - usage->hat_mi=
n + 1) + 1;
 		if (value < 0 || value > 8) value =3D 0;
diff -ur linux.orig/drivers/usb/hid.h linux/drivers/usb/hid.h
--- linux.orig/drivers/usb/hid.h	2002-10-26 20:31:21.000000000 -0400
+++ linux/drivers/usb/hid.h	2002-10-27 01:43:23.000000000 -0400
@@ -186,6 +186,8 @@
 #define HID_QUIRK_NOTOUCH	0x02
 #define HID_QUIRK_IGNORE	0x04
 #define HID_QUIRK_NOGET		0x08
+#define HID_QUIRK_2WHEEL_MOUSE_HACK	0x10
+#define HID_QUIRK_2WHEEL_MOUSE_HACK_ON	0x20
=20
 /*
  * This is the global enviroment of the parser. This information is

--bp/iNruPH9dso1Pn--

--wq9mPyueHGvFACwf
Content-Type: application/pgp-signature
Content-Disposition: inline

-----BEGIN PGP SIGNATURE-----
Version: GnuPG v1.2.1 (GNU/Linux)

iD8DBQE9wAMYRFMAi+ZaeAERAthzAJwKJ9lqLF43RoW4GwoN5KZE42NRiACg7o6d
DiljPaA1eJXoqi1dmd9lq5s=
=HSLN
-----END PGP SIGNATURE-----

--wq9mPyueHGvFACwf--