All of lore.kernel.org
 help / color / mirror / Atom feed
* [PATCH] usbhid: Add the malfunctioning SiGma Micro chip to the list of hid-quirks
@ 2011-08-14 14:43 Jeremiah Matthey
  2011-08-16 10:05 ` Jeremiah Matthey
  0 siblings, 1 reply; 5+ messages in thread
From: Jeremiah Matthey @ 2011-08-14 14:43 UTC (permalink / raw)
  To: linux-kernel

Patch to add SiGma Micro-based keyboards (1c4f:0002) to hid-quirks.

These keyboards dont seem to allow the records to be initialized, and hence a
timeout occurs when the usbhid driver attempts to initialize them. The patch
just adds the signature for these keyboards to the hid-quirks list with the
setting HID_QUIRK_NO_INIT_REPORTS. This removes the 5-10 second wait for the
timeout to occur.

Patch was carried out against 3.0.1, although due to the simplicity, will
probably work with HEAD even if someone has changed those files.



diff -ru linux-3.0.1-clean/drivers/hid/hid-ids.h linux-3.0.1-patched/drivers/hid/hid-ids.h
--- linux-3.0.1-clean/drivers/hid/hid-ids.h	2011-08-05 05:59:21.000000000 +0100
+++ linux-3.0.1-patched/drivers/hid/hid-ids.h	2011-08-14 15:25:58.129865922 +0100
@@ -568,6 +568,9 @@
 #define USB_DEVICE_ID_SAMSUNG_IR_REMOTE	0x0001
 #define USB_DEVICE_ID_SAMSUNG_WIRELESS_KBD_MOUSE	0x0600
 
+#define USB_VENDOR_ID_SIGMA_MICRO	0x1c4f
+#define USB_DEVICE_ID_SIGMA_MICRO_KEYBOARD	0x0002
+
 #define USB_VENDOR_ID_SKYCABLE			0x1223
 #define	USB_DEVICE_ID_SKYCABLE_WIRELESS_PRESENTER	0x3F07
 
diff -ru linux-3.0.1-clean/drivers/hid/usbhid/hid-quirks.c linux-3.0.1-patched/drivers/hid/usbhid/hid-quirks.c
--- linux-3.0.1-clean/drivers/hid/usbhid/hid-quirks.c	2011-08-05 05:59:21.000000000 +0100
+++ linux-3.0.1-patched/drivers/hid/usbhid/hid-quirks.c	2011-08-14 15:26:06.289865845 +0100
@@ -89,6 +89,7 @@
 
 	{ USB_VENDOR_ID_CHICONY, USB_DEVICE_ID_CHICONY_MULTI_TOUCH, HID_QUIRK_MULTI_INPUT },
 	{ USB_VENDOR_ID_CHICONY, USB_DEVICE_ID_CHICONY_WIRELESS, HID_QUIRK_MULTI_INPUT },
+	{ USB_VENDOR_ID_SIGMA_MICRO, USB_DEVICE_ID_SIGMA_MICRO_KEYBOARD, HID_QUIRK_NO_INIT_REPORTS },
 	{ 0, 0 }
 };
 

^ permalink raw reply	[flat|nested] 5+ messages in thread

* [PATCH] usbhid: Add the malfunctioning SiGma Micro chip to the list of hid-quirks
  2011-08-14 14:43 [PATCH] usbhid: Add the malfunctioning SiGma Micro chip to the list of hid-quirks Jeremiah Matthey
@ 2011-08-16 10:05 ` Jeremiah Matthey
  2011-08-18 12:31   ` Jiri Kosina
  0 siblings, 1 reply; 5+ messages in thread
From: Jeremiah Matthey @ 2011-08-16 10:05 UTC (permalink / raw)
  To: Jiri Kosina; +Cc: Benjamin_Tissoires, linux-kernel, linux-input

Patch to add SiGma Micro-based keyboards (1c4f:0002) to hid-quirks.

These keyboards dont seem to allow the records to be initialized, and hence a
timeout occurs when the usbhid driver attempts to initialize them. The patch
just adds the signature for these keyboards to the hid-quirks list with the
setting HID_QUIRK_NO_INIT_REPORTS. This removes the 5-10 second wait for the
timeout to occur.

This is technically a resubmit, first time i posted i didnt include the right people in the destination, was just aimed at the list, so might have gone unnoticed. (First (submitted) kernel patch, I'll learn with time :))


diff -ru linux-3.0.1-clean/drivers/hid/hid-ids.h linux-3.0.1-patched/drivers/hid/hid-ids.h
--- linux-3.0.1-clean/drivers/hid/hid-ids.h	2011-08-05 05:59:21.000000000 +0100
+++ linux-3.0.1-patched/drivers/hid/hid-ids.h	2011-08-14 15:25:58.129865922 +0100
@@ -568,6 +568,9 @@
 #define USB_DEVICE_ID_SAMSUNG_IR_REMOTE	0x0001
 #define USB_DEVICE_ID_SAMSUNG_WIRELESS_KBD_MOUSE	0x0600
 
+#define USB_VENDOR_ID_SIGMA_MICRO	0x1c4f
+#define USB_DEVICE_ID_SIGMA_MICRO_KEYBOARD	0x0002
+
 #define USB_VENDOR_ID_SKYCABLE			0x1223
 #define	USB_DEVICE_ID_SKYCABLE_WIRELESS_PRESENTER	0x3F07
 
diff -ru linux-3.0.1-clean/drivers/hid/usbhid/hid-quirks.c linux-3.0.1-patched/drivers/hid/usbhid/hid-quirks.c
--- linux-3.0.1-clean/drivers/hid/usbhid/hid-quirks.c	2011-08-05 05:59:21.000000000 +0100
+++ linux-3.0.1-patched/drivers/hid/usbhid/hid-quirks.c	2011-08-14 15:26:06.289865845 +0100
@@ -89,6 +89,7 @@
 
 	{ USB_VENDOR_ID_CHICONY, USB_DEVICE_ID_CHICONY_MULTI_TOUCH, HID_QUIRK_MULTI_INPUT },
 	{ USB_VENDOR_ID_CHICONY, USB_DEVICE_ID_CHICONY_WIRELESS, HID_QUIRK_MULTI_INPUT },
+	{ USB_VENDOR_ID_SIGMA_MICRO, USB_DEVICE_ID_SIGMA_MICRO_KEYBOARD, HID_QUIRK_NO_INIT_REPORTS },
 	{ 0, 0 }
 };
 

^ permalink raw reply	[flat|nested] 5+ messages in thread

* Re: [PATCH] usbhid: Add the malfunctioning SiGma Micro chip to the list of hid-quirks
  2011-08-16 10:05 ` Jeremiah Matthey
@ 2011-08-18 12:31   ` Jiri Kosina
  2011-08-19 21:59     ` Jeremiah Matthey
  0 siblings, 1 reply; 5+ messages in thread
From: Jiri Kosina @ 2011-08-18 12:31 UTC (permalink / raw)
  To: Jeremiah Matthey; +Cc: Benjamin_Tissoires, linux-kernel, linux-input

On Tue, 16 Aug 2011, Jeremiah Matthey wrote:

> Patch to add SiGma Micro-based keyboards (1c4f:0002) to hid-quirks.
> 
> These keyboards dont seem to allow the records to be initialized, and hence a
> timeout occurs when the usbhid driver attempts to initialize them. The patch
> just adds the signature for these keyboards to the hid-quirks list with the
> setting HID_QUIRK_NO_INIT_REPORTS. This removes the 5-10 second wait for the
> timeout to occur.
> 
> This is technically a resubmit, first time i posted i didnt include the 
> right people in the destination, was just aimed at the list, so might 
> have gone unnoticed. (First (submitted) kernel patch, I'll learn with 
> time :))

Thanks for the patch.

Please also send me your Signed-off-by line (as described in 
Documentation/SubmittingPatches) so that I could apply it to my tree.

> diff -ru linux-3.0.1-clean/drivers/hid/hid-ids.h linux-3.0.1-patched/drivers/hid/hid-ids.h
> --- linux-3.0.1-clean/drivers/hid/hid-ids.h	2011-08-05 05:59:21.000000000 +0100
> +++ linux-3.0.1-patched/drivers/hid/hid-ids.h	2011-08-14 15:25:58.129865922 +0100
> @@ -568,6 +568,9 @@
>  #define USB_DEVICE_ID_SAMSUNG_IR_REMOTE	0x0001
>  #define USB_DEVICE_ID_SAMSUNG_WIRELESS_KBD_MOUSE	0x0600
>  
> +#define USB_VENDOR_ID_SIGMA_MICRO	0x1c4f
> +#define USB_DEVICE_ID_SIGMA_MICRO_KEYBOARD	0x0002
> +
>  #define USB_VENDOR_ID_SKYCABLE			0x1223
>  #define	USB_DEVICE_ID_SKYCABLE_WIRELESS_PRESENTER	0x3F07
>  
> diff -ru linux-3.0.1-clean/drivers/hid/usbhid/hid-quirks.c linux-3.0.1-patched/drivers/hid/usbhid/hid-quirks.c
> --- linux-3.0.1-clean/drivers/hid/usbhid/hid-quirks.c	2011-08-05 05:59:21.000000000 +0100
> +++ linux-3.0.1-patched/drivers/hid/usbhid/hid-quirks.c	2011-08-14 15:26:06.289865845 +0100
> @@ -89,6 +89,7 @@
>  
>  	{ USB_VENDOR_ID_CHICONY, USB_DEVICE_ID_CHICONY_MULTI_TOUCH, HID_QUIRK_MULTI_INPUT },
>  	{ USB_VENDOR_ID_CHICONY, USB_DEVICE_ID_CHICONY_WIRELESS, HID_QUIRK_MULTI_INPUT },
> +	{ USB_VENDOR_ID_SIGMA_MICRO, USB_DEVICE_ID_SIGMA_MICRO_KEYBOARD, HID_QUIRK_NO_INIT_REPORTS },
>  	{ 0, 0 }
>  };
>  
> 

-- 
Jiri Kosina
SUSE Labs

^ permalink raw reply	[flat|nested] 5+ messages in thread

* Re: [PATCH] usbhid: Add the malfunctioning SiGma Micro chip to the list of hid-quirks
  2011-08-18 12:31   ` Jiri Kosina
@ 2011-08-19 21:59     ` Jeremiah Matthey
  2011-08-23  8:02       ` Jiri Kosina
  0 siblings, 1 reply; 5+ messages in thread
From: Jeremiah Matthey @ 2011-08-19 21:59 UTC (permalink / raw)
  To: Jiri Kosina; +Cc: Benjamin_Tissoires, linux-kernel, linux-input

Sorry about that, here it is.

Signed-off-by: Jeremiah Matthey <sprg86@gmail.com>

On Thu, Aug 18, 2011 at 02:31:57PM +0200, Jiri Kosina wrote:
> On Tue, 16 Aug 2011, Jeremiah Matthey wrote:
> 
> > Patch to add SiGma Micro-based keyboards (1c4f:0002) to hid-quirks.
> > 
> > These keyboards dont seem to allow the records to be initialized, and hence a
> > timeout occurs when the usbhid driver attempts to initialize them. The patch
> > just adds the signature for these keyboards to the hid-quirks list with the
> > setting HID_QUIRK_NO_INIT_REPORTS. This removes the 5-10 second wait for the
> > timeout to occur.
> > 
> > This is technically a resubmit, first time i posted i didnt include the 
> > right people in the destination, was just aimed at the list, so might 
> > have gone unnoticed. (First (submitted) kernel patch, I'll learn with 
> > time :))
> 
> Thanks for the patch.
> 
> Please also send me your Signed-off-by line (as described in 
> Documentation/SubmittingPatches) so that I could apply it to my tree.
> 
> > diff -ru linux-3.0.1-clean/drivers/hid/hid-ids.h linux-3.0.1-patched/drivers/hid/hid-ids.h
> > --- linux-3.0.1-clean/drivers/hid/hid-ids.h	2011-08-05 05:59:21.000000000 +0100
> > +++ linux-3.0.1-patched/drivers/hid/hid-ids.h	2011-08-14 15:25:58.129865922 +0100
> > @@ -568,6 +568,9 @@
> >  #define USB_DEVICE_ID_SAMSUNG_IR_REMOTE	0x0001
> >  #define USB_DEVICE_ID_SAMSUNG_WIRELESS_KBD_MOUSE	0x0600
> >  
> > +#define USB_VENDOR_ID_SIGMA_MICRO	0x1c4f
> > +#define USB_DEVICE_ID_SIGMA_MICRO_KEYBOARD	0x0002
> > +
> >  #define USB_VENDOR_ID_SKYCABLE			0x1223
> >  #define	USB_DEVICE_ID_SKYCABLE_WIRELESS_PRESENTER	0x3F07
> >  
> > diff -ru linux-3.0.1-clean/drivers/hid/usbhid/hid-quirks.c linux-3.0.1-patched/drivers/hid/usbhid/hid-quirks.c
> > --- linux-3.0.1-clean/drivers/hid/usbhid/hid-quirks.c	2011-08-05 05:59:21.000000000 +0100
> > +++ linux-3.0.1-patched/drivers/hid/usbhid/hid-quirks.c	2011-08-14 15:26:06.289865845 +0100
> > @@ -89,6 +89,7 @@
> >  
> >  	{ USB_VENDOR_ID_CHICONY, USB_DEVICE_ID_CHICONY_MULTI_TOUCH, HID_QUIRK_MULTI_INPUT },
> >  	{ USB_VENDOR_ID_CHICONY, USB_DEVICE_ID_CHICONY_WIRELESS, HID_QUIRK_MULTI_INPUT },
> > +	{ USB_VENDOR_ID_SIGMA_MICRO, USB_DEVICE_ID_SIGMA_MICRO_KEYBOARD, HID_QUIRK_NO_INIT_REPORTS },
> >  	{ 0, 0 }
> >  };
> >  
> > 
> 
> -- 
> Jiri Kosina
> SUSE Labs

^ permalink raw reply	[flat|nested] 5+ messages in thread

* Re: [PATCH] usbhid: Add the malfunctioning SiGma Micro chip to the list of hid-quirks
  2011-08-19 21:59     ` Jeremiah Matthey
@ 2011-08-23  8:02       ` Jiri Kosina
  0 siblings, 0 replies; 5+ messages in thread
From: Jiri Kosina @ 2011-08-23  8:02 UTC (permalink / raw)
  To: Jeremiah Matthey; +Cc: Benjamin_Tissoires, linux-kernel, linux-input

On Fri, 19 Aug 2011, Jeremiah Matthey wrote:

> Sorry about that, here it is.
> 
> Signed-off-by: Jeremiah Matthey <sprg86@gmail.com>

Applied, thanks for the patch.

-- 
Jiri Kosina
SUSE Labs

^ permalink raw reply	[flat|nested] 5+ messages in thread

end of thread, other threads:[~2011-08-23  8:02 UTC | newest]

Thread overview: 5+ messages (download: mbox.gz / follow: Atom feed)
-- links below jump to the message on this page --
2011-08-14 14:43 [PATCH] usbhid: Add the malfunctioning SiGma Micro chip to the list of hid-quirks Jeremiah Matthey
2011-08-16 10:05 ` Jeremiah Matthey
2011-08-18 12:31   ` Jiri Kosina
2011-08-19 21:59     ` Jeremiah Matthey
2011-08-23  8:02       ` Jiri Kosina

This is an external index of several public inboxes,
see mirroring instructions on how to clone and mirror
all data and code used by this external index.