On Thu, Aug 08, 2013 at 03:38:53AM +0000, Wang, Yu Y wrote: > > Hi Felipe, > > > > This patch brings up an interesting point: will a dwc3 PCI host use the xhci > > platform driver instead of the xhci PCI driver? > > > > If so, that seems less than ideal. Won't it not use the standard xHCI PCI > > suspend and resume functions if it's registered as a platform device? Also, it > > seems registering it that way means XHCI_BROKEN_MSI is set unconditionally. > > That leads to the xhci driver not enabling MSI or MSI-X for the PCI host, which > > will impact performance. > > > > [Yu:] The upstream dwc3 driver haven't enable power management for the host > mode until now. Actually, this is another big changes. In Intel intern tree, I have to > wrote another separate file to re-implemented the PM callback to support platform > device design. Maybe we need consider to add one new file(hcd-plat.c) which is familiar > with hcd-pci.c? Felipe, what would be the best approach to add PM support for xhci-plat devices? Yu, I will send you an updated patch to test shortly. Sarah Sharp -- 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