Remove no longer needed disconnected test from close, which is never
called post disconnect (and drivers must handle failed I/O during
disconnect anyway).

Signed-off-by: Johan Hovold <jhov...@gmail.com>
---
 drivers/usb/serial/sierra.c | 14 +++++---------
 1 file changed, 5 insertions(+), 9 deletions(-)

diff --git a/drivers/usb/serial/sierra.c b/drivers/usb/serial/sierra.c
index 854ac61581ba..74b417c91e30 100644
--- a/drivers/usb/serial/sierra.c
+++ b/drivers/usb/serial/sierra.c
@@ -772,16 +772,12 @@ static void sierra_close(struct usb_serial_port *port)
        portdata->rts_state = 0;
        portdata->dtr_state = 0;
 
-       mutex_lock(&serial->disc_mutex);
-       if (!serial->disconnected) {
-               /* odd error handling due to pm counters */
-               if (!usb_autopm_get_interface(serial->interface))
-                       sierra_send_setup(port);
-               else
-                       usb_autopm_get_interface_no_resume(serial->interface);
+       /* odd error handling due to pm counters */
+       if (!usb_autopm_get_interface(serial->interface))
+               sierra_send_setup(port);
+       else
+               usb_autopm_get_interface_no_resume(serial->interface);
 
-       }
-       mutex_unlock(&serial->disc_mutex);
        spin_lock_irq(&intfdata->susp_lock);
        portdata->opened = 0;
        if (--intfdata->open_ports == 0)
-- 
1.8.5.5

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