2014-10-29 21:22:25

by Anton Staaf

[permalink] [raw]
Subject: [PATCH] USB: simple: add Google simple serial SubClass support

Add support for Google devices that export simple serial
interfaces using the vendor specific SubClass/Protocol pair
0x50/0x01.

Signed-off-by: Anton Staaf <[email protected]>

---
drivers/usb/serial/usb-serial-simple.c | 11 +++++++++++
1 file changed, 11 insertions(+)

diff --git a/drivers/usb/serial/usb-serial-simple.c b/drivers/usb/serial/usb-serial-simple.c
index fb79775..85bb973 100644
--- a/drivers/usb/serial/usb-serial-simple.c
+++ b/drivers/usb/serial/usb-serial-simple.c
@@ -8,6 +8,7 @@
* Copyright (C) 2010 Zilogic Systems <[email protected]>
* Copyright (C) 2013 Wei Shuai <[email protected]>
* Copyright (C) 2013 Linux Foundation
+ * Copyright (C) 2014 Google, Inc.
*
* This program is free software; you can redistribute it and/or
* modify it under the terms of the GNU General Public License version
@@ -80,6 +81,14 @@ DEVICE(suunto, SUUNTO_IDS);
{ USB_DEVICE(0x908, 0x0004) }
DEVICE(siemens_mpi, SIEMENS_IDS);

+/* Google Serial USB SubClass */
+#define GOOGLE_IDS() \
+ { USB_VENDOR_AND_INTERFACE_INFO(0x18d1, \
+ USB_CLASS_VENDOR_SPEC, \
+ 0x50, \
+ 0x01) }
+DEVICE(google, GOOGLE_IDS);
+
/* All of the above structures mushed into two lists */
static struct usb_serial_driver * const serial_drivers[] = {
&zio_device,
@@ -90,6 +99,7 @@ static struct usb_serial_driver * const serial_drivers[] = {
&hp4x_device,
&suunto_device,
&siemens_mpi_device,
+ &google_device,
NULL
};

@@ -102,6 +112,7 @@ static const struct usb_device_id id_table[] = {
HP4X_IDS(),
SUUNTO_IDS(),
SIEMENS_IDS(),
+ GOOGLE_IDS(),
{ },
};
MODULE_DEVICE_TABLE(usb, id_table);
--
2.1.0.rc2.206.gedb03e5


2014-10-30 14:34:37

by Greg Kroah-Hartman

[permalink] [raw]
Subject: Re: [PATCH] USB: simple: add Google simple serial SubClass support

On Wed, Oct 29, 2014 at 02:21:56PM -0700, Anton Staaf wrote:
> Add support for Google devices that export simple serial
> interfaces using the vendor specific SubClass/Protocol pair
> 0x50/0x01.
>
> Signed-off-by: Anton Staaf <[email protected]>
>
> ---
> drivers/usb/serial/usb-serial-simple.c | 11 +++++++++++
> 1 file changed, 11 insertions(+)
>
> diff --git a/drivers/usb/serial/usb-serial-simple.c b/drivers/usb/serial/usb-serial-simple.c
> index fb79775..85bb973 100644
> --- a/drivers/usb/serial/usb-serial-simple.c
> +++ b/drivers/usb/serial/usb-serial-simple.c
> @@ -8,6 +8,7 @@
> * Copyright (C) 2010 Zilogic Systems <[email protected]>
> * Copyright (C) 2013 Wei Shuai <[email protected]>
> * Copyright (C) 2013 Linux Foundation
> + * Copyright (C) 2014 Google, Inc.
> *
> * This program is free software; you can redistribute it and/or
> * modify it under the terms of the GNU General Public License version

Based on advice from my lawyer, a change this size does not warrent a
copyright notice at the top of a file. If your lawyers feel otherwise,
please let me know and I will be glad to discuss it with them.

So, care to resend this without this change to the file?

thanks,

greg k-h

2014-10-31 04:54:57

by Anton Staaf

[permalink] [raw]
Subject: Re: [PATCH] USB: simple: add Google simple serial SubClass support

On Thu, Oct 30, 2014 at 7:33 AM, Greg Kroah-Hartman
<[email protected]> wrote:
>
> On Wed, Oct 29, 2014 at 02:21:56PM -0700, Anton Staaf wrote:
> > Add support for Google devices that export simple serial
> > interfaces using the vendor specific SubClass/Protocol pair
> > 0x50/0x01.
> >
> > Signed-off-by: Anton Staaf <[email protected]>
> >
> > ---
> > drivers/usb/serial/usb-serial-simple.c | 11 +++++++++++
> > 1 file changed, 11 insertions(+)
> >
> > diff --git a/drivers/usb/serial/usb-serial-simple.c b/drivers/usb/serial/usb-serial-simple.c
> > index fb79775..85bb973 100644
> > --- a/drivers/usb/serial/usb-serial-simple.c
> > +++ b/drivers/usb/serial/usb-serial-simple.c
> > @@ -8,6 +8,7 @@
> > * Copyright (C) 2010 Zilogic Systems <[email protected]>
> > * Copyright (C) 2013 Wei Shuai <[email protected]>
> > * Copyright (C) 2013 Linux Foundation
> > + * Copyright (C) 2014 Google, Inc.
> > *
> > * This program is free software; you can redistribute it and/or
> > * modify it under the terms of the GNU General Public License version
>
> Based on advice from my lawyer, a change this size does not warrent a
> copyright notice at the top of a file. If your lawyers feel otherwise,
> please let me know and I will be glad to discuss it with them.
>
> So, care to resend this without this change to the file?
>
> thanks,
>
> greg k-h

Sigh, sorry for the top posted HTML junk previously, turns out the
gmail app for android doesn't "do the right thing"...

But yes, I'm happy to send along a version without copyright, I don't
see any problem with that. Looks like it will
have to wait until Monday as I don't have things set up at home to
send the patch as [email protected]. :)

Thanks,
Anton

2014-11-03 08:54:50

by Johan Hovold

[permalink] [raw]
Subject: Re: [PATCH] USB: simple: add Google simple serial SubClass support

On Thu, Oct 30, 2014 at 09:54:54PM -0700, Anton Staaf wrote:
> On Thu, Oct 30, 2014 at 7:33 AM, Greg Kroah-Hartman
> <[email protected]> wrote:
> >
> > On Wed, Oct 29, 2014 at 02:21:56PM -0700, Anton Staaf wrote:
> > > Add support for Google devices that export simple serial
> > > interfaces using the vendor specific SubClass/Protocol pair
> > > 0x50/0x01.
> > >
> > > Signed-off-by: Anton Staaf <[email protected]>
> > >
> > > ---
> > > drivers/usb/serial/usb-serial-simple.c | 11 +++++++++++
> > > 1 file changed, 11 insertions(+)
> > >
> > > diff --git a/drivers/usb/serial/usb-serial-simple.c b/drivers/usb/serial/usb-serial-simple.c
> > > index fb79775..85bb973 100644
> > > --- a/drivers/usb/serial/usb-serial-simple.c
> > > +++ b/drivers/usb/serial/usb-serial-simple.c
> > > @@ -8,6 +8,7 @@
> > > * Copyright (C) 2010 Zilogic Systems <[email protected]>
> > > * Copyright (C) 2013 Wei Shuai <[email protected]>
> > > * Copyright (C) 2013 Linux Foundation
> > > + * Copyright (C) 2014 Google, Inc.
> > > *
> > > * This program is free software; you can redistribute it and/or
> > > * modify it under the terms of the GNU General Public License version
> >
> > Based on advice from my lawyer, a change this size does not warrent a
> > copyright notice at the top of a file. If your lawyers feel otherwise,
> > please let me know and I will be glad to discuss it with them.
> >
> > So, care to resend this without this change to the file?
> >
> > thanks,
> >
> > greg k-h
>
> Sigh, sorry for the top posted HTML junk previously, turns out the
> gmail app for android doesn't "do the right thing"...
>
> But yes, I'm happy to send along a version without copyright, I don't
> see any problem with that. Looks like it will
> have to wait until Monday as I don't have things set up at home to
> send the patch as [email protected]. :)

Please also make sure to generate the patch against v3.18-rc (the one
you sent would not apply).

Thanks,
Johan