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