[PATCH 1/2 v3] XHCI: Activating XHCI_TRUST_TX_LENGTH quirk for Renesas uPD720201

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

 



Activating XHCI_TRUST_TX_LENGTH on Renesas uPD720201 USB 3.0 hosts to get rid of
the following warnings when using webcams on this type of host:

xhci_hcd 0000:06:00.0: WARN Successful completion on short TX: needs
	XHCI_TRUST_TX_LENGTH quirk?

Signed-off-by: Rodrigo Severo <rodrigo@xxxxxxxxxxxxxxxxxxx>
---
Changes on v2:
	- fixed syntax error identified by Sergei Shtylyov.

Changes on v3:
	- separated the quirk definition for Renesas uPD720201 and the moving of the 
		quirk definitions for Renesas hosts on two diferent patchs as per Fabio 
		Estevam suggestion.
	- included these change explanations and version identification as Fabio 
		Estevam suggested.

 drivers/usb/host/xhci-pci.c | 4 +++-
 1 file changed, 3 insertions(+), 1 deletion(-)

diff --git a/drivers/usb/host/xhci-pci.c b/drivers/usb/host/xhci-pci.c
index b02641b..a094afe 100644
--- a/drivers/usb/host/xhci-pci.c
+++ b/drivers/usb/host/xhci-pci.c
@@ -43,6 +43,7 @@
 
 #define PCI_DEVICE_ID_ASMEDIA_ASM1042A	0x1142
 
+#define PCI_DEVICE_ID_RENESAS_UPD720201	0x0014
 #define PCI_DEVICE_ID_RENESAS_UPD720202	0x0015
 
 static const char hcd_name[] = "xhci_hcd";
@@ -178,7 +179,8 @@ static void xhci_pci_quirks(struct device *dev, struct xhci_hcd *xhci)
 				"QUIRK: Resetting on resume");
 
 	if (pdev->vendor == PCI_VENDOR_ID_RENESAS &&
-			pdev->device == PCI_DEVICE_ID_RENESAS_UPD720202)
+			(pdev->device == PCI_DEVICE_ID_RENESAS_UPD720201 ||
+			pdev->device == PCI_DEVICE_ID_RENESAS_UPD720202))
 		xhci->quirks |= XHCI_TRUST_TX_LENGTH;
 }
 
-- 
2.1.0

--
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




[Index of Archives]     [Linux Media]     [Linux Input]     [Linux Audio Users]     [Yosemite News]     [Linux Kernel]     [Linux SCSI]     [Old Linux USB Devel Archive]

  Powered by Linux