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;
MIME-Version: 1.0
References: <20200717185118.3640219-1-erik@kryo.se> <20200721072350.GC3634@localhost>
In-Reply-To: <20200721072350.GC3634@localhost>
From:   Erik Ekman <erik@kryo.se>
Date:   Tue, 21 Jul 2020 20:15:04 +0200
Message-ID: <CAGgu=sDBemKU4vzg5xWyWT-OoDuXrxXp2tkB5OOmNW6ncDjpow@mail.gmail.com>
Subject: Re: [PATCH] USB: serial: qcserial: add EM7305 QDL product ID
To:     Johan Hovold <johan@kernel.org>
Cc:     Greg Kroah-Hartman <gregkh@linuxfoundation.org>,
        linux-usb@vger.kernel.org, linux-kernel@vger.kernel.org
Content-Type: text/plain; charset="UTF-8"
Sender: linux-kernel-owner@vger.kernel.org
Precedence: bulk

On Tue, 21 Jul 2020 at 09:23, Johan Hovold <johan@kernel.org> wrote:
>
> On Fri, Jul 17, 2020 at 08:51:18PM +0200, Erik Ekman wrote:
> > When running qmi-firmware-update on the Sierra Wireless EM7305 in a Toshiba
> > laptop, it changed product ID to 0x9062 when entering QDL mode:
> >
> > usb 2-4: new high-speed USB device number 78 using xhci_hcd
> > usb 2-4: New USB device found, idVendor=1199, idProduct=9062, bcdDevice= 0.00
> > usb 2-4: New USB device strings: Mfr=1, Product=2, SerialNumber=0
> > usb 2-4: Product: EM7305
> > usb 2-4: Manufacturer: Sierra Wireless, Incorporated
> >
> > The upgrade could complete after running
> >  # echo 1199 9062 > /sys/bus/usb-serial/drivers/qcserial/new_id
> >
> > qcserial 2-4:1.0: Qualcomm USB modem converter detected
> > usb 2-4: Qualcomm USB modem converter now attached to ttyUSB0
> >
> > Signed-off-by: Erik Ekman <erik@kryo.se>
>
> Applied, thanks.
>
> Would you mind posting the output of "usb-devices" (or "lsusb -v") for
> this device for completeness?
>
(resend as plain text)

from usb-devices:

T:  Bus=02 Lev=01 Prnt=01 Port=03 Cnt=03 Dev#= 30 Spd=480 MxCh= 0
D:  Ver= 2.00 Cls=00(>ifc ) Sub=00 Prot=00 MxPS=64 #Cfgs=  1
P:  Vendor=1199 ProdID=9062 Rev=00.00
S:  Manufacturer=Sierra Wireless, Incorporated
S:  Product=EM7305
C:  #Ifs= 1 Cfg#= 1 Atr=80 MxPwr=2mA
I:  If#=0x0 Alt= 0 #EPs= 2 Cls=ff(vend.) Sub=ff Prot=ff Driver=(none)

and with the driver it instead ends with:
I:  If#=0x0 Alt= 0 #EPs= 2 Cls=ff(vend.) Sub=ff Prot=ff Driver=qcserial
as expected.

Cheers
/Erik

>
> Johan