Re: [PATCH] USB: serial: qcserial: add support for Sierra Wireless EM7511

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

 



On Sat, Aug 24, 2024 at 8:33 PM Reinhard Speyerer <rspmn@xxxxxxxx> wrote:
>
> On Fri, Aug 23, 2024 at 09:35:53PM +0300, alon.barlev@xxxxxxxxx wrote:
> > From: Alon Bar-Lev <alon.barlev@xxxxxxxxx>
> > +     {DEVICE_SWI(0x1199, 0x90b0)},   /* Sierra Wireless EM7511 QDL */
> > +     {DEVICE_SWI(0x1199, 0x90b1)},   /* Sierra Wireless EM7511 */
>
> Hi Alon,
>
> the interface layout for the DEVICE_SWI(0x1199, 0x90b1) entry from your patch
> disagrees with the usb-devices output from
> https://forum.sierrawireless.com/t/em7565-1199-90b1-usbx-qcserial-issue-ubuntu-20-04-mbpl-driver-fail-to-compile/26361
> and the Sierra Wireless MBPL_DRIVER_R27_ENG2-usb-src.tar.gz driver attached
> to this thread https://forum.sierrawireless.com/t/driver-make-error/30702 .
>
> As a consequence the DIAG interface would not be supported by the mainline
> Linux kernel.
>
> Regards,
> Reinhard

Hi Reinhard,

Thank you for the references.

The push back is regarding hardware specific modification of qcserial
to meet device requirement.
To make it harder, Sierra patches upstream modules and provides as-is
without the ability of side-by-side installation.
I compared the changes of qmi_wwan, usb_wwan most are trivial and
hopefully reach upstream.
I hope they workout and add the options support so this can be
upstreamed eventually.

In order to load firmware it should have been sufficient to have QDL
setup + somehow setup the tty for the AT channel.

    echo $vendor $product $interface >
/sys/bus/usb-serial/drivers/qcserial/new_id

Would have done the trick to be able to use this at least manually
without forcing the device specific nor options at this point, not
sure how simple it is to support this notation.

For now I will keep using the patched version of qcserial as it
actually works :)

Thank you again,
Alon





[Index of Archives]     [Linux Media]     [Linux Input]     [Linux Audio Users]     [Yosemite News]     [Linux Kernel]     [Linux SCSI]     [Old Linux USB Devel Archive]

  Powered by Linux