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  PHC 
Open Source and information security mailing list archives
 
Hash Suite: Windows password security audit tool. GUI, reports in PDF.
[<prev] [next>] [<thread-prev] [day] [month] [year] [list]
Date:   Mon, 31 Aug 2020 08:43:06 +0200
From:   Johan Hovold <johan@...nel.org>
To:     Aleksander Morgado <aleksander@...ksander.es>
Cc:     johan@...nel.org, gregkh@...uxfoundation.org,
        linux-usb@...r.kernel.org, linux-kernel@...r.kernel.org
Subject: Re: [PATCH] USB: serial: option: add support for
 SIM7070/SIM7080/SIM7090 modules

On Sat, Aug 29, 2020 at 11:05:39AM +0200, Aleksander Morgado wrote:
> These modules have 2 different USB layouts:
> 
> The default layout with PID 0x9205 (AT+CUSBSELNV=1) exposes 4 TTYs and
> an ECM interface:
> 
>   T:  Bus=02 Lev=01 Prnt=01 Port=02 Cnt=01 Dev#=  6 Spd=480 MxCh= 0
>   D:  Ver= 2.00 Cls=ef(misc ) Sub=02 Prot=01 MxPS=64 #Cfgs=  1
>   P:  Vendor=1e0e ProdID=9205 Rev=00.00
>   S:  Manufacturer=SimTech, Incorporated
>   S:  Product=SimTech SIM7080
>   S:  SerialNumber=1234567890ABCDEF
>   C:  #Ifs= 6 Cfg#= 1 Atr=e0 MxPwr=500mA
>   I:  If#=0x0 Alt= 0 #EPs= 2 Cls=ff(vend.) Sub=ff Prot=ff Driver=option
>   I:  If#=0x1 Alt= 0 #EPs= 2 Cls=ff(vend.) Sub=ff Prot=ff Driver=option
>   I:  If#=0x2 Alt= 0 #EPs= 2 Cls=ff(vend.) Sub=ff Prot=ff Driver=option
>   I:  If#=0x3 Alt= 0 #EPs= 3 Cls=ff(vend.) Sub=ff Prot=ff Driver=option
>   I:  If#=0x4 Alt= 0 #EPs= 1 Cls=02(commc) Sub=06 Prot=00 Driver=cdc_ether
>   I:  If#=0x5 Alt= 1 #EPs= 2 Cls=0a(data ) Sub=00 Prot=00 Driver=cdc_ether
> 
> The purpose of each TTY is as follows:
>  * ttyUSB0: DIAG/QCDM port.
>  * ttyUSB1: GNSS data.
>  * ttyUSB2: AT-capable port (control).
>  * ttyUSB3: AT-capable port (data).
> 
> In the secondary layout with PID=0x9206 (AT+CUSBSELNV=86) the module
> exposes 6 TTY ports:
> 
>   T:  Bus=02 Lev=01 Prnt=01 Port=02 Cnt=01 Dev#=  8 Spd=480 MxCh= 0
>   D:  Ver= 2.00 Cls=02(commc) Sub=00 Prot=00 MxPS=64 #Cfgs=  1
>   P:  Vendor=1e0e ProdID=9206 Rev=00.00
>   S:  Manufacturer=SimTech, Incorporated
>   S:  Product=SimTech SIM7080
>   S:  SerialNumber=1234567890ABCDEF
>   C:  #Ifs= 6 Cfg#= 1 Atr=e0 MxPwr=500mA
>   I:  If#=0x0 Alt= 0 #EPs= 2 Cls=ff(vend.) Sub=ff Prot=ff Driver=option
>   I:  If#=0x1 Alt= 0 #EPs= 2 Cls=ff(vend.) Sub=ff Prot=ff Driver=option
>   I:  If#=0x2 Alt= 0 #EPs= 2 Cls=ff(vend.) Sub=ff Prot=ff Driver=option
>   I:  If#=0x3 Alt= 0 #EPs= 2 Cls=ff(vend.) Sub=ff Prot=ff Driver=option
>   I:  If#=0x4 Alt= 0 #EPs= 2 Cls=ff(vend.) Sub=ff Prot=ff Driver=option
>   I:  If#=0x5 Alt= 0 #EPs= 3 Cls=ff(vend.) Sub=ff Prot=ff Driver=option
> 
> The purpose of each TTY is as follows:
>  * ttyUSB0: DIAG/QCDM port.
>  * ttyUSB1: GNSS data.
>  * ttyUSB2: AT-capable port (control).
>  * ttyUSB3: QFLOG interface.
>  * ttyUSB4: DAM interface.
>  * ttyUSB5: AT-capable port (data).
> 
> Signed-off-by: Aleksander Morgado <aleksander@...ksander.es>

Now applied. Thanks, Aleksander.

Johan

Powered by blists - more mailing lists