Since commit 31ca020b ("TTY: wake up processes last at hangup") there no
longer any need to check the hupping flag in the generic tiocmiwait
implementation, so remove it.

Signed-off-by: Johan Hovold <jhov...@gmail.com>
---
 drivers/usb/serial/generic.c | 13 ++-----------
 1 file changed, 2 insertions(+), 11 deletions(-)

diff --git a/drivers/usb/serial/generic.c b/drivers/usb/serial/generic.c
index ba45170..1f31e6b 100644
--- a/drivers/usb/serial/generic.c
+++ b/drivers/usb/serial/generic.c
@@ -460,12 +460,7 @@ static bool usb_serial_generic_msr_changed(struct 
tty_struct *tty,
        /*
         * Use tty-port initialised flag to detect all hangups including the
         * one generated at USB-device disconnect.
-        *
-        * FIXME: Remove hupping check once tty_port_hangup calls shutdown
-        *        (which clears the initialised flag) before wake up.
         */
-       if (test_bit(TTY_HUPPING, &tty->flags))
-               return true;
        if (!test_bit(ASYNCB_INITIALIZED, &port->port.flags))
                return true;
 
@@ -496,12 +491,8 @@ int usb_serial_generic_tiocmiwait(struct tty_struct *tty, 
unsigned long arg)
 
        ret = wait_event_interruptible(port->port.delta_msr_wait,
                        usb_serial_generic_msr_changed(tty, arg, &cnow));
-       if (!ret) {
-               if (test_bit(TTY_HUPPING, &tty->flags))
-                       ret = -EIO;
-               if (!test_bit(ASYNCB_INITIALIZED, &port->port.flags))
-                       ret = -EIO;
-       }
+       if (!ret && !test_bit(ASYNCB_INITIALIZED, &port->port.flags))
+               ret = -EIO;
 
        return ret;
 }
-- 
1.8.2.1

--
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