On Fri, 18 Jan 2013, Lan, Tianyu wrote: > HI Alan: > I just find Rafael's patch has resolved this issue. In this patch, enable runtime PM > right after executing subsystem/driver .resume_early() callbacks. When do resume(), > the device's runtime pm has been enabled. This patch now is already in the v3.8-rc4. > So this patchset will not cause problem with Rafael patch and I have tested. About the > port system suspend/resume() callback, Could I do this in the next step? Well, I guess you don't really need a resume callback now. Leaving the port in its old runtime PM state should be okay, because hub_activate will set the port power correctly. The important thing is that the hardware's actual power level should agree with the runtime PM status. However, you still need to enable async suspend for the port devices. The device_pm_wait_for_dev call isn't needed if the ports don't have a resume callback. Have you tested hibernation as well as system suspend? Alan Stern -- 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