USB: ipw: clean up dtr-rts handling
authorJohan Hovold <jhovold@gmail.com>
Thu, 13 May 2010 19:33:08 +0000 (21:33 +0200)
committerGreg Kroah-Hartman <gregkh@suse.de>
Thu, 20 May 2010 20:21:46 +0000 (13:21 -0700)
No need to raise dtr/rts in open as this is taken care of by tty layer.

Signed-off-by: Johan Hovold <jhovold@gmail.com>
Signed-off-by: Greg Kroah-Hartman <gregkh@suse.de>
drivers/usb/serial/ipw.c

index e1d07840cee66c65c2cb62feb5497bf27491cf82..18b335e0959c8d1d8790ef5674b1e5e6a2ee3f77 100644 (file)
@@ -267,35 +267,6 @@ static int ipw_open(struct tty_struct *tty, struct usb_serial_port *port)
                dev_err(&port->dev,
                        "initial flowcontrol failed (error = %d)\n", result);
 
-
-       /*--5: raise the dtr */
-       dbg("%s:raising dtr", __func__);
-       result = usb_control_msg(dev, usb_sndctrlpipe(dev, 0),
-                        IPW_SIO_SET_PIN,
-                        USB_TYPE_VENDOR | USB_RECIP_INTERFACE | USB_DIR_OUT,
-                        IPW_PIN_SETDTR,
-                        0,
-                        NULL,
-                        0,
-                        200000);
-       if (result < 0)
-               dev_err(&port->dev,
-                               "setting dtr failed (error = %d)\n", result);
-
-       /*--6: raise the rts */
-       dbg("%s:raising rts", __func__);
-       result = usb_control_msg(dev, usb_sndctrlpipe(dev, 0),
-                        IPW_SIO_SET_PIN,
-                        USB_TYPE_VENDOR | USB_RECIP_INTERFACE | USB_DIR_OUT,
-                        IPW_PIN_SETRTS,
-                        0,
-                        NULL,
-                        0,
-                        200000);
-       if (result < 0)
-               dev_err(&port->dev,
-                               "setting dtr failed (error = %d)\n", result);
-
        kfree(buf_flow_init);
        return 0;
 }
@@ -305,8 +276,8 @@ static void ipw_dtr_rts(struct usb_serial_port *port, int on)
        struct usb_device *dev = port->serial->dev;
        int result;
 
-       /*--1: drop the dtr */
-       dbg("%s:dropping dtr", __func__);
+       dbg("%s: on = %d", __func__, on);
+
        result = usb_control_msg(dev, usb_sndctrlpipe(dev, 0),
                         IPW_SIO_SET_PIN,
                         USB_TYPE_VENDOR | USB_RECIP_INTERFACE | USB_DIR_OUT,
@@ -316,22 +287,20 @@ static void ipw_dtr_rts(struct usb_serial_port *port, int on)
                         0,
                         200000);
        if (result < 0)
-               dev_err(&port->dev, "dropping dtr failed (error = %d)\n",
+               dev_err(&port->dev, "setting dtr failed (error = %d)\n",
                                                                result);
 
-       /*--2: drop the rts */
-       dbg("%s:dropping rts", __func__);
        result = usb_control_msg(dev, usb_sndctrlpipe(dev, 0),
                         IPW_SIO_SET_PIN, USB_TYPE_VENDOR |
-                                       USB_RECIP_INTERFACE | USB_DIR_OUT,
+                                       USB_RECIP_INTERFACE | USB_DIR_OUT,
                         on ? IPW_PIN_SETRTS : IPW_PIN_CLRRTS,
                         0,
                         NULL,
                         0,
                         200000);
        if (result < 0)
-               dev_err(&port->dev,
-                               "dropping rts failed (error = %d)\n", result);
+               dev_err(&port->dev, "setting rts failed (error = %d)\n",
+                                                               result);
 }
 
 static void ipw_close(struct usb_serial_port *port)