linux-scsi.vger.kernel.org archive mirror
 help / color / mirror / Atom feed
* [PATCH 1/2] xhci: Add broken-streams quirk for Fresco Logic FL1000G xhci controllers
@ 2014-12-05 10:11 Hans de Goede
  2014-12-05 10:11 ` [PATCH 2/2] uas: Add US_FL_NO_ATA_1X for Seagate devices with usb-id 0bc2:a013 Hans de Goede
  2014-12-05 11:15 ` [PATCH 1/2] xhci: Add broken-streams quirk for Fresco Logic FL1000G xhci controllers Sergei Shtylyov
  0 siblings, 2 replies; 3+ messages in thread
From: Hans de Goede @ 2014-12-05 10:11 UTC (permalink / raw)
  To: Mathias Nyman, Greg Kroah-Hartman
  Cc: Marcin Zajączkowski, linux-usb, linux-scsi, stable,
	Hans de Goede

Streams do not work reliabe on Fresco Logic FL1000G xhci controllers,
trying to use them results in errors like this:

21:37:33 kernel: xhci_hcd 0000:04:00.0: ERROR Transfer event for disabled endpoint or incorrect stream ring
21:37:33 kernel: xhci_hcd 0000:04:00.0: @00000000368b3570 9067b000 00000000 05000000 01078001
21:37:33 kernel: xhci_hcd 0000:04:00.0: ERROR Transfer event for disabled endpoint or incorrect stream ring
21:37:33 kernel: xhci_hcd 0000:04:00.0: @00000000368b3580 9067b400 00000000 05000000 01038001

As always I've ordered a pci-e addon card with a Fresco Logic controller for
myself to see if I can come up with a better fix then the big hammer, in
the mean time this will make uas devices work again (in usb-storage mode)
for FL1000G users.

Reported-by: Marcin Zajączkowski <mszpak@wp.pl>
Cc: stable@vger.kernel.org # 3.15
Signed-off-by: Hans de Goede <hdegoede@redhat.com>
---
 drivers/usb/host/xhci-pci.c | 2 ++
 1 file changed, 2 insertions(+)

diff --git a/drivers/usb/host/xhci-pci.c b/drivers/usb/host/xhci-pci.c
index 9a69b1f..e1071ce 100644
--- a/drivers/usb/host/xhci-pci.c
+++ b/drivers/usb/host/xhci-pci.c
@@ -82,6 +82,8 @@ static void xhci_pci_quirks(struct device *dev, struct xhci_hcd *xhci)
 				"must be suspended extra slowly",
 				pdev->revision);
 		}
+		if (pdev->device == PCI_DEVICE_ID_FRESCO_LOGIC_PDK)
+			xhci->quirks |= XHCI_BROKEN_STREAMS;
 		/* Fresco Logic confirms: all revisions of this chip do not
 		 * support MSI, even though some of them claim to in their PCI
 		 * capabilities.
-- 
2.1.0

^ permalink raw reply related	[flat|nested] 3+ messages in thread

* [PATCH 2/2] uas: Add US_FL_NO_ATA_1X for Seagate devices with usb-id 0bc2:a013
  2014-12-05 10:11 [PATCH 1/2] xhci: Add broken-streams quirk for Fresco Logic FL1000G xhci controllers Hans de Goede
@ 2014-12-05 10:11 ` Hans de Goede
  2014-12-05 11:15 ` [PATCH 1/2] xhci: Add broken-streams quirk for Fresco Logic FL1000G xhci controllers Sergei Shtylyov
  1 sibling, 0 replies; 3+ messages in thread
From: Hans de Goede @ 2014-12-05 10:11 UTC (permalink / raw)
  To: Mathias Nyman, Greg Kroah-Hartman
  Cc: Marcin Zajączkowski, linux-usb, linux-scsi, stable,
	Hans de Goede

This is yet another Seagate device which needs the US_FL_NO_ATA_1X quirk

Reported-by: Marcin Zajączkowski <mszpak@wp.pl>
Cc: stable@vger.kernel.org # 3.16
Signed-off-by: Hans de Goede <hdegoede@redhat.com>
---
 drivers/usb/storage/unusual_uas.h | 7 +++++++
 1 file changed, 7 insertions(+)

diff --git a/drivers/usb/storage/unusual_uas.h b/drivers/usb/storage/unusual_uas.h
index 18a283d..2918376 100644
--- a/drivers/usb/storage/unusual_uas.h
+++ b/drivers/usb/storage/unusual_uas.h
@@ -68,6 +68,13 @@ UNUSUAL_DEV(0x0bc2, 0xa003, 0x0000, 0x9999,
 		USB_SC_DEVICE, USB_PR_DEVICE, NULL,
 		US_FL_NO_ATA_1X),
 
+/* Reported-by: Marcin Zajączkowski <mszpak@wp.pl> */
+UNUSUAL_DEV(0x0bc2, 0xa013, 0x0000, 0x9999,
+		"Seagate",
+		"Backup Plus",
+		USB_SC_DEVICE, USB_PR_DEVICE, NULL,
+		US_FL_NO_ATA_1X),
+
 /* https://bbs.archlinux.org/viewtopic.php?id=183190 */
 UNUSUAL_DEV(0x0bc2, 0xab20, 0x0000, 0x9999,
 		"Seagate",
-- 
2.1.0

--
To unsubscribe from this list: send the line "unsubscribe linux-scsi" in
the body of a message to majordomo@vger.kernel.org
More majordomo info at  http://vger.kernel.org/majordomo-info.html

^ permalink raw reply related	[flat|nested] 3+ messages in thread

* Re: [PATCH 1/2] xhci: Add broken-streams quirk for Fresco Logic FL1000G xhci controllers
  2014-12-05 10:11 [PATCH 1/2] xhci: Add broken-streams quirk for Fresco Logic FL1000G xhci controllers Hans de Goede
  2014-12-05 10:11 ` [PATCH 2/2] uas: Add US_FL_NO_ATA_1X for Seagate devices with usb-id 0bc2:a013 Hans de Goede
@ 2014-12-05 11:15 ` Sergei Shtylyov
  1 sibling, 0 replies; 3+ messages in thread
From: Sergei Shtylyov @ 2014-12-05 11:15 UTC (permalink / raw)
  To: Hans de Goede, Mathias Nyman, Greg Kroah-Hartman
  Cc: Marcin Zajączkowski, linux-usb, linux-scsi, stable

Hello.

On 12/5/2014 1:11 PM, Hans de Goede wrote:

> Streams do not work reliabe on Fresco Logic FL1000G xhci controllers,

    s/reliabe/reliably/?

> trying to use them results in errors like this:

> 21:37:33 kernel: xhci_hcd 0000:04:00.0: ERROR Transfer event for disabled endpoint or incorrect stream ring
> 21:37:33 kernel: xhci_hcd 0000:04:00.0: @00000000368b3570 9067b000 00000000 05000000 01078001
> 21:37:33 kernel: xhci_hcd 0000:04:00.0: ERROR Transfer event for disabled endpoint or incorrect stream ring
> 21:37:33 kernel: xhci_hcd 0000:04:00.0: @00000000368b3580 9067b400 00000000 05000000 01038001

> As always I've ordered a pci-e addon card with a Fresco Logic controller for
> myself to see if I can come up with a better fix then the big hammer, in
> the mean time this will make uas devices work again (in usb-storage mode)
> for FL1000G users.

> Reported-by: Marcin Zajączkowski <mszpak@wp.pl>
> Cc: stable@vger.kernel.org # 3.15
> Signed-off-by: Hans de Goede <hdegoede@redhat.com>

WBR, Sergei

--
To unsubscribe from this list: send the line "unsubscribe linux-scsi" in
the body of a message to majordomo@vger.kernel.org
More majordomo info at  http://vger.kernel.org/majordomo-info.html

^ permalink raw reply	[flat|nested] 3+ messages in thread

end of thread, other threads:[~2014-12-05 11:15 UTC | newest]

Thread overview: 3+ messages (download: mbox.gz follow: Atom feed
-- links below jump to the message on this page --
2014-12-05 10:11 [PATCH 1/2] xhci: Add broken-streams quirk for Fresco Logic FL1000G xhci controllers Hans de Goede
2014-12-05 10:11 ` [PATCH 2/2] uas: Add US_FL_NO_ATA_1X for Seagate devices with usb-id 0bc2:a013 Hans de Goede
2014-12-05 11:15 ` [PATCH 1/2] xhci: Add broken-streams quirk for Fresco Logic FL1000G xhci controllers Sergei Shtylyov

This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox;
as well as URLs for NNTP newsgroup(s).