Return-Path: Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S1756901Ab0FUICy (ORCPT ); Mon, 21 Jun 2010 04:02:54 -0400 Received: from stag.seas.upenn.edu ([158.130.70.79]:48457 "EHLO stag.seas.upenn.edu" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S1754202Ab0FUICw (ORCPT ); Mon, 21 Jun 2010 04:02:52 -0400 Message-ID: <4C1F1AFB.1010301@seas.upenn.edu> Date: Mon, 21 Jun 2010 03:55:39 -0400 From: Rafi Rubin User-Agent: Mozilla/5.0 (X11; U; Linux i686 (x86_64); en-US; rv:1.9.1.10) Gecko/20100512 Thunderbird/3.0.5 MIME-Version: 1.0 To: Micki Balanga CC: Jiri Kosina , chatty@enac.fr, dhprince-devel@yahoo.co.uk, linux-input@vger.kernel.org, linux-kernel@vger.kernel.org Subject: Re: [PATCH] HID: Add support to new N-trig firwmare References: <48A28051AC6D7A48B64F28272458190333674B@Exchange-IL.n-trig.com> <48A28051AC6D7A48B64F28272458190326E03E@Exchange-IL.n-trig.com> <48A28051AC6D7A48B64F28272458190338280A@Exchange-IL.n-trig.com> <4C1F0348.8000804@seas.upenn.edu> <48A28051AC6D7A48B64F28272458190338281E@Exchange-IL.n-trig.com> In-Reply-To: <48A28051AC6D7A48B64F28272458190338281E@Exchange-IL.n-trig.com> X-Enigmail-Version: 1.0.1 Content-Type: text/plain; charset=ISO-8859-1 Content-Transfer-Encoding: 7bit Sender: linux-kernel-owner@vger.kernel.org List-ID: X-Mailing-List: linux-kernel@vger.kernel.org Content-Length: 2681 Lines: 86 -----BEGIN PGP SIGNED MESSAGE----- Hash: SHA1 On 06/21/10 02:39, Micki Balanga wrote: > Hi rafi > Thank for the remarks > Before submitting the patch I used checkpatch.pl script > And checked whitespace in the code I inserted. I'm not saying anything in the patch is a style violation, just looks like you changed two lines unintentionally. Take a look at the patch itself, or try out "git difftool drivers/hid/hid-ids.h" and look at the ADS_TECH and EZKEY lines. > Yes I aware there is no product id 0x002. (it's ok) Ok, just sanity checking. > -----Original Message----- > From: Rafi Rubin [mailto:rafi@seas.upenn.edu] > Sent: Monday, June 21, 2010 9:15 AM > To: Micki Balanga > Cc: Jiri Kosina; chatty@enac.fr; dhprince-devel@yahoo.co.uk; > linux-input@vger.kernel.org; linux-kernel@vger.kernel.org > Subject: Re: [PATCH] HID: Add support to new N-trig firwmare > > On 06/21/10 01:06, Micki Balanga wrote: >> Hi Jiri >> We have some new product so I have to update the patch, >> When I update my git branch I didn't see the changes regarding N-trig >> Product id. If you can insert this patch, if there a problem with the >> patch >> Let me know and hi will update it >> Thank you >> Micki > > >> -----Original Message----- >> From: Jiri Kosina [mailto:jkosina@suse.cz] >> Sent: Saturday, June 05, 2010 6:42 PM >> To: Micki Balanga >> Cc: chatty@enac.fr; dhprince-devel@yahoo.co.uk; >> linux-input@vger.kernel.org; linux-kernel@vger.kernel.org; Rafi Rubin >> Subject: RE: [PATCH] HID: Add support to new N-trig firwmare > >> On Fri, 4 Jun 2010, Micki Balanga wrote: > >>> Hi Jiri, >>> Attach the patch via file. >>> (Also fix 3 warning: please , no space before tabs) > >> Thanks Micki, I have applied the patch. > > > Micki, looks like there are a couple of accidental whitespace > modifications: > > -#define USB_VENDOR_ID_ADS_TECH 0x06e1 > +#define USB_VENDOR_ID_ADS_TECH 0x06e1 > > -#define USB_VENDOR_ID_EZKEY 0x0518 > +#define USB_VENDOR_ID_EZKEY 0x0518 > > > Also, in the list of product ids, did you mean to skip 2? > > > Jiri, is there a better way than listing each device individually? > > > Rafi -----BEGIN PGP SIGNATURE----- Version: GnuPG v1.4.10 (GNU/Linux) Comment: Using GnuPG with Mozilla - http://enigmail.mozdev.org/ iEYEARECAAYFAkwfGvkACgkQwuRiAT9o608vqACfZgAPtUYpEJHZMPwjGR/0u2l/ I/UAoNUrl8V75kFOBVmMCNRKIfnYB7Pk =55Y2 -----END PGP SIGNATURE----- -- To unsubscribe from this list: send the line "unsubscribe linux-kernel" in the body of a message to majordomo@vger.kernel.org More majordomo info at http://vger.kernel.org/majordomo-info.html Please read the FAQ at http://www.tux.org/lkml/