2022-09-27 07:05:42

by Frank Wunderlich

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

Hi

> Gesendet: Dienstag, 27. September 2022 um 08:31 Uhr
> Von: "Johan Hovold" <[email protected]>
> An: "Frank Wunderlich" <[email protected]>
> Cc: [email protected], "Frank Wunderlich" <[email protected]>, "Bjørn Mork" <[email protected]>, "David S. Miller" <[email protected]>, "Eric Dumazet" <[email protected]>, "Jakub Kicinski" <[email protected]>, "Paolo Abeni" <[email protected]>, "Greg Kroah-Hartman" <[email protected]>, [email protected], [email protected], [email protected]
> Betreff: Re: [PATCH 1/2] USB: serial: qcserial: add new usb-id for Dell branded EM7455
>
> On Mon, Sep 26, 2022 at 05:07:39PM +0200, Frank Wunderlich wrote:
> > From: Frank Wunderlich <[email protected]>

> > +++ 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 :)

> 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.

> > {DEVICE_SWI(0x413c, 0x81cb)}, /* Dell Wireless 5816e QDL */
> > {DEVICE_SWI(0x413c, 0x81cc)}, /* Dell Wireless 5816e */
> > {DEVICE_SWI(0x413c, 0x81cf)}, /* Dell Wireless 5819 */
>
> Johan

regards Frank


2022-09-27 07:09:18

by Johan Hovold

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

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" <[email protected]>

> > On Mon, Sep 26, 2022 at 05:07:39PM +0200, Frank Wunderlich wrote:
> > > From: Frank Wunderlich <[email protected]>
>
> > > +++ 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