All of lore.kernel.org
 help / color / mirror / Atom feed
From: Teemu Likonen <tlikonen@iki.fi>
To: Johan Hovold <johan@kernel.org>
Cc: linux-usb@vger.kernel.org, "Dan Williams" <dcbw@redhat.com>,
	"Bjørn Mork" <bjorn@mork.no>,
	stable@vger.kernel.org
Subject: Re: [PATCH] USB: serial: option: add Telewell TW-3G HSPA+ device id
Date: Mon, 12 Jun 2017 13:11:18 +0300	[thread overview]
Message-ID: <87lgoxecs9.fsf@mithlond.arda> (raw)
In-Reply-To: <20170612091612.30919-1-johan@kernel.org> (Johan Hovold's message of "Mon, 12 Jun 2017 11:16:12 +0200")

[-- Attachment #1: Type: text/plain, Size: 1606 bytes --]

Johan Hovold [2017-06-12 11:16:12+02] wrote:

> Add device-id entry for the Telewell TW-3G HSPA+ modem, which uses the
> Longcheer vendor id.
>
> Reported-by: Teemu Likonen <tlikonen@iki.fi>
> Cc: stable <stable@vger.kernel.org>
> Signed-off-by: Johan Hovold <johan@kernel.org>
> ---
>  drivers/usb/serial/option.c | 4 ++++
>  1 file changed, 4 insertions(+)
>
> Teemu, would you be able to test this patch?

It seems that you will do another patch but I'll confirm anyway that
this works. I compiled the mainline kernel v4.12-rc5 with this patch.

> diff --git a/drivers/usb/serial/option.c b/drivers/usb/serial/option.c
> index 3bf61acfc26b..b48f3557b2e4 100644
> --- a/drivers/usb/serial/option.c
> +++ b/drivers/usb/serial/option.c
> @@ -388,6 +388,9 @@ static void option_instat_callback(struct urb *urb);
>  /* SpeedUp SU9800 usb 3g modem */
>  #define SPEEDUP_PRODUCT_SU9800			0x9800
>  
> +/* Telewell */
> +#define TELEWELL_PRODUCT_TW_3G_HSPAP		0x9801
> +
>  /* Haier products */
>  #define HAIER_VENDOR_ID				0x201e
>  #define HAIER_PRODUCT_CE81B			0x10f8
> @@ -1877,6 +1880,7 @@ static const struct usb_device_id option_ids[] = {
>  	  .driver_info = (kernel_ulong_t)&four_g_w100_blacklist
>  	},
>  	{ USB_DEVICE_INTERFACE_CLASS(LONGCHEER_VENDOR_ID, SPEEDUP_PRODUCT_SU9800, 0xff) },
> +	{ USB_DEVICE_INTERFACE_CLASS(LONGCHEER_VENDOR_ID, TELEWELL_PRODUCT_TW_3G_HSPAP, 0xff) },
>  	{ USB_DEVICE(LONGCHEER_VENDOR_ID, ZOOM_PRODUCT_4597) },
>  	{ USB_DEVICE(LONGCHEER_VENDOR_ID, IBALL_3_5G_CONNECT) },
>  	{ USB_DEVICE(HAIER_VENDOR_ID, HAIER_PRODUCT_CE100) },

[-- Attachment #2: signature.asc --]
[-- Type: application/pgp-signature, Size: 487 bytes --]

  parent reply	other threads:[~2017-06-12 10:11 UTC|newest]

Thread overview: 11+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2017-06-12  9:16 [PATCH] USB: serial: option: add Telewell TW-3G HSPA+ device id Johan Hovold
2017-06-12  9:50 ` Bjørn Mork
2017-06-12 10:09   ` Oliver Neukum
2017-06-12 10:12   ` Johan Hovold
2017-06-12 10:14   ` Johan Hovold
2017-06-12 10:31   ` Lars Melin
2017-06-12 10:37     ` Bjørn Mork
2017-06-12 11:02       ` Lars Melin
2017-06-12 14:29         ` Johan Hovold
2017-06-12 10:11 ` Teemu Likonen [this message]
2017-06-12 10:16   ` Johan Hovold

Reply instructions:

You may reply publicly to this message via plain-text email
using any one of the following methods:

* Save the following mbox file, import it into your mail client,
  and reply-to-all from there: mbox

  Avoid top-posting and favor interleaved quoting:
  https://en.wikipedia.org/wiki/Posting_style#Interleaved_style

* Reply using the --to, --cc, and --in-reply-to
  switches of git-send-email(1):

  git send-email \
    --in-reply-to=87lgoxecs9.fsf@mithlond.arda \
    --to=tlikonen@iki.fi \
    --cc=bjorn@mork.no \
    --cc=dcbw@redhat.com \
    --cc=johan@kernel.org \
    --cc=linux-usb@vger.kernel.org \
    --cc=stable@vger.kernel.org \
    /path/to/YOUR_REPLY

  https://kernel.org/pub/software/scm/git/docs/git-send-email.html

* If your mail client supports setting the In-Reply-To header
  via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line before the message body.
This is an external index of several public inboxes,
see mirroring instructions on how to clone and mirror
all data and code used by this external index.