linux-kernel - Re: [PATCH] USB: serial: option: simplify 3 D-Link device entries

lists.openwall.net		lists / announce owl-users owl-dev john-users john-dev passwdqc-users yescrypt popa3d-users / oss-security kernel-hardening musl sabotage tlsify passwords / crypt-dev xvendor / Bugtraq Full-Disclosure linux-kernel linux-netdev linux-ext4 linux-hardening linux-cve-announce PHC
Open Source and information security mailing list archives

Hash Suite: Windows password security audit tool. GUI, reports in PDF.

[<prev] [next>] [<thread-prev] [thread-next>] [day] [month] [year] [list]

Date:   Tue, 29 Aug 2017 16:41:21 +0200
From:   Johan Hovold <johan@...nel.org>
To:     Bjørn Mork <bjorn@...k.no>
Cc:     Johan Hovold <johan@...nel.org>,
        "Maciej S . Szmigiero" <mail@...iej.szmigiero.name>,
        Greg Kroah-Hartman <gregkh@...uxfoundation.org>,
        linux-usb@...r.kernel.org, linux-kernel@...r.kernel.org
Subject: Re: [PATCH] USB: serial: option: simplify 3 D-Link device entries

On Tue, Aug 29, 2017 at 10:45:13AM +0200, Bjørn Mork wrote:
> All the vendor specific interfaces on these devices are serial
> functions handled by this driver, so we can use a single class
> match entry for each.
> 
>  P:  Vendor=2001 ProdID=7d01 Rev= 3.00
>  S:  Manufacturer=D-Link,Inc
>  S:  Product=D-Link DWM-156
>  C:* #Ifs= 7 Cfg#= 1 Atr=a0 MxPwr=500mA
>  A:  FirstIf#= 0 IfCount= 2 Cls=02(comm.) Sub=0e Prot=00
>  I:* If#= 0 Alt= 0 #EPs= 1 Cls=02(comm.) Sub=0e Prot=00 Driver=cdc_mbim
>  E:  Ad=88(I) Atr=03(Int.) MxPS=  64 Ivl=125us
>  I:  If#= 1 Alt= 0 #EPs= 0 Cls=0a(data ) Sub=00 Prot=02 Driver=cdc_mbim
>  I:* If#= 1 Alt= 1 #EPs= 2 Cls=0a(data ) Sub=00 Prot=02 Driver=cdc_mbim
>  E:  Ad=81(I) Atr=02(Bulk) MxPS= 512 Ivl=0ms
>  E:  Ad=01(O) Atr=02(Bulk) MxPS= 512 Ivl=0ms
>  I:* If#= 2 Alt= 0 #EPs= 3 Cls=ff(vend.) Sub=02 Prot=01 Driver=option
>  E:  Ad=87(I) Atr=03(Int.) MxPS=  64 Ivl=500us
>  E:  Ad=82(I) Atr=02(Bulk) MxPS= 512 Ivl=0ms
>  E:  Ad=02(O) Atr=02(Bulk) MxPS= 512 Ivl=0ms
>  I:* If#= 3 Alt= 0 #EPs= 2 Cls=ff(vend.) Sub=00 Prot=00 Driver=option
>  E:  Ad=83(I) Atr=02(Bulk) MxPS= 512 Ivl=0ms
>  E:  Ad=03(O) Atr=02(Bulk) MxPS= 512 Ivl=0ms
>  I:* If#= 4 Alt= 0 #EPs= 2 Cls=ff(vend.) Sub=00 Prot=00 Driver=option
>  E:  Ad=84(I) Atr=02(Bulk) MxPS= 512 Ivl=0ms
>  E:  Ad=04(O) Atr=02(Bulk) MxPS= 512 Ivl=0ms
>  I:* If#= 5 Alt= 0 #EPs= 2 Cls=ff(vend.) Sub=00 Prot=00 Driver=option
>  E:  Ad=85(I) Atr=02(Bulk) MxPS= 512 Ivl=0ms
>  E:  Ad=05(O) Atr=02(Bulk) MxPS= 512 Ivl=0ms
>  I:* If#= 6 Alt= 0 #EPs= 2 Cls=08(stor.) Sub=06 Prot=50 Driver=usb-storage
>  E:  Ad=86(I) Atr=02(Bulk) MxPS= 512 Ivl=0ms
>  E:  Ad=06(O) Atr=02(Bulk) MxPS= 512 Ivl=0ms
> 
> Signed-off-by: Bjørn Mork <bjorn@...k.no>
> ---
> 
> Johan Hovold <johan@...nel.org> writes:
> 
> > Do you want to submit a patch or should I do it?
> 
> Well, I can save you that job if this is fine with you.  Feel
> free to add a stable cc if you think it is appropriate.  I am not
> sure...

Thanks! Yeah, not sure about stable either. Maybe I'll just apply this
one on top the new entry coming in, and we can defer the decision to the
stable maintainers next time there's a backport conflict.

Johan