All of lore.kernel.org
 help / color / mirror / Atom feed
From: Jeremiah Matthey <sprg86@gmail.com>
To: Jiri Kosina <jkosina@suse.cz>
Cc: Benjamin_Tissoires@logitech.com, linux-kernel@vger.kernel.org,
	linux-input@vger.kernel.org
Subject: Re: [PATCH] usbhid: Add the malfunctioning SiGma Micro chip to the list of hid-quirks
Date: Fri, 19 Aug 2011 22:59:11 +0100	[thread overview]
Message-ID: <20110819215909.GA1868@netstar> (raw)
In-Reply-To: <alpine.LNX.2.00.1108181431110.27118@pobox.suse.cz>

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

  reply	other threads:[~2011-08-19 21:58 UTC|newest]

Thread overview: 5+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
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 [this message]
2011-08-23  8:02       ` Jiri Kosina

Reply instructions:

You may reply publicly to this message via plain-text email
using any one of the following methods:

* Save the following mbox file, import it into your mail client,
  and reply-to-all from there: mbox

  Avoid top-posting and favor interleaved quoting:
  https://en.wikipedia.org/wiki/Posting_style#Interleaved_style

* Reply using the --to, --cc, and --in-reply-to
  switches of git-send-email(1):

  git send-email \
    --in-reply-to=20110819215909.GA1868@netstar \
    --to=sprg86@gmail.com \
    --cc=Benjamin_Tissoires@logitech.com \
    --cc=jkosina@suse.cz \
    --cc=linux-input@vger.kernel.org \
    --cc=linux-kernel@vger.kernel.org \
    /path/to/YOUR_REPLY

  https://kernel.org/pub/software/scm/git/docs/git-send-email.html

* If your mail client supports setting the In-Reply-To header
  via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line before the message body.
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.