From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: Received: from mail-lf0-f66.google.com ([209.85.215.66]:51322 "EHLO mail-lf0-f66.google.com" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S1751802AbdJaSye (ORCPT ); Tue, 31 Oct 2017 14:54:34 -0400 Date: Tue, 31 Oct 2017 19:54:32 +0100 From: Johan Hovold To: Gal Shalif Cc: Johan Hovold , Gal Shalif , =?iso-8859-1?Q?Bj=F8rn?= Mork , Oliver Neukum , "linux-usb@vger.kernel.org" , "stable@vger.kernel.org" Subject: Re: [PATCH] USB: serial: option: add Novatel USB730L enterprise mode Message-ID: <20171031185432.GQ7223@localhost> References: <20171030142309.GI7223@localhost> MIME-Version: 1.0 Content-Type: text/plain; charset=us-ascii Content-Disposition: inline In-Reply-To: Sender: stable-owner@vger.kernel.org List-ID: On Tue, Oct 31, 2017 at 04:54:56PM +0000, Gal Shalif wrote: > From 436a7eb28215081d34c386a92213bd6932d0779a Mon Sep 17 00:00:00 2001 > From: Gal Shalif > Date: Tue, 31 Oct 2017 18:00:36 +0200 > Subject: [PATCH] USB: serial: option: add Novatel USB730L enterprise mode > > Add the serial interface of the Novatel USB730L > enterprise mode. > Change is done according to: > Novatel Linux Integration Guide for USB730L at: > https://www.verizonwireless.com/dam/support/pdf/verizon-usb730l-integration-guide.pdf > > Signed-off-by: Gal Shalif Your SoB should match the From: field. > Cc: Gal Shalif Work > Cc: Bjorn Mork > Cc: Oliver Neukum > Cc: stable@vger.kernel.org > Signed-off-by: Johan Hovold And you must never add someone else's SoB (I'll add that myself when I apply the patch). > --- > drivers/usb/serial/option.c | 2 ++ Also this is a p0 patch so fails to apply without some manual intervention. You could try using git-format-patch and git-send-email and send the patch to yourself first to make sure it applies using git-am. > 1 file changed, 2 insertions(+) > > diff --git drivers/usb/serial/option.c drivers/usb/serial/option.c > index ba672cf..5ea9fa5 100644 > --- drivers/usb/serial/option.c > +++ drivers/usb/serial/option.c > @@ -162,6 +162,7 @@ static void option_instat_callback(struct urb *urb); > #define NOVATELWIRELESS_PRODUCT_E362 0x9010 > #define NOVATELWIRELESS_PRODUCT_E371 0x9011 > #define NOVATELWIRELESS_PRODUCT_U620L 0x9022 > +#define NOVATELWIRELESS_PRODUCT_ENTERPRISE_U730L 0x9032 > #define NOVATELWIRELESS_PRODUCT_G2 0xA010 > #define NOVATELWIRELESS_PRODUCT_MC551 0xB001 > > @@ -1121,6 +1122,7 @@ static const struct usb_device_id option_ids[] = { > { USB_DEVICE_AND_INTERFACE_INFO(NOVATELWIRELESS_VENDOR_ID, NOVATELWIRELESS_PRODUCT_E362, 0xff, 0xff, 0xff) }, > { USB_DEVICE_AND_INTERFACE_INFO(NOVATELWIRELESS_VENDOR_ID, NOVATELWIRELESS_PRODUCT_E371, 0xff, 0xff, 0xff) }, > { USB_DEVICE_AND_INTERFACE_INFO(NOVATELWIRELESS_VENDOR_ID, NOVATELWIRELESS_PRODUCT_U620L, 0xff, 0x00, 0x00) }, > + { USB_DEVICE_AND_INTERFACE_INFO(NOVATELWIRELESS_VENDOR_ID, NOVATELWIRELESS_PRODUCT_ENTERPRISE_U730L, 0xff, 0x00, 0x00) }, > > { USB_DEVICE(AMOI_VENDOR_ID, AMOI_PRODUCT_H01) }, > { USB_DEVICE(AMOI_VENDOR_ID, AMOI_PRODUCT_H01A) }, Other than that, this looks to good to me. Perhaps you could include the output of usb-devices for reference as well. Care to send a v2 with the right Signed-off-by? Thanks, Johan