Received-SPF: pass (google.com: domain of linux-kernel-owner@vger.kernel.org designates 23.128.96.18 as permitted sender) client-ip=23.128.96.18;
Date:   Mon, 11 Oct 2021 10:31:50 +0200
From:   Johan Hovold <johan@kernel.org>
To:     Yu-Tung Chang <mtwget@gmail.com>
Cc:     linux-usb@vger.kernel.org, linux-kernel@vger.kernel.org
Subject: Re: [PATCH v1] USB: serial: option: add Quectel EC200S-CN module
 support
Message-ID: <YWP2dpna7WSGen76@hovoldconsulting.com>
References: <20210930021112.330396-1-mtwget@gmail.com>
 <YV6mD02BVLXXcTr8@hovoldconsulting.com>
 <YV6oTZae5zbgX8cV@hovoldconsulting.com>
 <CAHRgzyNcmhTJHC9vzS-1mbxMLVHw5bYWHVy1+KCvUF61tRnr=A@mail.gmail.com>
 <CAHRgzyO=VOCAV8oGSR15hiifp7NqHCu13KjcTrv3aA2QiURz_g@mail.gmail.com>
MIME-Version: 1.0
Content-Type: text/plain; charset=utf-8
Content-Disposition: inline
Content-Transfer-Encoding: 8bit
In-Reply-To: <CAHRgzyO=VOCAV8oGSR15hiifp7NqHCu13KjcTrv3aA2QiURz_g@mail.gmail.com>
Precedence: bulk

[ 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