diff options
author | Ian Abbott <abbotti@mev.co.uk> | 2005-06-20 12:10:19 -0400 |
---|---|---|
committer | Greg Kroah-Hartman <gregkh@suse.de> | 2005-07-12 14:52:55 -0400 |
commit | 7e33ae67815372a93e8e77624fd47e39a986415d (patch) | |
tree | 12efd319015d15888e6a52a45506cc5254a4763c /drivers/usb/serial/ftdi_sio.c | |
parent | 8f977e4201fcc0bd512eb01e775894e0a9c34a39 (diff) |
[PATCH] USB ftdi_sio: remove redundant TIOCMBIS and TIOCMBIC code
ftdi_sio: Remove redundant handling of TIOCMBIS and TIOCMBIC ioctls
as they are handled in the tty layer and never reach this driver.
Signed-off-by: Ian Abbott <abbotti@mev.co.uk>
Signed-off-by: Greg Kroah-Hartman <gregkh@suse.de>
Diffstat (limited to 'drivers/usb/serial/ftdi_sio.c')
-rw-r--r-- | drivers/usb/serial/ftdi_sio.c | 47 |
1 files changed, 0 insertions, 47 deletions
diff --git a/drivers/usb/serial/ftdi_sio.c b/drivers/usb/serial/ftdi_sio.c index f843f7358867..aad2f4d0e09c 100644 --- a/drivers/usb/serial/ftdi_sio.c +++ b/drivers/usb/serial/ftdi_sio.c | |||
@@ -1985,53 +1985,6 @@ static int ftdi_ioctl (struct usb_serial_port *port, struct file * file, unsigne | |||
1985 | /* Based on code from acm.c and others */ | 1985 | /* Based on code from acm.c and others */ |
1986 | switch (cmd) { | 1986 | switch (cmd) { |
1987 | 1987 | ||
1988 | case TIOCMBIS: /* turns on (Sets) the lines as specified by the mask */ | ||
1989 | dbg("%s TIOCMBIS", __FUNCTION__); | ||
1990 | if (get_user(mask, (unsigned long __user *) arg)) | ||
1991 | return -EFAULT; | ||
1992 | if (mask & TIOCM_DTR){ | ||
1993 | if ((ret = set_dtr(port, HIGH)) < 0) { | ||
1994 | err("Urb to set DTR failed"); | ||
1995 | return(ret); | ||
1996 | } | ||
1997 | } | ||
1998 | if (mask & TIOCM_RTS) { | ||
1999 | if ((ret = set_rts(port, HIGH)) < 0){ | ||
2000 | err("Urb to set RTS failed"); | ||
2001 | return(ret); | ||
2002 | } | ||
2003 | } | ||
2004 | return(0); | ||
2005 | break; | ||
2006 | |||
2007 | case TIOCMBIC: /* turns off (Clears) the lines as specified by the mask */ | ||
2008 | dbg("%s TIOCMBIC", __FUNCTION__); | ||
2009 | if (get_user(mask, (unsigned long __user *) arg)) | ||
2010 | return -EFAULT; | ||
2011 | if (mask & TIOCM_DTR){ | ||
2012 | if ((ret = set_dtr(port, LOW)) < 0){ | ||
2013 | err("Urb to unset DTR failed"); | ||
2014 | return(ret); | ||
2015 | } | ||
2016 | } | ||
2017 | if (mask & TIOCM_RTS) { | ||
2018 | if ((ret = set_rts(port, LOW)) < 0){ | ||
2019 | err("Urb to unset RTS failed"); | ||
2020 | return(ret); | ||
2021 | } | ||
2022 | } | ||
2023 | return(0); | ||
2024 | break; | ||
2025 | |||
2026 | /* | ||
2027 | * I had originally implemented TCSET{A,S}{,F,W} and | ||
2028 | * TCGET{A,S} here separately, however when testing I | ||
2029 | * found that the higher layers actually do the termios | ||
2030 | * conversions themselves and pass the call onto | ||
2031 | * ftdi_sio_set_termios. | ||
2032 | * | ||
2033 | */ | ||
2034 | |||
2035 | case TIOCGSERIAL: /* gets serial port data */ | 1988 | case TIOCGSERIAL: /* gets serial port data */ |
2036 | return get_serial_info(port, (struct serial_struct __user *) arg); | 1989 | return get_serial_info(port, (struct serial_struct __user *) arg); |
2037 | 1990 | ||