On Thu, 12 Aug 2010, Chris Ball wrote: > The JooJoo tablet (http://thejoojoo.com/) contains an "eGalax Inc. USB > TouchController", and this patch hooks it up to the egalax-touch driver. > Without the patch we don't get any cursor motion, since it comes through > Z/RX rather than X/Y. > > Signed-off-by: Chris Ball <cjb@xxxxxxxxxx> > --- > drivers/hid/hid-egalax.c | 2 ++ > drivers/hid/hid-ids.h | 1 + > 2 files changed, 3 insertions(+), 0 deletions(-) > > diff --git a/drivers/hid/hid-egalax.c b/drivers/hid/hid-egalax.c > index f44bdc0..22e3b35 100644 > --- a/drivers/hid/hid-egalax.c > +++ b/drivers/hid/hid-egalax.c > @@ -246,6 +246,8 @@ static void egalax_remove(struct hid_device *hdev) > static const struct hid_device_id egalax_devices[] = { > { HID_USB_DEVICE(USB_VENDOR_ID_DWAV, > USB_DEVICE_ID_DWAV_EGALAX_MULTITOUCH) }, > + { HID_USB_DEVICE(USB_VENDOR_ID_DWAV, > + USB_DEVICE_ID_DWAV_EGALAX_MULTITOUCH1) }, > { } > }; > MODULE_DEVICE_TABLE(hid, egalax_devices); > diff --git a/drivers/hid/hid-ids.h b/drivers/hid/hid-ids.h > index 6af77ed..01c3009 100644 > --- a/drivers/hid/hid-ids.h > +++ b/drivers/hid/hid-ids.h > @@ -185,6 +185,7 @@ > #define USB_VENDOR_ID_DWAV 0x0eef > #define USB_DEVICE_ID_EGALAX_TOUCHCONTROLLER 0x0001 > #define USB_DEVICE_ID_DWAV_EGALAX_MULTITOUCH 0x480d > +#define USB_DEVICE_ID_DWAV_EGALAX_MULTITOUCH1 0x720c > > #define USB_VENDOR_ID_ELO 0x04E7 > #define USB_DEVICE_ID_ELO_TS2700 0x0020 Hi Chris, has this patch been really tested? I am almost certain that we need to have the device ID to be added to hid_blacklist[] in drivers/hid/hid-core.c as well. Otherwise the hid-egalax driver wouldn't bind to the device automatically in a proper way. Thanks, -- Jiri Kosina SUSE Labs, Novell Inc. -- To unsubscribe from this list: send the line "unsubscribe linux-usb" in the body of a message to majordomo@xxxxxxxxxxxxxxx More majordomo info at http://vger.kernel.org/majordomo-info.html
