Greg,

The following patch looks good.

Signed-off-by: Phil Dibowitz <[EMAIL PROTECTED]>

----- Forwarded message from Jaco Kroon <[EMAIL PROTECTED]> -----

Date: Wed, 15 Nov 2006 10:33:39 +0200
From: Jaco Kroon <[EMAIL PROTECTED]>
To: Phil Dibowitz <[EMAIL PROTECTED]>
Cc: Jaco Kroon <[EMAIL PROTECTED]>, linux-usb-devel@lists.sourceforge.net,
        [EMAIL PROTECTED], [EMAIL PROTECTED],
        Dave Wells <[EMAIL PROTECTED]>,
        USB Storage list <[EMAIL PROTECTED]>
Subject: [PATCH] Add Digitech USB-Storage to unusual_devs.h (attempt 2)

The mass storage device from Digitech designed for Flash Cards, as found
on (for example) the GNX4 device has issues with residue, similar to the
bug report at http://kerneltrap.org/node/6297.  This patch adds the
faulty storage device to unusual_devs.h, this not only reduces the noise
in dmesg but also increases the transfer speeds by a factor of 7x for me
(89kB/s -> 637kB/s).

T:  Bus=02 Lev=02 Prnt=02 Port=01 Cnt=02 Dev#=  4 Spd=12  MxCh= 0
D:  Ver= 1.10 Cls=00(>ifc ) Sub=00 Prot=00 MxPS=64 #Cfgs=  1
P:  Vendor=1210 ProdID=0003 Rev= 1.00
S:  Manufacturer=DigiTech HMG
S:  Product=DigiTech Mass Storage
C:* #Ifs= 1 Cfg#= 1 Atr=c0 MxPwr=  0mA
I:  If#= 0 Alt= 0 #EPs= 2 Cls=08(stor.) Sub=06 Prot=50
Driver=usb-storage
E:  Ad=02(O) Atr=02(Bulk) MxPS=  64 Ivl=0ms
E:  Ad=82(I) Atr=02(Bulk) MxPS=  64 Ivl=0ms

Signed-off-by: Jaco Kroon <[EMAIL PROTECTED]>


--- linux-2.6.18.1.orig/drivers/usb/storage/unusual_devs.h      2006-10-14 
05:34:03.000000000 +0200
+++ linux-2.6.18.1/drivers/usb/storage/unusual_devs.h   2006-11-14 
23:22:32.000000000 +0200
@@ -1271,6 +1271,16 @@
                US_SC_DEVICE, US_PR_DEVICE, usb_stor_ucr61s2b_init,
                0 ),
 
+/* Reported by Jaco Kroon <[EMAIL PROTECTED]>
+ * The usb-storage module found on the Digitech GNX4 (and supposedly other
+ * devices) misbehaves and causes a bunch of invalid I/O errors.
+ */
+UNUSUAL_DEV(  0x1210, 0x0003, 0x0100, 0x0100,
+               "Digitech HMG",
+               "DigiTech Mass Storage",
+               US_SC_DEVICE, US_PR_DEVICE, NULL,
+               US_FL_IGNORE_RESIDUE ),
+
 /* Reported by Vilius Bilinkevicius <vilisas AT xxx DOT lt) */
 UNUSUAL_DEV(  0x132b, 0x000b, 0x0001, 0x0001,
                "Minolta",

----- End forwarded message -----

-- 
Phil Dibowitz                             [EMAIL PROTECTED]
Freeware and Technical Pages              Insanity Palace of Metallica
http://www.phildev.net/                   http://www.ipom.com/

"They that can give up essential liberty to obtain a little temporary
safety deserve neither liberty nor safety."
 - Benjamin Franklin, 1759


-------------------------------------------------------------------------
Take Surveys. Earn Cash. Influence the Future of IT
Join SourceForge.net's Techsay panel and you'll get the chance to share your
opinions on IT & business topics through brief surveys - and earn cash
http://www.techsay.com/default.php?page=join.php&p=sourceforge&CID=DEVDEV
_______________________________________________
linux-usb-devel@lists.sourceforge.net
To unsubscribe, use the last form field at:
https://lists.sourceforge.net/lists/listinfo/linux-usb-devel

Reply via email to