All of lore.kernel.org
 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 an external index of several public inboxes,
see mirroring instructions on how to clone and mirror
all data and code used by this external index.