W dniu 23.11.2011 11:00, Andiry Xu pisze: > On 11/23/2011 05:39 PM, Julian Sikorski wrote: >> W dniu 18.11.2011 07:04, Andiry Xu pisze: >>> On 11/18/2011 07:14 AM, Sarah Sharp wrote: >>>> Hi Andiry, >>>> >>>> Is this necessary for all NEC uPD720200 chipsets, or just the one in >>>> Julian's board? I have one of those chipsets in my x220 laptop, so I >>>> can double check if you don't know. >>>> >>> >>> I'm not sure. I have a NEC uPD720200 external controller, but I can't >>> test the patch because the host fail to resume on my platform and >>> re-initialize during resume without the quirk. So it would be great if >>> you can double check. >>> >>> Thanks, >>> Andiry >>> >> What is the status of this? I would also like to ask in which timeframe >> could this patch be merged with? 3.2? 3.3? 3.1.x? >> > > Please be patient, patch merging needs some time, and maintainer has a > lot of things to take care of:) > > Thanks, > Andiry > Sorry about that. I am perfectly fine with applying this patch myself for a while, I am just curious about the merging process :) Regards, Julian >> >> >>>> >>>> On Thu, Nov 17, 2011 at 01:07:44PM +0800, Andiry Xu wrote: >>>>> Julian Sikorski reports NEC uPD720200 does not work stable after >>>>> suspend >>>>> and resume. Re-initialize the host in xhci_resume(). >>>>> >>>>> Reported-by: Julian Sikorski<belegdol@xxxxxxxxx> >>>>> Signed-off-by: Andiry Xu<andiry.xu@xxxxxxx> >>>>> Tested-by: Julian Sikorski<belegdol@xxxxxxxxx> >>>>> --- >>>>> drivers/usb/host/xhci-pci.c | 7 ++++++- >>>>> 1 files changed, 6 insertions(+), 1 deletions(-) >>>>> >>>>> diff --git a/drivers/usb/host/xhci-pci.c b/drivers/usb/host/xhci-pci.c >>>>> index 9f51f88..f0ef354 100644 >>>>> --- a/drivers/usb/host/xhci-pci.c >>>>> +++ b/drivers/usb/host/xhci-pci.c >>>>> @@ -32,6 +32,8 @@ >>>>> #define PCI_VENDOR_ID_ETRON 0x1b6f >>>>> #define PCI_DEVICE_ID_ASROCK_P67 0x7023 >>>>> >>>>> +#define PCI_DEVICE_ID_NEC_uPD720200 0x0194 >>>>> + >>>>> static const char hcd_name[] = "xhci_hcd"; >>>>> >>>>> /* called after powerup, by probe or system-pm "wakeup" */ >>>>> @@ -73,8 +75,11 @@ static void xhci_pci_quirks(struct device *dev, >>>>> struct xhci_hcd *xhci) >>>>> pdev->revision); >>>>> } >>>>> >>>>> - if (pdev->vendor == PCI_VENDOR_ID_NEC) >>>>> + if (pdev->vendor == PCI_VENDOR_ID_NEC) { >>>>> xhci->quirks |= XHCI_NEC_HOST; >>>>> + if (pdev->device == PCI_DEVICE_ID_NEC_uPD720200) >>>>> + xhci->quirks |= XHCI_RESET_ON_RESUME; >>>>> + } >>>>> >>>>> if (pdev->vendor == PCI_VENDOR_ID_AMD&& xhci->hci_version == >>>>> 0x96) >>>>> xhci->quirks |= XHCI_AMD_0x96_HOST; >>>>> -- >>>>> 1.7.4.1 >>>>> >>>>> >>>> >>> >>> >> >> > > -- 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