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

lists.openwall.net		lists / announce owl-users owl-dev john-users john-dev passwdqc-users yescrypt popa3d-users / oss-security kernel-hardening musl sabotage tlsify passwords / crypt-dev xvendor / Bugtraq Full-Disclosure linux-kernel linux-netdev linux-ext4 linux-hardening linux-cve-announce PHC
Open Source and information security mailing list archives

Hash Suite: Windows password security audit tool. GUI, reports in PDF.

[<prev] [next>] [<thread-prev] [thread-next>] [day] [month] [year] [list]

Message-id: <6EF693376E00E2458E26FEC229232B128AB242@Exchange2010-IL.n-trig.com>
Date:	Wed, 06 Oct 2010 04:47:24 +0000
From:	Micki Balanga <micki@...rig.com>
To:	Greg KH <greg@...ah.com>
Cc:	Jiri Kosina <jkosina@...e.cz>, "gregkh@...e.de" <gregkh@...e.de>,
	"jim.collar@...are.net" <jim.collar@...are.net>,
	"linux-usb@...r.kernel.org" <linux-usb@...r.kernel.org>,
	"linux-kernel@...r.kernel.org" <linux-kernel@...r.kernel.org>
Subject: RE: [PATCH] USB: misc: Add N-trig firmware driver



> -----Original Message-----
> From: Greg KH [mailto:greg@...ah.com]
> Sent: Tuesday, October 05, 2010 2:47 PM
> To: Micki Balanga
> Cc: Jiri Kosina; gregkh@...e.de; jim.collar@...are.net; linux-
> usb@...r.kernel.org; linux-kernel@...r.kernel.org
> Subject: Re: [PATCH] USB: misc: Add N-trig firmware driver
> 
> On Tue, Oct 05, 2010 at 06:59:43AM +0000, Micki Balanga wrote:
> >
> >
> > > -----Original Message-----
> > > From: Jiri Kosina [mailto:jkosina@...e.cz]
> > > Sent: Monday, October 04, 2010 2:58 PM
> > > To: Greg KH
> > > Cc: Micki Balanga; gregkh@...e.de; jim.collar@...are.net; linux-
> > > usb@...r.kernel.org; linux-kernel@...r.kernel.org
> > > 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.
> 
> Why does it have to be a driver at all?  Why can't you just use libusb
> or usbfs to implement this?

The driver is used for several debug applications we use and libusb or usbfs caused us several problem.
we also run on several platform. The user space application which communicate with the driver
is common for several OS. 
> 
> > 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.
> 
> You might want to check that last step, it didn't seem to work properly
> as the patch you sent could not be applied, due to it being base64
> encoded :)
> 
> thanks,
> 
> greg k-h
--
To unsubscribe from this list: send the line "unsubscribe linux-kernel" in
the body of a message to majordomo@...r.kernel.org
More majordomo info at  http://vger.kernel.org/majordomo-info.html
Please read the FAQ at  http://www.tux.org/lkml/