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:   Thu, 7 Oct 2021 09:57:01 +0200
From:   Johan Hovold <johan@kernel.org>
To:     Yu-Tung Chang <mtwget@gmail.com>
Cc:     gregkh@linuxfoundation.org, 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: <YV6oTZae5zbgX8cV@hovoldconsulting.com>
References: <20210930021112.330396-1-mtwget@gmail.com>
 <YV6mD02BVLXXcTr8@hovoldconsulting.com>
MIME-Version: 1.0
Content-Type: text/plain; charset=us-ascii
Content-Disposition: inline
In-Reply-To: <YV6mD02BVLXXcTr8@hovoldconsulting.com>
Precedence: bulk

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.

Johan