There no need to reinitialise the private urb at every open.

Signed-off-by: Johan Hovold <[email protected]>
---
 drivers/usb/serial/opticon.c | 13 ++++++-------
 1 file changed, 6 insertions(+), 7 deletions(-)

diff --git a/drivers/usb/serial/opticon.c b/drivers/usb/serial/opticon.c
index cb8674e..8c66471 100644
--- a/drivers/usb/serial/opticon.c
+++ b/drivers/usb/serial/opticon.c
@@ -184,13 +184,6 @@ static int opticon_open(struct tty_struct *tty, struct 
usb_serial_port *port)
        /* Clear RTS line */
        send_control_msg(port, CONTROL_RTS, 0);
 
-       /* Setup the read URB and start reading from the device */
-       usb_fill_bulk_urb(priv->bulk_read_urb, priv->udev,
-                         usb_rcvbulkpipe(priv->udev,
-                                         priv->bulk_address),
-                         priv->bulk_in_buffer, priv->buffer_size,
-                         opticon_read_bulk_callback, priv);
-
        /* clear the halt status of the enpoint */
        usb_clear_halt(priv->udev, priv->bulk_read_urb->pipe);
 
@@ -530,6 +523,12 @@ static int opticon_startup(struct usb_serial *serial)
                goto error;
        }
 
+       usb_fill_bulk_urb(priv->bulk_read_urb, serial->dev,
+                               usb_rcvbulkpipe(serial->dev,
+                                               priv->bulk_address),
+                               priv->bulk_in_buffer, priv->buffer_size,
+                               opticon_read_bulk_callback, priv);
+
        usb_set_serial_data(serial, priv);
        return 0;
 
-- 
1.7.12.4

--
To unsubscribe from this list: send the line "unsubscribe linux-usb" in
the body of a message to [email protected]
More majordomo info at  http://vger.kernel.org/majordomo-info.html

Reply via email to