diff options
author | Alan Cox <alan@lxorguk.ukuu.org.uk> | 2008-02-20 15:51:45 -0500 |
---|---|---|
committer | Greg Kroah-Hartman <gregkh@suse.de> | 2008-04-25 00:16:39 -0400 |
commit | e298449401463dd18f24a87c48f9b0ec62bad936 (patch) | |
tree | 12f84e1370d9e81b15f742dfba1ee5b3b60ef108 | |
parent | 9b0f2582d57d4c9081307c86e11afc9169de7d3e (diff) |
USB: serial: Note mos7480 and option don't lock modem status
Signed-off-by: Alan Cox <alan@redhat.com>
Signed-off-by: Greg Kroah-Hartman <gregkh@suse.de>
-rw-r--r-- | drivers/usb/serial/mos7840.c | 1 | ||||
-rw-r--r-- | drivers/usb/serial/option.c | 1 |
2 files changed, 2 insertions, 0 deletions
diff --git a/drivers/usb/serial/mos7840.c b/drivers/usb/serial/mos7840.c index aeeb9cb20999..0b29c5383dc8 100644 --- a/drivers/usb/serial/mos7840.c +++ b/drivers/usb/serial/mos7840.c | |||
@@ -1722,6 +1722,7 @@ static int mos7840_tiocmset(struct usb_serial_port *port, struct file *file, | |||
1722 | if (mos7840_port == NULL) | 1722 | if (mos7840_port == NULL) |
1723 | return -ENODEV; | 1723 | return -ENODEV; |
1724 | 1724 | ||
1725 | /* FIXME: What locks the port registers ? */ | ||
1725 | mcr = mos7840_port->shadowMCR; | 1726 | mcr = mos7840_port->shadowMCR; |
1726 | if (clear & TIOCM_RTS) | 1727 | if (clear & TIOCM_RTS) |
1727 | mcr &= ~MCR_RTS; | 1728 | mcr &= ~MCR_RTS; |
diff --git a/drivers/usb/serial/option.c b/drivers/usb/serial/option.c index d101025a4c63..0e7eeb2820e2 100644 --- a/drivers/usb/serial/option.c +++ b/drivers/usb/serial/option.c | |||
@@ -458,6 +458,7 @@ static int option_tiocmset(struct usb_serial_port *port, struct file *file, | |||
458 | 458 | ||
459 | portdata = usb_get_serial_port_data(port); | 459 | portdata = usb_get_serial_port_data(port); |
460 | 460 | ||
461 | /* FIXME: what locks portdata fields ? */ | ||
461 | if (set & TIOCM_RTS) | 462 | if (set & TIOCM_RTS) |
462 | portdata->rts_state = 1; | 463 | portdata->rts_state = 1; |
463 | if (set & TIOCM_DTR) | 464 | if (set & TIOCM_DTR) |