Skip to content

Commit

Permalink
USB: opticon: pass port to get_serial_info
Browse files Browse the repository at this point in the history
Pass port rather then private data to get_serial_info, which only used
the private data to access the port.

Signed-off-by: Johan Hovold <jhovold@gmail.com>
Signed-off-by: Greg Kroah-Hartman <gregkh@linuxfoundation.org>
  • Loading branch information
Johan Hovold authored and Greg Kroah-Hartman committed Nov 21, 2012
1 parent 70d25ee commit 56be1a1
Showing 1 changed file with 3 additions and 4 deletions.
7 changes: 3 additions & 4 deletions drivers/usb/serial/opticon.c
Original file line number Diff line number Diff line change
Expand Up @@ -425,7 +425,7 @@ static int opticon_tiocmset(struct tty_struct *tty,
return ret;
}

static int get_serial_info(struct opticon_private *priv,
static int get_serial_info(struct usb_serial_port *port,
struct serial_struct __user *serial)
{
struct serial_struct tmp;
Expand All @@ -437,7 +437,7 @@ static int get_serial_info(struct opticon_private *priv,

/* fake emulate a 16550 uart to make userspace code happy */
tmp.type = PORT_16550A;
tmp.line = priv->port->serial->minor;
tmp.line = port->serial->minor;
tmp.port = 0;
tmp.irq = 0;
tmp.flags = ASYNC_SKIP_TEST | ASYNC_AUTO_IRQ;
Expand All @@ -455,13 +455,12 @@ static int opticon_ioctl(struct tty_struct *tty,
unsigned int cmd, unsigned long arg)
{
struct usb_serial_port *port = tty->driver_data;
struct opticon_private *priv = usb_get_serial_port_data(port);

dev_dbg(&port->dev, "%s - port %d, cmd = 0x%x\n", __func__, port->number, cmd);

switch (cmd) {
case TIOCGSERIAL:
return get_serial_info(priv,
return get_serial_info(port,
(struct serial_struct __user *)arg);
}

Expand Down

0 comments on commit 56be1a1

Please sign in to comment.