Re: [PATCH v1] USB: serial: option: add Quectel EC200S-CN module support

[Johan Hovold <johan@kernel.org>]

[ Adding back the lists on CC so that this gets archived. ]

On Sat, Oct 09, 2021 at 01:50:40PM +0800, Yu-Tung Chang wrote:
> Yu-Tung Chang <mtwget@gmail.com> 于2021年10月9日周六 下午1:39写道：
> >
> > Johan Hovold <johan@kernel.org> 于2021年10月7日周四 下午3:57写道：
> > >
> > > On Thu, Oct 07, 2021 at 09:47:28AM +0200, Johan Hovold wrote:
> > > > On Thu, Sep 30, 2021 at 10:11:12AM +0800, Yu-Tung Chang wrote:
> > > > > Add usb product id of the Quectel EC200S-CN module.
> > > > >
> > > > > usb-devices output for 0x6002:
> > > > > T:  Bus=01 Lev=01 Prnt=01 Port=00 Cnt=01 Dev#=  3 Spd=480 MxCh= 0
> > > > > D:  Ver= 2.00 Cls=ef(misc ) Sub=02 Prot=01 MxPS=64 #Cfgs=  1
> > > > > P:  Vendor=2c7c ProdID=6002 Rev=03.18
> > > > > S:  Manufacturer=Android
> > > > > S:  Product=Android
> > > > > S:  SerialNumber=0000
> > > > > C:  #Ifs= 5 Cfg#= 1 Atr=e0 MxPwr=500mA
> > > > > I:  If#=0x0 Alt= 0 #EPs= 1 Cls=02(commc) Sub=06 Prot=00 Driver=cdc_ether
> > > > > I:  If#=0x1 Alt= 1 #EPs= 2 Cls=0a(data ) Sub=00 Prot=00 Driver=cdc_ether
> > > > > I:  If#=0x2 Alt= 0 #EPs= 2 Cls=ff(vend.) Sub=00 Prot=00 Driver=(none)
> > > > > I:  If#=0x3 Alt= 0 #EPs= 3 Cls=ff(vend.) Sub=00 Prot=00 Driver=(none)
> > > > > I:  If#=0x4 Alt= 0 #EPs= 3 Cls=ff(vend.) Sub=00 Prot=00 Driver=(none)
> > > >
> > > > Looks good now, but for completeness, could you also mention here what
> > > > the three ports are used for (and which is which)?
> > >
> > > I've applied the patch now, so no need to resend, but please list the
> > > three ports in a reply if you have that information.
> >
> > ttyUSB0 used for debugging, ttyUSB1 used for AT commands,
> > ttyUSB2 used for PPP dialing.

> and lsusb.py -i -e output info:
>   1-1               2c7c:6002 ef 5IFs [USB 2.00,   480 Mbps, 500mA]
> (Android Android 0000)
>                       (EP) 00: Control attr 00 len 07 max 040
>     1-1:1.0           (IF) 02:06:00 1EP  (Communications) cdc_ether net/usb0
>                         (EP) 87: Interrupt (4096ms) attr 03 len 07 max 040
>     1-1:1.1           (IF) 0a:00:00 2EPs (CDC Data) cdc_ether
>                         (EP) 83: Bulk attr 02 len 07 max 200
>                         (EP) 0c: Bulk attr 02 len 07 max 200
>     1-1:1.2           (IF) ff:00:00 2EPs (Vendor Specific Class) option ttyUSB0
>                         (EP) 0b: Bulk attr 02 len 07 max 200
>                         (EP) 82: Bulk attr 02 len 07 max 200
>     1-1:1.3           (IF) ff:00:00 3EPs (Vendor Specific Class) option ttyUSB1
>                         (EP) 89: Interrupt (4096ms) attr 03 len 07 max 040
>                         (EP) 86: Bulk attr 02 len 07 max 200
>                         (EP) 0f: Bulk attr 02 len 07 max 200
>     1-1:1.4           (IF) ff:00:00 3EPs (Vendor Specific Class) option ttyUSB2
>                         (EP) 81: Bulk attr 02 len 07 max 200
>                         (EP) 0a: Bulk attr 02 len 07 max 200
>                         (EP) 88: Interrupt (4096ms) attr 03 len 07 max 040

Perfect, thanks for the details.

Johan