Re: [PATCHv2] USB: serial: option: blacklist intf1 for Huawei E173s-6

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

 



On Tue, Nov 05, 2013 at 07:20:41AM -0300, Gustavo Zacarias wrote:
> Interface 1 on this device isn't for option to bind to otherwise an oops
> on usb_wwan with log flooding will happen when accessing the port:
> 
> tty_release: ttyUSB1: read/write wait queue active!
> 
> It doesn't seem to respond to QMI if it's added to qmi_wwan so don't add
> it there.
> 
> Signed-off-by: Gustavo Zacarias <gustavo@xxxxxxxxxxxxxxx>

Looks good to me, but perhaps you can change the subject line to read

	"USB: option: add support for Huawei E173s-6"

or similar, as you're adding a new device id.

Bjørn, do you want any more info for the commit message (e.g. interface
layout)?

Thanks,
Johan

> ---
> 
> v2:
> - Sort define by id as pointed by Sergei Shtylyov
> 
>  drivers/usb/serial/option.c | 3 +++
>  1 file changed, 3 insertions(+)
> 
> diff --git a/drivers/usb/serial/option.c b/drivers/usb/serial/option.c
> index acaee06..44741b6 100644
> --- a/drivers/usb/serial/option.c
> +++ b/drivers/usb/serial/option.c
> @@ -85,6 +85,7 @@ static void option_instat_callback(struct urb *urb);
>  #define HUAWEI_PRODUCT_K4505			0x1464
>  #define HUAWEI_PRODUCT_K3765			0x1465
>  #define HUAWEI_PRODUCT_K4605			0x14C6
> +#define HUAWEI_PRODUCT_E173S			0x1C07
>  
>  #define QUANTA_VENDOR_ID			0x0408
>  #define QUANTA_PRODUCT_Q101			0xEA02
> @@ -572,6 +573,8 @@ static const struct usb_device_id option_ids[] = {
>  	{ USB_DEVICE_AND_INTERFACE_INFO(HUAWEI_VENDOR_ID, 0x1c23, USB_CLASS_COMM, 0x02, 0xff) },
>  	{ USB_DEVICE_AND_INTERFACE_INFO(HUAWEI_VENDOR_ID, HUAWEI_PRODUCT_E173, 0xff, 0xff, 0xff),
>  		.driver_info = (kernel_ulong_t) &net_intf1_blacklist },
> +	{ USB_DEVICE_AND_INTERFACE_INFO(HUAWEI_VENDOR_ID, HUAWEI_PRODUCT_E173S, 0xff, 0xff, 0xff),
> +		.driver_info = (kernel_ulong_t) &net_intf1_blacklist },
>  	{ USB_DEVICE_AND_INTERFACE_INFO(HUAWEI_VENDOR_ID, HUAWEI_PRODUCT_E1750, 0xff, 0xff, 0xff),
>  		.driver_info = (kernel_ulong_t) &net_intf2_blacklist },
>  	{ USB_DEVICE_AND_INTERFACE_INFO(HUAWEI_VENDOR_ID, 0x1441, USB_CLASS_COMM, 0x02, 0xff) },
> -- 
> 1.8.1.5
> 
> --
> 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
--
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