usb: chipidea: otg: use usb autosuspend to suspend bus for HNP
authorLi Jun <jun.li@freescale.com>
Tue, 15 Dec 2015 09:51:29 +0000 (17:51 +0800)
committerPeter Chen <peter.chen@freescale.com>
Thu, 24 Dec 2015 06:18:07 +0000 (14:18 +0800)
Directly manipulate the controller regsiter to suspend the usb bus
for HNP is not the proper way, this should be done through the usbcore
by usb autosuspend. So to start HNP, autosuspend support should be
added for OTG devices interface driver if it's not enabled.

Signed-off-by: Li Jun <jun.li@freescale.com>
Signed-off-by: Peter Chen <peter.chen@freescale.com>
drivers/usb/chipidea/otg_fsm.c

index 00ab59d45da1bf6708b3d77e3cfcfe31ac4e2304..ba90dc66703de3631bf4e6b533944c64bafeb257 100644 (file)
@@ -485,20 +485,30 @@ static void ci_otg_loc_conn(struct otg_fsm *fsm, int on)
 
 /*
  * Generate SOF by host.
- * This is controlled through suspend/resume the port.
  * In host mode, controller will automatically send SOF.
  * Suspend will block the data on the port.
+ *
+ * This is controlled through usbcore by usb autosuspend,
+ * so the usb device class driver need support autosuspend,
+ * otherwise the bus suspend will not happen.
  */
 static void ci_otg_loc_sof(struct otg_fsm *fsm, int on)
 {
-       struct ci_hdrc  *ci = container_of(fsm, struct ci_hdrc, fsm);
+       struct usb_device *udev;
 
-       if (on)
-               hw_write(ci, OP_PORTSC, PORTSC_W1C_BITS | PORTSC_FPR,
-                                                       PORTSC_FPR);
-       else
-               hw_write(ci, OP_PORTSC, PORTSC_W1C_BITS | PORTSC_SUSP,
-                                                       PORTSC_SUSP);
+       if (!fsm->otg->host)
+               return;
+
+       udev = usb_hub_find_child(fsm->otg->host->root_hub, 1);
+       if (!udev)
+               return;
+
+       if (on) {
+               usb_disable_autosuspend(udev);
+       } else {
+               pm_runtime_set_autosuspend_delay(&udev->dev, 0);
+               usb_enable_autosuspend(udev);
+       }
 }
 
 /*