usb: usbtmc: Fix suspend/resume
authorGuido Kiener <guido@kiener-muenchen.de>
Wed, 12 Sep 2018 08:50:57 +0000 (10:50 +0200)
committerGreg Kroah-Hartman <gregkh@linuxfoundation.org>
Thu, 20 Sep 2018 11:04:02 +0000 (13:04 +0200)
Submitted urbs are not allowed when system is suspended.
Thus the submitted urb waiting at interrupt pipe is killed
during suspend callback and submitted again when system resumes.

Signed-off-by: Guido Kiener <guido.kiener@rohde-schwarz.com>
Reviewed-by: Steve Bayless <steve_bayless@keysight.com>
Signed-off-by: Greg Kroah-Hartman <gregkh@linuxfoundation.org>
drivers/usb/class/usbtmc.c

index dc6c04f..e4c80b4 100644 (file)
@@ -2306,7 +2306,9 @@ static void usbtmc_free_int(struct usbtmc_device_data *data)
                return;
        usb_kill_urb(data->iin_urb);
        kfree(data->iin_buffer);
+       data->iin_buffer = NULL;
        usb_free_urb(data->iin_urb);
+       data->iin_urb = NULL;
        kref_put(&data->kref, usbtmc_delete);
 }
 
@@ -2488,13 +2490,25 @@ static int usbtmc_suspend(struct usb_interface *intf, pm_message_t message)
                                       file_elem);
                usbtmc_draw_down(file_data);
        }
+
+       if (data->iin_ep_present && data->iin_urb)
+               usb_kill_urb(data->iin_urb);
+
        mutex_unlock(&data->io_mutex);
        return 0;
 }
 
 static int usbtmc_resume(struct usb_interface *intf)
 {
-       return 0;
+       struct usbtmc_device_data *data = usb_get_intfdata(intf);
+       int retcode = 0;
+
+       if (data->iin_ep_present && data->iin_urb)
+               retcode = usb_submit_urb(data->iin_urb, GFP_KERNEL);
+       if (retcode)
+               dev_err(&intf->dev, "Failed to submit iin_urb\n");
+
+       return retcode;
 }
 
 static int usbtmc_pre_reset(struct usb_interface *intf)