linux-kernel.vger.kernel.org archive mirror
 help / color / mirror / Atom feed
* [PATCH v2] USB: serial: option: Add Telit FT980-KS composition
@ 2020-10-04 15:58 Leonid Bloch
  2020-10-05  8:12 ` Johan Hovold
  0 siblings, 1 reply; 2+ messages in thread
From: Leonid Bloch @ 2020-10-04 15:58 UTC (permalink / raw)
  To: linux-kernel, Lars Melin
  Cc: linux-usb, Greg Kroah-Hartman, Johan Hovold, Leonid Bloch

This commit adds the following Telit FT980-KS composition:

0x1054: rndis, diag, adb, nmea, modem, modem, aux

AT commands can be sent to /dev/ttyUSB2.

Signed-off-by: Leonid Bloch <lb.workbox@gmail.com>
---

The full composition is not tested, and it is the default one according
to Telit support. What is tested, is that this commit makes
/dev/ttyUSB{0..4} appear upon connecting the FT980-KS, and allows
sending AT commands to /dev/ttyUSB2.

Changes since v1:

* Interface #3 (ADB) is blacklisted.
* NCTRL flag is set to the diag interface.

These changes (relative to v1) also allow ModemManager to recognize the
device.

 drivers/usb/serial/option.c | 2 ++
 1 file changed, 2 insertions(+)

diff --git a/drivers/usb/serial/option.c b/drivers/usb/serial/option.c
index 0c6f160a214a..fe76710167f8 100644
--- a/drivers/usb/serial/option.c
+++ b/drivers/usb/serial/option.c
@@ -1186,6 +1186,8 @@ static const struct usb_device_id option_ids[] = {
 	  .driver_info = NCTRL(2) | RSVD(3) },
 	{ USB_DEVICE_INTERFACE_CLASS(TELIT_VENDOR_ID, 0x1053, 0xff),	/* Telit FN980 (ECM) */
 	  .driver_info = NCTRL(0) | RSVD(1) },
+	{ USB_DEVICE_INTERFACE_CLASS(TELIT_VENDOR_ID, 0x1054, 0xff),	/* Telit FT980-KS */
+	  .driver_info = NCTRL(2) | RSVD(3) },
 	{ USB_DEVICE(TELIT_VENDOR_ID, TELIT_PRODUCT_ME910),
 	  .driver_info = NCTRL(0) | RSVD(1) | RSVD(3) },
 	{ USB_DEVICE(TELIT_VENDOR_ID, TELIT_PRODUCT_ME910_DUAL_MODEM),
-- 
2.28.0


^ permalink raw reply related	[flat|nested] 2+ messages in thread

* Re: [PATCH v2] USB: serial: option: Add Telit FT980-KS composition
  2020-10-04 15:58 [PATCH v2] USB: serial: option: Add Telit FT980-KS composition Leonid Bloch
@ 2020-10-05  8:12 ` Johan Hovold
  0 siblings, 0 replies; 2+ messages in thread
From: Johan Hovold @ 2020-10-05  8:12 UTC (permalink / raw)
  To: Leonid Bloch
  Cc: linux-kernel, Lars Melin, linux-usb, Greg Kroah-Hartman,
	Johan Hovold, Leonid Bloch

On Sun, Oct 04, 2020 at 06:58:13PM +0300, Leonid Bloch wrote:
> This commit adds the following Telit FT980-KS composition:
> 
> 0x1054: rndis, diag, adb, nmea, modem, modem, aux
> 
> AT commands can be sent to /dev/ttyUSB2.
> 
> Signed-off-by: Leonid Bloch <lb.workbox@gmail.com>
> ---
> 
> The full composition is not tested, and it is the default one according
> to Telit support. What is tested, is that this commit makes
> /dev/ttyUSB{0..4} appear upon connecting the FT980-KS, and allows
> sending AT commands to /dev/ttyUSB2.
> 
> Changes since v1:
> 
> * Interface #3 (ADB) is blacklisted.
> * NCTRL flag is set to the diag interface.
> 
> These changes (relative to v1) also allow ModemManager to recognize the
> device.

Now applied.

And thanks for reviewing, Lars.

Johan

^ permalink raw reply	[flat|nested] 2+ messages in thread

end of thread, other threads:[~2020-10-05  8:12 UTC | newest]

Thread overview: 2+ messages (download: mbox.gz / follow: Atom feed)
-- links below jump to the message on this page --
2020-10-04 15:58 [PATCH v2] USB: serial: option: Add Telit FT980-KS composition Leonid Bloch
2020-10-05  8:12 ` Johan Hovold

This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox;
as well as URLs for NNTP newsgroup(s).