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.
BTW, is it possible to have this associated with my Github account for record
keeping?
Thanks,
Vittorio
--
To unsubscribe from this list: send the line "unsubscribe linux-usb" in
the body of a message to majord...@vger.kernel.org
More majordomo info at http://vger.kernel.org/majordomo-info.html
