RE: [PATCH] USB: misc: Add N-trig firmware driver

From: Micki Balanga
Date: Tue Oct 05 2010 - 03:00:58 EST




> -----Original Message-----
> From: Jiri Kosina [mailto:jkosina@xxxxxxx]
> Sent: Monday, October 04, 2010 2:58 PM
> To: Greg KH
> Cc: Micki Balanga; gregkh@xxxxxxx; jim.collar@xxxxxxxxxx; linux-
> usb@xxxxxxxxxxxxxxx; linux-kernel@xxxxxxxxxxxxxxx
> Subject: Re: [PATCH] USB: misc: Add N-trig firmware driver
>
> On Sun, 3 Oct 2010, Greg KH wrote:
>
> > >
> >
> > Can you please attach this in a non-base64 format so it can be
> properly
> > reviewed, and then, applied if applicable?
> >
> > Note, please run the patch through the scripts/checkpatch.pl tool
> first
> > before sending it out, so we don't just say the same things it finds
> :)
>
> As we already have HID driver for n-trig, could this whole thing
> perhaps
> be made part of it?
>
> --
> Jiri Kosina
> SUSE Labs, Novell Inc.

The driver communicates with the bulk in/out endpoint.

It was written as a standalone module, because it does not use any HID subsystem services,
and to ensure that the HID driver would not be too complex.

The driver has been tested over several weeks at our company.
The patch was prepared and checked using the Checkpatch utility.

I await your input as to how you wish to proceed.
Micki

--
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/