USB: serial: qcserial: expose methods for modem control
authorMagnus Lynch <maglyx@gmail.com>
Mon, 12 Jun 2017 18:52:41 +0000 (11:52 -0700)
committerJohan Hovold <johan@kernel.org>
Tue, 13 Jun 2017 07:06:25 +0000 (09:06 +0200)
The qcserial driver fails to expose the .tiocmget and .tiocmset methods
available from usb_wwan. These methods are required by ioctl commands
dealing with the modem control signals DTR, RTS, etc.

With these methods not set ioctl calls intended to control the DTR state
will fail. For example, pppd drops and raises DTR in preparation to
dialing the modem, which handles the case of the modem already being
connected by making it hang up and return to command mode. DTR control
being unavailable will lead to a protracted failure to connect as the
modem will be stuck in a state not responsive to command.

I have tested that with this patch the described case is handled
successfully. There is an analogous method for .ioctl available from
usb_wwan (as used in option.c) but I conservatively omitted that for
lack of familiarity.

Signed-off-by: Magnus Lynch <maglyx@gmail.com>
Signed-off-by: Johan Hovold <johan@kernel.org>
drivers/usb/serial/qcserial.c

index fd509ed6cf7065725c097ab1bc4eac84359be1f6..4ac137d070fbc55fef25556c52d2f9449a866dbc 100644 (file)
@@ -454,6 +454,8 @@ static struct usb_serial_driver qcdevice = {
        .write               = usb_wwan_write,
        .write_room          = usb_wwan_write_room,
        .chars_in_buffer     = usb_wwan_chars_in_buffer,
+       .tiocmget            = usb_wwan_tiocmget,
+       .tiocmset            = usb_wwan_tiocmset,
        .attach              = qc_attach,
        .release             = qc_release,
        .port_probe          = usb_wwan_port_probe,