Return-Path: Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S1756585Ab3GVHXX (ORCPT ); Mon, 22 Jul 2013 03:23:23 -0400 Received: from bear.ext.ti.com ([192.94.94.41]:38509 "EHLO bear.ext.ti.com" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S1755752Ab3GVHXV (ORCPT ); Mon, 22 Jul 2013 03:23:21 -0400 Message-ID: <51ECDDE3.1040209@ti.com> Date: Mon, 22 Jul 2013 12:53:15 +0530 From: George Cherian User-Agent: Mozilla/5.0 (Windows NT 6.1; WOW64; rv:17.0) Gecko/20130620 Thunderbird/17.0.7 MIME-Version: 1.0 To: Shuduo Sang CC: Sarah Sharp , , , , Subject: Re: [PATCH v2] usb: host: xhci: Enable XHCI_SPURIOUS_SUCCESS for all controllers with xhci 1.0 References: <1372656552-18824-1-git-send-email-george.cherian@ti.com> <20130702032712.GD6262@xanatos> In-Reply-To: Content-Type: text/plain; charset="ISO-8859-1"; format=flowed Content-Transfer-Encoding: 7bit Sender: linux-kernel-owner@vger.kernel.org List-ID: X-Mailing-List: linux-kernel@vger.kernel.org Content-Length: 3892 Lines: 105 Hi Shuduo, On 7/22/2013 12:35 PM, Shuduo Sang wrote: > Hi Sarah and George, > > I met this issue at my side too. Per lspci output and HCIVERSION, I > see the host is Lynx > Point xhci 1.0 host. > > I notice George's patch has not been merged in kernel 3.11-rc1. Just to make sure, without the patch are you getting the following error. "ERROR Transfer event TRB DMA ptr not part of current TD" > Then I manually applied George's patch against 3.10.0 but the issue > still happen when I use > camera to capture picture. Can you explain what is the exact issue you face after applying the patch? Are you still getting "ERROR Transfer event TRB DMA ptr not part of current TD" > So I doubt some corner test has not been > covered by patch. > Pls let me know what I can do if you need more information. > > Thanks, > Shuduo > > On Tue, Jul 2, 2013 at 11:27 AM, Sarah Sharp > wrote: >> Thanks George, this looks fine. I will munge the description a bit when >> I commit it, and mark it for stable as well. >> >> Unfortunately, due to the timing of the merge window, this patch will >> have to wait for 2-3 weeks until 3.11-rc1 is out. >> >> Sarah Sharp >> >> On Mon, Jul 01, 2013 at 10:59:12AM +0530, George Cherian wrote: >>> Xhci controllers with hci_version > 0.96 gives spurious success >>> events on short packet completion. During webcam capture the >>> "ERROR Transfer event TRB DMA ptr not part of current TD" was observed. >>> The same application works fine with synopsis controllers hci_version 0.96. >>> The same Issue is seen with Intel Pantherpoint xhci controller. So enabling >>> this quirk in xhci_gen_setup if controller verion is greater than 0.96. >>> For xhci-pci move the quirk to much generic place xhci_gen_setup. >>> >>> Signed-off-by: George Cherian >>> --- >>> drivers/usb/host/xhci-pci.c | 1 - >>> drivers/usb/host/xhci.c | 7 +++++++ >>> 2 files changed, 7 insertions(+), 1 deletion(-) >>> >>> diff --git a/drivers/usb/host/xhci-pci.c b/drivers/usb/host/xhci-pci.c >>> index cc24e39..f00cb20 100644 >>> --- a/drivers/usb/host/xhci-pci.c >>> +++ b/drivers/usb/host/xhci-pci.c >>> @@ -93,7 +93,6 @@ static void xhci_pci_quirks(struct device *dev, struct xhci_hcd *xhci) >>> } >>> if (pdev->vendor == PCI_VENDOR_ID_INTEL && >>> pdev->device == PCI_DEVICE_ID_INTEL_PANTHERPOINT_XHCI) { >>> - xhci->quirks |= XHCI_SPURIOUS_SUCCESS; >>> xhci->quirks |= XHCI_EP_LIMIT_QUIRK; >>> xhci->limit_active_eps = 64; >>> xhci->quirks |= XHCI_SW_BW_CHECKING; >>> diff --git a/drivers/usb/host/xhci.c b/drivers/usb/host/xhci.c >>> index d8f640b..0f7be59 100644 >>> --- a/drivers/usb/host/xhci.c >>> +++ b/drivers/usb/host/xhci.c >>> @@ -4697,6 +4697,13 @@ int xhci_gen_setup(struct usb_hcd *hcd, xhci_get_quirks_t get_quirks) >>> >>> get_quirks(dev, xhci); >>> >>> + /* In xhci controllers which follow xhci 1.0 spec gives a spurious >>> + * success event after a short transfer. This quirk will ignore such >>> + * spurious event. >>> + */ >>> + if (xhci->hci_version > 0x96) >>> + xhci->quirks |= XHCI_SPURIOUS_SUCCESS; >>> + >>> /* Make sure the HC is halted. */ >>> retval = xhci_halt(xhci); >>> if (retval) >>> -- >>> 1.8.1.4 >>> >> -- >> To unsubscribe from this list: send the line "unsubscribe linux-kernel" in >> the body of a message to majordomo@vger.kernel.org >> More majordomo info at http://vger.kernel.org/majordomo-info.html >> Please read the FAQ at http://www.tux.org/lkml/ -- -George -- To unsubscribe from this list: send the line "unsubscribe linux-kernel" in the body of a message to majordomo@vger.kernel.org More majordomo info at http://vger.kernel.org/majordomo-info.html Please read the FAQ at http://www.tux.org/lkml/