On Wed, Oct 23, 2013 at 02:45:39PM -0400, Alan Stern wrote: > On Wed, 23 Oct 2013, Felipe Balbi wrote: > > > Hi, > > > > On Wed, Oct 23, 2013 at 10:46:09AM -0400, Alan Stern wrote: > > > On Tue, 22 Oct 2013, Peter Chen wrote: > > > > > > > When the port goes to suspend or finishes resme, it needs to > > > > notify PHY, it is not a standard EHCI operation, so we add a > > > > quirk for it. > > > > > > Actually, this _should_ be a standard EHCI operation. But we have to > > > figure out a way to do it that will work on all platforms. > > > > > > Felipe, any ideas? > > > > isn't it enough to call usb_phy_set_suspend() at apropriate places ? > > > > This should work on all platforms if the PHY driver is implemented > > correctly. > > On Tue, 22 Oct 2013, Peter Chen wrote: > > > + /* > > + * notify the USB PHY, it is for global > > + * suspend case. > > + */ > > + usb_phy_notify_suspend(hcd->phy, > > + USB_SPEED_HIGH); > > Hmmm. This calls usb_phy_notify_suspend(), and later on, > usb_phy_notify_resume(). AFAICT, those routines don't exist. > Hi Alan and Felipe, It is at my another patchset: "Add power management support for MXS PHY" http://marc.info/?l=linux-usb&m=138242248913823&w=2 Since they are two things, one is from PHY, the other is for controller. I split them to two patchset. For these two notifications, please see: [Patch v2 07/14] usb: phy: add notify suspend and resume callback http://marc.info/?l=linux-usb&m=138242252713848&w=2 [Patch v2 08/14] usb: phy-mxs: Add implementation of nofity_suspend and notify_resume http://marc.info/?l=linux-usb&m=138242253313856&w=2 > Instead we have usb_phy_set_suspend(), as Felipe mentioned. It has a > different interface, because the second argument specifies whether we > are entering or leaving suspend, not the connection speed. > > Peter, you need to straighten this out. .set_suspend is different with .notify_suspend. .set_suspend is called when the PHY enters suspend. .notify_suspend is called when port enters suspends (setting portsc.suspendM, and stop sending SOF). After calling .notify_suspend, the PHY may still need to access, eg, set PHY wakeup setting, etc. The .notify_suspend is in ehci operation, .set_suspend can be at controller driver, since when to put the PHY enters suspend may platform specific. > > Also, there's the question of where are the appropriate places to make > the calls? After the root hub goes into suspend, I suppose. But when > is the right time during resume? > If we can take it as standard EHCI operation, we need to put it at below places: For .notify_suspend, we need to put it after set portsc.suspendM. - For selective suspend, at ehci_hub_control - For global suspend, at ehci_bus_suspend For .notify_resume, we need to put it after clear portsc.fpr, it is similar for .notify_suspend - For selective suspend, at ehci_hub_control - For global suspend, at ehci_bus_resume -- Best Regards, Peter Chen -- To unsubscribe from this list: send the line "unsubscribe linux-usb" in the body of a message to majordomo@xxxxxxxxxxxxxxx More majordomo info at http://vger.kernel.org/majordomo-info.html