Re: Aw: Re: [PATCH 1/2] USB: serial: qcserial: add new usb-id for Dell branded EM7455

[Johan Hovold <johan@kernel.org>]

On Tue, Sep 27, 2022 at 08:48:23AM +0200, Frank Wunderlich wrote:
> Hi
> 
> > Gesendet: Dienstag, 27. September 2022 um 08:31 Uhr
> > Von: "Johan Hovold" <johan@kernel.org>

> > On Mon, Sep 26, 2022 at 05:07:39PM +0200, Frank Wunderlich wrote:
> > > From: Frank Wunderlich <frank-w@public-files.de>
> 
> > > +++ b/drivers/usb/serial/qcserial.c
> > > @@ -177,6 +177,7 @@ static const struct usb_device_id id_table[] = {
> > >  	{DEVICE_SWI(0x413c, 0x81b3)},	/* Dell Wireless 5809e Gobi(TM) 4G LTE Mobile Broadband Card (rev3) */
> > >  	{DEVICE_SWI(0x413c, 0x81b5)},	/* Dell Wireless 5811e QDL */
> > >  	{DEVICE_SWI(0x413c, 0x81b6)},	/* Dell Wireless 5811e QDL */
> > > +	{DEVICE_SWI(0x413c, 0x81c2)},	/* Dell Wireless 5811e QDL */
> > 
> > I assume this is not just for QDL mode as the comment indicates.
> 
> to be honest, have not found out yet what QDL means and assumed that
> it's like the other dw5811e, so not changed comment :)

I believe that's Qualcomm Download mode or similar, for flashing the
device (cf. 5816e which has two entries, one for QDL mode).

> > Could you post the output of usb-devices for this device?
> 
> Bus 001 Device 004: ID 413c:81c2 Sierra Wireless, Incorporated DW5811e Snapdragon™ X7 LTE
> 
> 
> /:  Bus 01.Port 1: Dev 1, Class=root_hub, Driver=xhci-mtk/2p, 480M                                                                  
>     ID 1d6b:0002 Linux Foundation 2.0 root hub                                                                                      
>     |__ Port 1: Dev 2, If 0, Class=Hub, Driver=hub/4p, 480M                                                                         
>         ID 1a40:0101 Terminus Technology Inc. Hub                                                                                   
>         |__ Port 1: Dev 6, If 0, Class=Vendor Specific Class, Driver=qcserial, 480M                                                 
>             ID 413c:81c2 Dell Computer Corp.                                                                                        
>         |__ Port 1: Dev 6, If 2, Class=Vendor Specific Class, Driver=qcserial, 480M                                                 
>             ID 413c:81c2 Dell Computer Corp.                                                                                        
>         |__ Port 1: Dev 6, If 3, Class=Vendor Specific Class, Driver=qcserial, 480M                                                 
>             ID 413c:81c2 Dell Computer Corp.                                                                                        
>         |__ Port 1: Dev 6, If 8, Class=Vendor Specific Class, Driver=qmi_wwan, 480M                                                 
>             ID 413c:81c2 Dell Computer Corp.        

Thanks. The above doesn't include all the details that usb-devices (or
lsusb -v) would but still confirms the basic bits so I've applied the
patch now after amending the comment.

Johan