Re: [PATCH 1/1] hid: prevent ILITEK multi-touch devices from beinghandled by usbhid

[Jiri Kosina]

On Sat, 11 Sep 2010, Bradley Chen wrote:

> From: Bradley Chen <bradley_chen@xxxxxxxxxx>
> 
> The ILITEK multi-touch devices are incorrectly handled by the generic 
> USB driver(usbhid). This patch puts the IDs of ILITEK and its 
> multi-touch devices into hid_blacklist in order to have the products be 
> handled by specific drivers.
> 
> Signed-off-by: Bradley Chen <bradley_chen@xxxxxxxxxx>
> ---
> diff -uprN -X linux-2.6.35.4-vanilla/Documentation/dontdiff linux-2.6.35.4-vanilla/drivers/hid/hid-core.c linux-2.6.35.4/drivers/hid
> /hid-core.c
> --- linux-2.6.35.4-vanilla/drivers/hid/hid-core.c	2010-09-11 16:12:25.295859677 +0800
> +++ linux-2.6.35.4/drivers/hid/hid-core.c	2010-08-27 07:47:12.000000000 +0800
> @@ -1302,7 +1302,6 @@ static const struct hid_device_id hid_bl
>  	{ HID_USB_DEVICE(USB_VENDOR_ID_GYRATION, USB_DEVICE_ID_GYRATION_REMOTE) },
>  	{ HID_USB_DEVICE(USB_VENDOR_ID_GYRATION, USB_DEVICE_ID_GYRATION_REMOTE_2) },
>  	{ HID_USB_DEVICE(USB_VENDOR_ID_GYRATION, USB_DEVICE_ID_GYRATION_REMOTE_3) },
> -	{ HID_USB_DEVICE(USB_VENDOR_ID_ILITEK, USB_DEVICE_ID_ILITEK_MT2101) },
>  	{ HID_USB_DEVICE(USB_VENDOR_ID_KENSINGTON, USB_DEVICE_ID_KS_SLIMBLADE) },
>  	{ HID_USB_DEVICE(USB_VENDOR_ID_KYE, USB_DEVICE_ID_KYE_ERGO_525V) },
>  	{ HID_USB_DEVICE(USB_VENDOR_ID_LABTEC, USB_DEVICE_ID_LABTEC_WIRELESS_KEYBOARD) },
> diff -uprN -X linux-2.6.35.4-vanilla/Documentation/dontdiff linux-2.6.35.4-vanilla/drivers/hid/hid-ids.h linux-2.6.35.4/drivers/hid/
> hid-ids.h
> --- linux-2.6.35.4-vanilla/drivers/hid/hid-ids.h	2010-09-11 16:10:37.704360067 +0800
> +++ linux-2.6.35.4/drivers/hid/hid-ids.h	2010-08-27 07:47:12.000000000 +0800
> @@ -290,9 +290,6 @@
>  #define USB_DEVICE_ID_UGCI_FLYING	0x0020
>  #define USB_DEVICE_ID_UGCI_FIGHTING	0x0030
>  
> -#define USB_VENDOR_ID_ILITEK		0x222a
> -#define USB_DEVICE_ID_ILITEK_MT2101 	0x0001
> -
>  #define USB_VENDOR_ID_IMATION		0x0718
>  #define USB_DEVICE_ID_DISC_STAKKA	0xd000

Hi Bradley,

first, it seems that the patch is actually doing quite the reverse of what 
you wanted to do, right? (it's removing lines that have never been in 
kernel).

Second, what are the specific drivers? Are they going to be posted for 
upstream kernel inclusion?
How does the device behave with USB HID driver? It doesn't work at all, or 
it's just that it only providing basic HID capabilities without any 
vendor-specific extensions?

Thanks,

-- 
Jiri Kosina
SUSE Labs, Novell Inc.
--
To unsubscribe from this list: send the line "unsubscribe linux-kernel" in
the body of a message to majordomo@xxxxxxxxxxxxxxx
More majordomo info at  http://vger.kernel.org/majordomo-info.html
Please read the FAQ at  http://www.tux.org/lkml/