From mboxrd@z Thu Jan 1 00:00:00 1970 From: Peter Hutterer Subject: Re: [PATCH 1/2] hid: ntrig NO_INIT_REPORTS Date: Fri, 11 Mar 2011 11:42:20 +1000 Message-ID: <20110311014220.GC8970@barra.bne.redhat.com> References: <1299731632-9814-1-git-send-email-rafi@seas.upenn.edu> Mime-Version: 1.0 Content-Type: text/plain; charset=us-ascii Return-path: Received: from leo.clearchain.com ([199.73.29.74]:22271 "EHLO mail.clearchain.com" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S1754444Ab1CKBmp (ORCPT ); Thu, 10 Mar 2011 20:42:45 -0500 Content-Disposition: inline In-Reply-To: <1299731632-9814-1-git-send-email-rafi@seas.upenn.edu> Sender: linux-input-owner@vger.kernel.org List-Id: linux-input@vger.kernel.org To: Rafi Rubin Cc: jkosina@suse.cz, linux-input@vger.kernel.org, linux-kernel@vger.kernel.org, micki@n-trig.com, rydberg@euromail.se, chatty@enac.fr On Wed, Mar 09, 2011 at 11:33:51PM -0500, Rafi Rubin wrote: > Probing reports does bad things with some ntrig firmwares, better to > just leave them alone. > > Signed-off-by: Rafi Rubin > > --- > > This resolves an annoying delay on initialization and reduces the > chance of the device coming up in an odd state or crashing. never saw those delays but this patch doesn't make anything worse for me. Tested-by: Peter Hutterer Cheers, Peter > --- > drivers/hid/hid-ntrig.c | 3 ++- > 1 files changed, 2 insertions(+), 1 deletions(-) > > diff --git a/drivers/hid/hid-ntrig.c b/drivers/hid/hid-ntrig.c > index b1e9cca..850ba14 100644 > --- a/drivers/hid/hid-ntrig.c > +++ b/drivers/hid/hid-ntrig.c > @@ -837,7 +837,8 @@ static int ntrig_probe(struct hid_device *hdev, const struct hid_device_id *id) > struct hid_report *report; > > if (id->driver_data) > - hdev->quirks |= HID_QUIRK_MULTI_INPUT; > + hdev->quirks |= HID_QUIRK_MULTI_INPUT > + | HID_QUIRK_NO_INIT_REPORTS; > > nd = kmalloc(sizeof(struct ntrig_data), GFP_KERNEL); > if (!nd) { > -- > 1.7.4.1 > > -- > To unsubscribe from this list: send the line "unsubscribe linux-input" in > the body of a message to majordomo@vger.kernel.org > More majordomo info at http://vger.kernel.org/majordomo-info.html >