Commit 97c4965d authored by Al Viro's avatar Al Viro Committed by Linus Torvalds

[PATCH] mos7840 annotations

__user annotations, NULL noise removal, %p use for pointers
Signed-off-by: default avatarAl Viro <viro@zeniv.linux.org.uk>
Signed-off-by: default avatarLinus Torvalds <torvalds@osdl.org>
parent 5a42b81f
...@@ -1087,7 +1087,7 @@ static int mos7840_open(struct usb_serial_port *port, struct file *filp) ...@@ -1087,7 +1087,7 @@ static int mos7840_open(struct usb_serial_port *port, struct file *filp)
mos7840_port->icount.tx = 0; mos7840_port->icount.tx = 0;
mos7840_port->icount.rx = 0; mos7840_port->icount.rx = 0;
dbg("\n\nusb_serial serial:%x mos7840_port:%x\n usb_serial_port port:%x\n\n", (unsigned int)serial, (unsigned int)mos7840_port, (unsigned int)port); dbg("\n\nusb_serial serial:%p mos7840_port:%p\n usb_serial_port port:%p\n\n", serial, mos7840_port, port);
return 0; return 0;
...@@ -1420,7 +1420,6 @@ static int mos7840_write(struct usb_serial_port *port, ...@@ -1420,7 +1420,6 @@ static int mos7840_write(struct usb_serial_port *port,
int i; int i;
int bytes_sent = 0; int bytes_sent = 0;
int transfer_size; int transfer_size;
int from_user = 0;
struct moschip_port *mos7840_port; struct moschip_port *mos7840_port;
struct usb_serial *serial; struct usb_serial *serial;
...@@ -1511,15 +1510,7 @@ static int mos7840_write(struct usb_serial_port *port, ...@@ -1511,15 +1510,7 @@ static int mos7840_write(struct usb_serial_port *port,
} }
transfer_size = min(count, URB_TRANSFER_BUFFER_SIZE); transfer_size = min(count, URB_TRANSFER_BUFFER_SIZE);
if (from_user) {
if (copy_from_user
(urb->transfer_buffer, current_position, transfer_size)) {
bytes_sent = -EFAULT;
goto exit;
}
} else {
memcpy(urb->transfer_buffer, current_position, transfer_size); memcpy(urb->transfer_buffer, current_position, transfer_size);
}
/* fill urb with data and submit */ /* fill urb with data and submit */
usb_fill_bulk_urb(urb, usb_fill_bulk_urb(urb,
...@@ -2225,7 +2216,7 @@ static void mos7840_set_termios(struct usb_serial_port *port, ...@@ -2225,7 +2216,7 @@ static void mos7840_set_termios(struct usb_serial_port *port,
*****************************************************************************/ *****************************************************************************/
static int mos7840_get_lsr_info(struct moschip_port *mos7840_port, static int mos7840_get_lsr_info(struct moschip_port *mos7840_port,
unsigned int *value) unsigned int __user *value)
{ {
int count; int count;
unsigned int result = 0; unsigned int result = 0;
...@@ -2248,7 +2239,7 @@ static int mos7840_get_lsr_info(struct moschip_port *mos7840_port, ...@@ -2248,7 +2239,7 @@ static int mos7840_get_lsr_info(struct moschip_port *mos7840_port,
*****************************************************************************/ *****************************************************************************/
static int mos7840_get_bytes_avail(struct moschip_port *mos7840_port, static int mos7840_get_bytes_avail(struct moschip_port *mos7840_port,
unsigned int *value) unsigned int __user *value)
{ {
unsigned int result = 0; unsigned int result = 0;
struct tty_struct *tty = mos7840_port->port->tty; struct tty_struct *tty = mos7840_port->port->tty;
...@@ -2271,7 +2262,7 @@ static int mos7840_get_bytes_avail(struct moschip_port *mos7840_port, ...@@ -2271,7 +2262,7 @@ static int mos7840_get_bytes_avail(struct moschip_port *mos7840_port,
*****************************************************************************/ *****************************************************************************/
static int mos7840_set_modem_info(struct moschip_port *mos7840_port, static int mos7840_set_modem_info(struct moschip_port *mos7840_port,
unsigned int cmd, unsigned int *value) unsigned int cmd, unsigned int __user *value)
{ {
unsigned int mcr; unsigned int mcr;
unsigned int arg; unsigned int arg;
...@@ -2341,7 +2332,7 @@ static int mos7840_set_modem_info(struct moschip_port *mos7840_port, ...@@ -2341,7 +2332,7 @@ static int mos7840_set_modem_info(struct moschip_port *mos7840_port,
*****************************************************************************/ *****************************************************************************/
static int mos7840_get_modem_info(struct moschip_port *mos7840_port, static int mos7840_get_modem_info(struct moschip_port *mos7840_port,
unsigned int *value) unsigned int __user *value)
{ {
unsigned int result = 0; unsigned int result = 0;
__u16 msr; __u16 msr;
...@@ -2370,7 +2361,7 @@ static int mos7840_get_modem_info(struct moschip_port *mos7840_port, ...@@ -2370,7 +2361,7 @@ static int mos7840_get_modem_info(struct moschip_port *mos7840_port,
*****************************************************************************/ *****************************************************************************/
static int mos7840_get_serial_info(struct moschip_port *mos7840_port, static int mos7840_get_serial_info(struct moschip_port *mos7840_port,
struct serial_struct *retinfo) struct serial_struct __user *retinfo)
{ {
struct serial_struct tmp; struct serial_struct tmp;
...@@ -2405,6 +2396,7 @@ static int mos7840_get_serial_info(struct moschip_port *mos7840_port, ...@@ -2405,6 +2396,7 @@ static int mos7840_get_serial_info(struct moschip_port *mos7840_port,
static int mos7840_ioctl(struct usb_serial_port *port, struct file *file, static int mos7840_ioctl(struct usb_serial_port *port, struct file *file,
unsigned int cmd, unsigned long arg) unsigned int cmd, unsigned long arg)
{ {
void __user *argp = (void __user *)arg;
struct moschip_port *mos7840_port; struct moschip_port *mos7840_port;
struct tty_struct *tty; struct tty_struct *tty;
...@@ -2433,16 +2425,13 @@ static int mos7840_ioctl(struct usb_serial_port *port, struct file *file, ...@@ -2433,16 +2425,13 @@ static int mos7840_ioctl(struct usb_serial_port *port, struct file *file,
case TIOCINQ: case TIOCINQ:
dbg("%s (%d) TIOCINQ", __FUNCTION__, port->number); dbg("%s (%d) TIOCINQ", __FUNCTION__, port->number);
return mos7840_get_bytes_avail(mos7840_port, return mos7840_get_bytes_avail(mos7840_port, argp);
(unsigned int *)arg);
break;
case TIOCOUTQ: case TIOCOUTQ:
dbg("%s (%d) TIOCOUTQ", __FUNCTION__, port->number); dbg("%s (%d) TIOCOUTQ", __FUNCTION__, port->number);
return put_user(tty->driver->chars_in_buffer ? return put_user(tty->driver->chars_in_buffer ?
tty->driver->chars_in_buffer(tty) : 0, tty->driver->chars_in_buffer(tty) : 0,
(int __user *)arg); (int __user *)arg);
break;
case TCFLSH: case TCFLSH:
retval = tty_check_change(tty); retval = tty_check_change(tty);
...@@ -2472,13 +2461,13 @@ static int mos7840_ioctl(struct usb_serial_port *port, struct file *file, ...@@ -2472,13 +2461,13 @@ static int mos7840_ioctl(struct usb_serial_port *port, struct file *file,
case TCGETS: case TCGETS:
if (kernel_termios_to_user_termios if (kernel_termios_to_user_termios
((struct termios __user *)arg, tty->termios)) ((struct termios __user *)argp, tty->termios))
return -EFAULT; return -EFAULT;
return 0; return 0;
case TIOCSERGETLSR: case TIOCSERGETLSR:
dbg("%s (%d) TIOCSERGETLSR", __FUNCTION__, port->number); dbg("%s (%d) TIOCSERGETLSR", __FUNCTION__, port->number);
return mos7840_get_lsr_info(mos7840_port, (unsigned int *)arg); return mos7840_get_lsr_info(mos7840_port, argp);
return 0; return 0;
case TIOCMBIS: case TIOCMBIS:
...@@ -2487,19 +2476,16 @@ static int mos7840_ioctl(struct usb_serial_port *port, struct file *file, ...@@ -2487,19 +2476,16 @@ static int mos7840_ioctl(struct usb_serial_port *port, struct file *file,
dbg("%s (%d) TIOCMSET/TIOCMBIC/TIOCMSET", __FUNCTION__, dbg("%s (%d) TIOCMSET/TIOCMBIC/TIOCMSET", __FUNCTION__,
port->number); port->number);
mosret = mosret =
mos7840_set_modem_info(mos7840_port, cmd, mos7840_set_modem_info(mos7840_port, cmd, argp);
(unsigned int *)arg);
return mosret; return mosret;
case TIOCMGET: case TIOCMGET:
dbg("%s (%d) TIOCMGET", __FUNCTION__, port->number); dbg("%s (%d) TIOCMGET", __FUNCTION__, port->number);
return mos7840_get_modem_info(mos7840_port, return mos7840_get_modem_info(mos7840_port, argp);
(unsigned int *)arg);
case TIOCGSERIAL: case TIOCGSERIAL:
dbg("%s (%d) TIOCGSERIAL", __FUNCTION__, port->number); dbg("%s (%d) TIOCGSERIAL", __FUNCTION__, port->number);
return mos7840_get_serial_info(mos7840_port, return mos7840_get_serial_info(mos7840_port, argp);
(struct serial_struct *)arg);
case TIOCSSERIAL: case TIOCSSERIAL:
dbg("%s (%d) TIOCSSERIAL", __FUNCTION__, port->number); dbg("%s (%d) TIOCSSERIAL", __FUNCTION__, port->number);
...@@ -2549,7 +2535,7 @@ static int mos7840_ioctl(struct usb_serial_port *port, struct file *file, ...@@ -2549,7 +2535,7 @@ static int mos7840_ioctl(struct usb_serial_port *port, struct file *file,
dbg("%s (%d) TIOCGICOUNT RX=%d, TX=%d", __FUNCTION__, dbg("%s (%d) TIOCGICOUNT RX=%d, TX=%d", __FUNCTION__,
port->number, icount.rx, icount.tx); port->number, icount.rx, icount.tx);
if (copy_to_user((void *)arg, &icount, sizeof(icount))) if (copy_to_user(argp, &icount, sizeof(icount)))
return -EFAULT; return -EFAULT;
return 0; return 0;
...@@ -2817,7 +2803,7 @@ static int mos7840_startup(struct usb_serial *serial) ...@@ -2817,7 +2803,7 @@ static int mos7840_startup(struct usb_serial *serial)
/* setting configuration feature to one */ /* setting configuration feature to one */
usb_control_msg(serial->dev, usb_sndctrlpipe(serial->dev, 0), usb_control_msg(serial->dev, usb_sndctrlpipe(serial->dev, 0),
(__u8) 0x03, 0x00, 0x01, 0x00, 0x00, 0x00, 5 * HZ); (__u8) 0x03, 0x00, 0x01, 0x00, NULL, 0x00, 5 * HZ);
return 0; return 0;
} }
......
Markdown is supported
0%
or
You are about to add 0 people to the discussion. Proceed with caution.
Finish editing this message first!
Please register or to comment