Drop the redundant read-urb check from open. The presence of a bulk-in
endpoint is now verified during probe and core has allocated the
corresponding resources.

Signed-off-by: Johan Hovold <jo...@kernel.org>
---
 drivers/usb/serial/io_ti.c | 6 ------
 1 file changed, 6 deletions(-)

diff --git a/drivers/usb/serial/io_ti.c b/drivers/usb/serial/io_ti.c
index a962082cf3b0..f3ed131d14bf 100644
--- a/drivers/usb/serial/io_ti.c
+++ b/drivers/usb/serial/io_ti.c
@@ -1952,12 +1952,6 @@ static int edge_open(struct tty_struct *tty, struct 
usb_serial_port *port)
 
        /* start up our bulk read urb */
        urb = port->read_urb;
-       if (!urb) {
-               dev_err(&port->dev, "%s - no read urb present, exiting\n",
-                                                               __func__);
-               status = -EINVAL;
-               goto unlink_int_urb;
-       }
        edge_port->ep_read_urb_state = EDGE_READ_URB_RUNNING;
        urb->context = edge_port;
        status = usb_submit_urb(urb, GFP_KERNEL);
-- 
2.12.0

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

Reply via email to