On Mon, 14 May 2012, Nikolai Kondrashov wrote: > Add support for UC-Logic Wireless Tablet TWHL850. > It is known to be sold as Genius MousePen M508W. > > This tablet has a bug in the default (compatibility) mode which is used in this > driver: frame button assignments are mixed up. This is to be fixed with a driver > using proprietary protocol, which is to be fully reverse-engineered yet. > > Signed-off-by: Nikolai Kondrashov <spbnick@xxxxxxxxx> > --- > > This version has IFACE_RDESC_FIXUP macro expanded as requested by Jiri. > > drivers/hid/hid-core.c | 1 + > drivers/hid/hid-ids.h | 1 + > drivers/hid/hid-uclogic.c | 141 +++++++++++++++++++++++++++++++++++++++++++++ > 3 files changed, 143 insertions(+) > > diff --git a/drivers/hid/hid-core.c b/drivers/hid/hid-core.c > index 4da66b4..c25fc49 100644 > --- a/drivers/hid/hid-core.c > +++ b/drivers/hid/hid-core.c > @@ -1562,6 +1562,7 @@ static const struct hid_device_id hid_have_special_driver[] = { > { HID_USB_DEVICE(USB_VENDOR_ID_UCLOGIC, USB_DEVICE_ID_UCLOGIC_TABLET_WP5540U) }, > { HID_USB_DEVICE(USB_VENDOR_ID_UCLOGIC, USB_DEVICE_ID_UCLOGIC_TABLET_WP8060U) }, > { HID_USB_DEVICE(USB_VENDOR_ID_UCLOGIC, USB_DEVICE_ID_UCLOGIC_TABLET_WP1062) }, > + { HID_USB_DEVICE(USB_VENDOR_ID_UCLOGIC, USB_DEVICE_ID_UCLOGIC_WIRELESS_TABLET_TWHL850) }, > { HID_USB_DEVICE(USB_VENDOR_ID_UNITEC, USB_DEVICE_ID_UNITEC_USB_TOUCH_0709) }, > { HID_USB_DEVICE(USB_VENDOR_ID_UNITEC, USB_DEVICE_ID_UNITEC_USB_TOUCH_0A19) }, > { HID_USB_DEVICE(USB_VENDOR_ID_WISEGROUP, USB_DEVICE_ID_SMARTJOY_PLUS) }, > diff --git a/drivers/hid/hid-ids.h b/drivers/hid/hid-ids.h > index e39aecb..ad8d743 100644 > --- a/drivers/hid/hid-ids.h > +++ b/drivers/hid/hid-ids.h > @@ -726,6 +726,7 @@ > #define USB_DEVICE_ID_UCLOGIC_TABLET_WP5540U 0x0004 > #define USB_DEVICE_ID_UCLOGIC_TABLET_WP8060U 0x0005 > #define USB_DEVICE_ID_UCLOGIC_TABLET_WP1062 0x0064 > +#define USB_DEVICE_ID_UCLOGIC_WIRELESS_TABLET_TWHL850 0x0522 > > #define USB_VENDOR_ID_UNITEC 0x227d > #define USB_DEVICE_ID_UNITEC_USB_TOUCH_0709 0x0709 > diff --git a/drivers/hid/hid-uclogic.c b/drivers/hid/hid-uclogic.c > index 1f11289..3aba02b 100644 > --- a/drivers/hid/hid-uclogic.c > +++ b/drivers/hid/hid-uclogic.c > @@ -14,6 +14,7 @@ > #include <linux/device.h> > #include <linux/hid.h> > #include <linux/module.h> > +#include <linux/usb.h> > > #include "hid-ids.h" > > @@ -352,9 +353,125 @@ static __u8 pf1209_rdesc_fixed[] = { > 0xC0 /* End Collection */ > }; > > +/* > + * See TWHL850 description, device and HID report descriptors at > + * http://sf.net/apps/mediawiki/digimend/?title=UC-Logic_Wireless_Tablet_TWHL850 > + */ > + > +/* Size of the original descriptors of TWHL850 tablet */ > +#define TWHL850_RDESC_ORIG_SIZE0 182 > +#define TWHL850_RDESC_ORIG_SIZE1 161 > +#define TWHL850_RDESC_ORIG_SIZE2 92 > + > +/* Fixed PID 0522 tablet report descriptor, interface 0 (stylus) */ > +static __u8 twhl850_rdesc_fixed0[] = { > + 0x05, 0x0D, /* Usage Page (Digitizer), */ > + 0x09, 0x02, /* Usage (Pen), */ > + 0xA1, 0x01, /* Collection (Application), */ > + 0x85, 0x09, /* Report ID (9), */ > + 0x09, 0x20, /* Usage (Stylus), */ > + 0xA0, /* Collection (Physical), */ > + 0x14, /* Logical Minimum (0), */ > + 0x25, 0x01, /* Logical Maximum (1), */ > + 0x75, 0x01, /* Report Size (1), */ > + 0x95, 0x03, /* Report Count (3), */ > + 0x09, 0x42, /* Usage (Tip Switch), */ > + 0x09, 0x44, /* Usage (Barrel Switch), */ > + 0x09, 0x46, /* Usage (Tablet Pick), */ > + 0x81, 0x02, /* Input (Variable), */ > + 0x81, 0x03, /* Input (Constant, Variable), */ > + 0x95, 0x01, /* Report Count (1), */ > + 0x09, 0x32, /* Usage (In Range), */ > + 0x81, 0x02, /* Input (Variable), */ > + 0x81, 0x03, /* Input (Constant, Variable), */ > + 0x75, 0x10, /* Report Size (16), */ > + 0xA4, /* Push, */ > + 0x05, 0x01, /* Usage Page (Desktop), */ > + 0x65, 0x13, /* Unit (Inch), */ > + 0x55, 0xFD, /* Unit Exponent (-3), */ > + 0x34, /* Physical Minimum (0), */ > + 0x09, 0x30, /* Usage (X), */ > + 0x46, 0x40, 0x1F, /* Physical Maximum (8000), */ > + 0x26, 0x00, 0x7D, /* Logical Maximum (32000), */ > + 0x81, 0x02, /* Input (Variable), */ > + 0x09, 0x31, /* Usage (Y), */ > + 0x46, 0x88, 0x13, /* Physical Maximum (5000), */ > + 0x26, 0x20, 0x4E, /* Logical Maximum (20000), */ > + 0x81, 0x02, /* Input (Variable), */ > + 0xB4, /* Pop, */ > + 0x09, 0x30, /* Usage (Tip Pressure), */ > + 0x26, 0xFF, 0x03, /* Logical Maximum (1023), */ > + 0x81, 0x02, /* Input (Variable), */ > + 0xC0, /* End Collection, */ > + 0xC0 /* End Collection */ > +}; > + > +/* Fixed PID 0522 tablet report descriptor, interface 1 (mouse) */ > +static __u8 twhl850_rdesc_fixed1[] = { > + 0x05, 0x01, /* Usage Page (Desktop), */ > + 0x09, 0x02, /* Usage (Mouse), */ > + 0xA1, 0x01, /* Collection (Application), */ > + 0x85, 0x01, /* Report ID (1), */ > + 0x09, 0x01, /* Usage (Pointer), */ > + 0xA0, /* Collection (Physical), */ > + 0x05, 0x09, /* Usage Page (Button), */ > + 0x75, 0x01, /* Report Size (1), */ > + 0x95, 0x03, /* Report Count (3), */ > + 0x19, 0x01, /* Usage Minimum (01h), */ > + 0x29, 0x03, /* Usage Maximum (03h), */ > + 0x14, /* Logical Minimum (0), */ > + 0x25, 0x01, /* Logical Maximum (1), */ > + 0x81, 0x02, /* Input (Variable), */ > + 0x95, 0x05, /* Report Count (5), */ > + 0x81, 0x03, /* Input (Constant, Variable), */ > + 0x05, 0x01, /* Usage Page (Desktop), */ > + 0x09, 0x30, /* Usage (X), */ > + 0x09, 0x31, /* Usage (Y), */ > + 0x16, 0x00, 0x80, /* Logical Minimum (-32768), */ > + 0x26, 0xFF, 0x7F, /* Logical Maximum (32767), */ > + 0x75, 0x10, /* Report Size (16), */ > + 0x95, 0x02, /* Report Count (2), */ > + 0x81, 0x06, /* Input (Variable, Relative), */ > + 0x09, 0x38, /* Usage (Wheel), */ > + 0x15, 0xFF, /* Logical Minimum (-1), */ > + 0x25, 0x01, /* Logical Maximum (1), */ > + 0x95, 0x01, /* Report Count (1), */ > + 0x75, 0x08, /* Report Size (8), */ > + 0x81, 0x06, /* Input (Variable, Relative), */ > + 0x81, 0x03, /* Input (Constant, Variable), */ > + 0xC0, /* End Collection, */ > + 0xC0 /* End Collection */ > +}; > + > +/* Fixed PID 0522 tablet report descriptor, interface 2 (frame buttons) */ > +static __u8 twhl850_rdesc_fixed2[] = { > + 0x05, 0x01, /* Usage Page (Desktop), */ > + 0x09, 0x06, /* Usage (Keyboard), */ > + 0xA1, 0x01, /* Collection (Application), */ > + 0x85, 0x03, /* Report ID (3), */ > + 0x05, 0x07, /* Usage Page (Keyboard), */ > + 0x14, /* Logical Minimum (0), */ > + 0x19, 0xE0, /* Usage Minimum (KB Leftcontrol), */ > + 0x29, 0xE7, /* Usage Maximum (KB Right GUI), */ > + 0x25, 0x01, /* Logical Maximum (1), */ > + 0x75, 0x01, /* Report Size (1), */ > + 0x95, 0x08, /* Report Count (8), */ > + 0x81, 0x02, /* Input (Variable), */ > + 0x18, /* Usage Minimum (None), */ > + 0x29, 0xFF, /* Usage Maximum (FFh), */ > + 0x26, 0xFF, 0x00, /* Logical Maximum (255), */ > + 0x75, 0x08, /* Report Size (8), */ > + 0x95, 0x06, /* Report Count (6), */ > + 0x80, /* Input, */ > + 0xC0 /* End Collection */ > +}; > + > static __u8 *uclogic_report_fixup(struct hid_device *hdev, __u8 *rdesc, > unsigned int *rsize) > { > + struct usb_interface *iface = to_usb_interface(hdev->dev.parent); > + __u8 iface_num = iface->cur_altsetting->desc.bInterfaceNumber; > + > switch (hdev->product) { > case USB_DEVICE_ID_UCLOGIC_TABLET_PF1209: > if (*rsize == PF1209_RDESC_ORIG_SIZE) { > @@ -386,6 +503,28 @@ static __u8 *uclogic_report_fixup(struct hid_device *hdev, __u8 *rdesc, > *rsize = sizeof(wp1062_rdesc_fixed); > } > break; > + case USB_DEVICE_ID_UCLOGIC_WIRELESS_TABLET_TWHL850: > + switch (iface_num) { > + case 0: > + if (*rsize == TWHL850_RDESC_ORIG_SIZE0) { > + rdesc = twhl850_rdesc_fixed0; > + *rsize = sizeof(twhl850_rdesc_fixed0); > + } > + break; > + case 1: > + if (*rsize == TWHL850_RDESC_ORIG_SIZE1) { > + rdesc = twhl850_rdesc_fixed1; > + *rsize = sizeof(twhl850_rdesc_fixed1); > + } > + break; > + case 2: > + if (*rsize == TWHL850_RDESC_ORIG_SIZE2) { > + rdesc = twhl850_rdesc_fixed2; > + *rsize = sizeof(twhl850_rdesc_fixed2); > + } > + break; > + } > + break; > } > > return rdesc; > @@ -402,6 +541,8 @@ static const struct hid_device_id uclogic_devices[] = { > USB_DEVICE_ID_UCLOGIC_TABLET_WP8060U) }, > { HID_USB_DEVICE(USB_VENDOR_ID_UCLOGIC, > USB_DEVICE_ID_UCLOGIC_TABLET_WP1062) }, > + { HID_USB_DEVICE(USB_VENDOR_ID_UCLOGIC, > + USB_DEVICE_ID_UCLOGIC_WIRELESS_TABLET_TWHL850) }, > { } > }; > MODULE_DEVICE_TABLE(hid, uclogic_devices); Applied. Thanks Nikolai. -- Jiri Kosina SUSE Labs -- To unsubscribe from this list: send the line "unsubscribe linux-input" in the body of a message to majordomo@xxxxxxxxxxxxxxx More majordomo info at http://vger.kernel.org/majordomo-info.html