Subject: kern/23875: TrekStor MusicStick needs umass quirks
To: None <gnats-bugs@gnats.netbsd.org>
From: None <mlelstv@serpens.de>
List: netbsd-bugs
Date: 12/24/2003 21:51:33
>Number:         23875
>Category:       kern
>Synopsis:       TrekStor MusicStick requires umass quirks
>Confidential:   no
>Severity:       non-critical
>Priority:       medium
>Responsible:    kern-bug-people
>State:          open
>Class:          change-request
>Submitter-Id:   net
>Arrival-Date:   Wed Dec 24 20:52:00 UTC 2003
>Closed-Date:
>Last-Modified:
>Originator:     Michael van Elst
>Release:        NetBSD 1.6.2_RC3
>Organization:
-- 
                                Michael van Elst
Internet: mlelstv@serpens.de
                                "A potential Snark may lurk in every tree."
>Environment:
	
	
System: NetBSD pepew 1.6.2_RC3 NetBSD 1.6.2_RC3 (PEPEW) #49: Wed Dec 24 18:33:00 MET 2003 src@pepew:/amd/fud/d/0/src/sys/arch/i386/compile/PEPEW i386
Architecture: i386
Machine: i386
>Description:
The TrekStor MusicStick is a USB MP3 player with a umass interface using
the bulk-only transfer method. The UBS driver identifies it as:

Dec 24 12:51:57 pussyfoot /netbsd: umass0: TrekStor, Inc. TrekStor MusicStick, r
ev 1.10/0.01, addr 2
Dec 24 12:51:57 pussyfoot /netbsd: umass0: using SCSI over Bulk-Only

When the device receives a command that it does not understand it
seems to fail on all subsequent data transfers showing:

Dec 24 12:52:29 pussyfoot /netbsd: umass0: BBB reset failed, IOERROR
Dec 24 12:52:29 pussyfoot /netbsd: umass0: BBB bulk-in clear stall failed, IOERR
OR
Dec 24 12:52:29 pussyfoot /netbsd: umass0: BBB bulk-out clear stall failed, IOER
ROR
Dec 24 12:52:29 pussyfoot /netbsd: sd0(umass0:0:1:0): generic HBA error

>How-To-Repeat:
Plug in a TrekStor and try to access it, e.g with 'disklabel sd0'.

>Fix:
I have added NODOORLOCK and NOSYNCCACHE quirks to avoid the corresponding
commands.

--- umass_quirks.c      2002/04/22 12:48:40     1.13
+++ umass_quirks.c      2003/12/24 20:49:12
@@ -254,6 +254,13 @@
          UMATCH_VENDOR_PRODUCT,
          NULL, NULL
        },
+       { { USB_VENDOR_SIGMATEL, USB_PRODUCT_SIGMATEL_MUSICSTICK },
+         UMASS_WPROTO_UNSPEC, UMASS_CPROTO_UNSPEC,
+         0,
+         PQUIRK_NODOORLOCK | PQUIRK_NOSYNCCACHE,
+         UMATCH_VENDOR_PRODUCT,
+         NULL, NULL
+       },
 };

--- usbdevs     2003/01/26 09:14:25     1.250.2.2
+++ usbdevs     2003/12/24 20:50:24
@@ -1127,6 +1127,7 @@
 
 /* Sigmatel products */
 product SIGMATEL IRDA          0x4200  IrDA
+product SIGMATEL MUSICSTICK     0x8134  TrekStor MusicStick
 
 /* SIIG products */
 product SIIG DIGIFILMREADER    0x0004  DigiFilm-Combo Reader


It is possible that some other fixes from -current in the scsipi
layer are required as well.

>Release-Note:
>Audit-Trail:
>Unformatted: