From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: Received: from mail-lf0-f54.google.com ([209.85.215.54]:35738 "EHLO mail-lf0-f54.google.com" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S1755464AbcDGJeN (ORCPT ); Thu, 7 Apr 2016 05:34:13 -0400 Date: Thu, 7 Apr 2016 11:34:14 +0200 From: Johan Hovold To: =?iso-8859-1?Q?Bj=F8rn?= Mork Cc: Johan Hovold , Thomas =?iso-8859-1?Q?Sch=E4fer?= , linux-usb@vger.kernel.org, stable@vger.kernel.org Subject: Re: [PATCH] USB: option: add "D-Link DWM-221 B1" device id Message-ID: <20160407093414.GE25008@localhost> References: <1459197455-11180-1-git-send-email-bjorn@mork.no> MIME-Version: 1.0 Content-Type: text/plain; charset=iso-8859-1 Content-Disposition: inline Content-Transfer-Encoding: 8bit In-Reply-To: <1459197455-11180-1-git-send-email-bjorn@mork.no> Sender: stable-owner@vger.kernel.org List-ID: On Mon, Mar 28, 2016 at 10:37:35PM +0200, Bj�rn Mork wrote: > Thomas reports: > "Windows: > > 00 diagnostics > 01 modem > 02 at-port > 03 nmea > 04 nic > > Linux: > > T: Bus=02 Lev=01 Prnt=01 Port=03 Cnt=01 Dev#= 4 Spd=480 MxCh= 0 > D: Ver= 2.00 Cls=00(>ifc ) Sub=00 Prot=00 MxPS=64 #Cfgs= 1 > P: Vendor=2001 ProdID=7e19 Rev=02.32 > S: Manufacturer=Mobile Connect > S: Product=Mobile Connect > S: SerialNumber=0123456789ABCDEF > C: #Ifs= 6 Cfg#= 1 Atr=a0 MxPwr=500mA > I: If#= 0 Alt= 0 #EPs= 2 Cls=ff(vend.) Sub=ff Prot=ff Driver=option > I: If#= 1 Alt= 0 #EPs= 3 Cls=ff(vend.) Sub=00 Prot=00 Driver=option > I: If#= 2 Alt= 0 #EPs= 3 Cls=ff(vend.) Sub=00 Prot=00 Driver=option > I: If#= 3 Alt= 0 #EPs= 3 Cls=ff(vend.) Sub=00 Prot=00 Driver=option > I: If#= 4 Alt= 0 #EPs= 3 Cls=ff(vend.) Sub=ff Prot=ff Driver=qmi_wwan > I: If#= 5 Alt= 0 #EPs= 2 Cls=08(stor.) Sub=06 Prot=50 Driver=usb-storage" > > Reported-by: Thomas Sch�fer > Cc: > Signed-off-by: Bj�rn Mork > --- > drivers/usb/serial/option.c | 2 ++ > 1 file changed, 2 insertions(+) > > diff --git a/drivers/usb/serial/option.c b/drivers/usb/serial/option.c > index 348e19834b83..6ca2cc5790c0 100644 > --- a/drivers/usb/serial/option.c > +++ b/drivers/usb/serial/option.c > @@ -1818,6 +1818,8 @@ static const struct usb_device_id option_ids[] = { > { USB_DEVICE_AND_INTERFACE_INFO(0x2001, 0x7d02, 0xff, 0x00, 0x00) }, > { USB_DEVICE_AND_INTERFACE_INFO(0x2001, 0x7d03, 0xff, 0x02, 0x01) }, > { USB_DEVICE_AND_INTERFACE_INFO(0x2001, 0x7d03, 0xff, 0x00, 0x00) }, > + { USB_DEVICE(0x2001, 0x7e19), /* D-Link DWM-221 B1 */ > + .driver_info = (kernel_ulong_t)&net_intf4_blacklist }, Why not use USB_DEVICE_INTERFACE_CLASS here to avoid even trying to probe the mass-storage interface? Thanks, Johan