Re: [PATCH] USB: cp210x: Add ID for Parrot NMEA GPS Flight Recorder

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

 



On Sun, Feb 28, 2016 at 06:18:00PM +0100, Vittorio Alfieri wrote:
> On Sun, Feb 28, 2016 at 02:59:43AM +0000, Greg KH wrote:
> > On Sun, Feb 28, 2016 at 02:17:39AM +0100, Vittorio Alfieri wrote:
> > > The Parrot NMEA GPS Flight Recorder is a USB composite device consisting of hub, flash storage, and cp210x usb to serial chip.
> > > It is an accessory to the mass-produced Parrot AR Drone 2.
> > > The device emits standard NMEA messages which make the it compatible with NMEA compatible software.
> > > I tested using gpsd as an NMEA interpreter and the official Parrot Flight Recorder.
> > > It works with gpsd version 3.11-3.
> > > ---
> > >  drivers/usb/serial/cp210x.c | 1 +
> > >  1 file changed, 1 insertion(+)
> > > 
> > > diff --git a/drivers/usb/serial/cp210x.c b/drivers/usb/serial/cp210x.c
> > > index 987813b..ef422db 100644
> > > --- a/drivers/usb/serial/cp210x.c
> > > +++ b/drivers/usb/serial/cp210x.c
> > > @@ -163,6 +163,7 @@ static const struct usb_device_id id_table[] = {
> > >  	{ USB_DEVICE(0x1843, 0x0200) }, /* Vaisala USB Instrument Cable */
> > >  	{ USB_DEVICE(0x18EF, 0xE00F) }, /* ELV USB-I2C-Interface */
> > >  	{ USB_DEVICE(0x18EF, 0xE025) }, /* ELV Marble Sound Board 1 */
> > > +	{ USB_DEVICE(0x19CF, 0x3000) }, /* Parrot NMEA GPS Flight Recorder */
> > >  	{ USB_DEVICE(0x1ADB, 0x0001) }, /* Schweitzer Engineering C662 Cable */
> > >  	{ USB_DEVICE(0x1B1C, 0x1C00) }, /* Corsair USB Dongle */
> > >  	{ USB_DEVICE(0x1BA4, 0x0002) },	/* Silicon Labs 358x factory default */
> > > -- 
> > > 2.1.4
> > 
> > You forgot a Signed-off-by: line, which is required for all changes,
> > please see Documenation/SubmittingPatches for what that is.
> > 
> > Also please line-wrap your changelog comments at a sane column (i.e. 72,
> > you didn't pick anything...)
> > 
> > thanks,
> > 
> > greg k-h
> 
> Hello,
> 
> Thank you Greg for your fast answer and your suggestions.
> This is my first patch submitted for the linux kernel.
> 
> I will submit a corrected patch in a separate email.

Great.

> BTW, is it possible to have this associated with my Github account for
> record keeping?

I don't understand what you mean by this, please explain it more.

greg k-h
--
To unsubscribe from this list: send the line "unsubscribe linux-usb" in
the body of a message to majordomo@xxxxxxxxxxxxxxx
More majordomo info at  http://vger.kernel.org/majordomo-info.html



[Index of Archives]     [Linux Media]     [Linux Input]     [Linux Audio Users]     [Yosemite News]     [Linux Kernel]     [Linux SCSI]     [Old Linux USB Devel Archive]

  Powered by Linux