linux-kernel - RE: [PATCH] USB: serial: option: add Fibocom to DELL custom modem FM101R-GL

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-next>] [day] [month] [year] [list]

Message-ID: <TYZPR02MB5088B70C88A3751EC69819E289D7A@TYZPR02MB5088.apcprd02.prod.outlook.com>
Date:   Mon, 16 Oct 2023 08:36:30 +0000
From:   "Puliang Lu(Puliang)" <puliang.lu@...ocom.com>
To:     Johan Hovold <johan@...nel.org>
CC:     "gregkh@...uxfoundation.org" <gregkh@...uxfoundation.org>,
        "linux-usb@...r.kernel.org" <linux-usb@...r.kernel.org>,
        "linux-kernel@...r.kernel.org" <linux-kernel@...r.kernel.org>
Subject: RE: [PATCH] USB: serial: option: add Fibocom to DELL custom modem
 FM101R-GL


On Mon, Oct 16, 2023 at 03:36:16PM +0800, L-鲁璞亮 wrote:
>> From: Puliang Lu <puliang.lu@...ocom.com>
>>
>> Update the USB serial option driver support for the Fibocom
>> FM101R-GL
>
>> T:  Bus=04 Lev=01 Prnt=01 Port=01 Cnt=01 Dev#=  2 Spd=5000 MxCh= 0
>> D:  Ver= 3.20 Cls=00(>ifc ) Sub=00 Prot=00 MxPS= 9 #Cfgs=  1
>> P:  Vendor=413c ProdID=8213 Rev= 5.04
>> S:  Manufacturer=Fibocom Wireless Inc.
>> S:  Product=Fibocom FM101-GL Module
>> S:  SerialNumber=a3b7cbf0
>> C:* #Ifs= 3 Cfg#= 1 Atr=a0 MxPwr=896mA
>> 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=81(I) Atr=03(Int.) MxPS=  64 Ivl=32ms
>> 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=8e(I) Atr=02(Bulk) MxPS=1024 Ivl=0ms
>> E:  Ad=0f(O) Atr=02(Bulk) MxPS=1024 Ivl=0ms
>> I:* If#= 2 Alt= 0 #EPs= 3 Cls=ff(vend.) Sub=00 Prot=40 Driver=(none)
>> E:  Ad=83(I) Atr=03(Int.) MxPS=  10 Ivl=32ms
>> E:  Ad=82(I) Atr=02(Bulk) MxPS=1024 Ivl=0ms
>> E:  Ad=01(O) Atr=02(Bulk) MxPS=1024 Ivl=0ms
> 
> Thanks for resending with the author address fixed.
> 
> You did however not answer my question whether there could ever be more
> vendor class interfaces (e.g. for adb) so that you should be using a
> more specific match for the entries (i.e. USB_DEVICE_AND_INTERFACE_INFO())?

0x8213 and 0x8215 have only one mbim and one tty, Thanks.

>
>> +	{ USB_DEVICE_INTERFACE_CLASS(DELL_VENDOR_ID, DELL_PRODUCT_FM101R, 0xff) },
>> +	{ USB_DEVICE_INTERFACE_CLASS(DELL_VENDOR_ID, DELL_PRODUCT_FM101R_ESIM, 0xff) },
>
> Johan