[PATCH] [resend] unusual_devs.h: Fix capacity for SL11R-IDE 2.6c

[PATCH] [resend] unusual_devs.h: Fix capacity for SL11R-IDE 2.6c

[Ondrej Zary]

SL11R-IDE 2.6c (at least) reports wrong capacity (one sector more).
Reading that last sector causes the device not to work anymore (and looks
like HAL or something does that automatically after plugging in):
sd 5:0:0:0: [sdc] Device not ready
sd 5:0:0:0: [sdc] Result: hostbyte=0x00 driverbyte=0x08
sd 5:0:0:0: [sdc] Sense Key : 0x2 [current]
sd 5:0:0:0: [sdc] ASC=0x0 ASCQ=0x0
sd 5:0:0:0: [sdc] CDB: cdb[0]=0x28: 28 00 04 a8 b5 70 00 00 01 00

Add unusual_devs entry to fix the capacity.

Signed-off-by: Ondrej Zary <linux@rainbow-software.org>

--- linux-2.6.32-orig/drivers/usb/storage/unusual_devs.h	2009-12-03 04:51:21.000000000 +0100
+++ linux-2.6.32/drivers/usb/storage/unusual_devs.h	2010-02-21 20:23:12.000000000 +0100
@@ -374,6 +374,12 @@ UNUSUAL_DEV(  0x04ce, 0x0002, 0x0074, 0x
 		US_SC_DEVICE, US_PR_DEVICE, NULL,
 		US_FL_FIX_INQUIRY),
 
+UNUSUAL_DEV(  0x04ce, 0x0002, 0x026c, 0x026c,
+		"ScanLogic",
+		"SL11R-IDE",
+		US_SC_DEVICE, US_PR_DEVICE, NULL,
+		US_FL_FIX_CAPACITY),
+
 /* Reported by Kriston Fincher <kriston@airmail.net>
  * Patch submitted by Sean Millichamp <sean@bruenor.org>
  * This is to support the Panasonic PalmCam PV-SD4090

-- 
Ondrej Zary

Re: [usb-storage] [PATCH] [resend] unusual_devs.h: Fix capacity for SL11R-IDE 2.6c

[Matthew Dharm]

[-- Attachment #1: Type: text/plain, Size: 972 bytes --]

On Wed, Feb 24, 2010 at 11:41:08AM +0100, Ondrej Zary wrote:
> SL11R-IDE 2.6c (at least) reports wrong capacity (one sector more).
> Reading that last sector causes the device not to work anymore (and looks
> like HAL or something does that automatically after plugging in):
> sd 5:0:0:0: [sdc] Device not ready
> sd 5:0:0:0: [sdc] Result: hostbyte=0x00 driverbyte=0x08
> sd 5:0:0:0: [sdc] Sense Key : 0x2 [current]
> sd 5:0:0:0: [sdc] ASC=0x0 ASCQ=0x0
> sd 5:0:0:0: [sdc] CDB: cdb[0]=0x28: 28 00 04 a8 b5 70 00 00 01 00
> 
> Add unusual_devs entry to fix the capacity.

Why not just adjust the version range of the entry above where you added a
new one?

If you just expand the upper version of the previous entry, it should cover
both devices.

Matt

-- 
Matthew Dharm                              Home: mdharm-usb@one-eyed-alien.net 
Maintainer, Linux USB Mass Storage Driver

Oh BAY-bee.
					-- Dust Puppy to Greg
User Friendly, 12/13/1997

[-- Attachment #2: Type: application/pgp-signature, Size: 189 bytes --]

Re: [usb-storage] [PATCH] [resend] unusual_devs.h: Fix capacity for SL11R-IDE 2.6c

[Ondrej Zary]

On Wednesday 24 February 2010 20:29:59 Matthew Dharm wrote:
> On Wed, Feb 24, 2010 at 11:41:08AM +0100, Ondrej Zary wrote:
> > SL11R-IDE 2.6c (at least) reports wrong capacity (one sector more).
> > Reading that last sector causes the device not to work anymore (and looks
> > like HAL or something does that automatically after plugging in):
> > sd 5:0:0:0: [sdc] Device not ready
> > sd 5:0:0:0: [sdc] Result: hostbyte=0x00 driverbyte=0x08
> > sd 5:0:0:0: [sdc] Sense Key : 0x2 [current]
> > sd 5:0:0:0: [sdc] ASC=0x0 ASCQ=0x0
> > sd 5:0:0:0: [sdc] CDB: cdb[0]=0x28: 28 00 04 a8 b5 70 00 00 01 00
> >
> > Add unusual_devs entry to fix the capacity.
>
> Why not just adjust the version range of the entry above where you added a
> new one?

Because that entry fixes IDENTIFY. My version does not seem to have any 
problems with IDENTIFY. Or maybe it has but kernel can now handle it 
automatically:
usb 1-1: new full speed USB device using uhci_hcd and address 4
usb 1-1: configuration #1 chosen from 1 choice
scsi5 : SCSI emulation for USB Mass Storage devices
usb-storage: device found at 4
usb-storage: waiting for device to settle before scanning
scsi scan: INQUIRY result too short (5), using 36
scsi 5:0:0:0: Direct-Access     WDC WD40 0BB-00AUA1       18.2 PQ: 0 ANSI: 1
sd 5:0:0:0: Attached scsi generic sg3 type 0


> If you just expand the upper version of the previous entry, it should cover
> both devices.
>
> Matt



-- 
Ondrej Zary

Re: [usb-storage] [PATCH] [resend] unusual_devs.h: Fix capacity for SL11R-IDE 2.6c

[Matthew Dharm]

[-- Attachment #1: Type: text/plain, Size: 2242 bytes --]

Phil D. --

I recommend circling around with pwaechtler@loewe-komp.de to see if we
can't condense these into a single entry.

Matt

On Wed, Feb 24, 2010 at 10:22:18PM +0100, Ondrej Zary wrote:
> On Wednesday 24 February 2010 20:29:59 Matthew Dharm wrote:
> > On Wed, Feb 24, 2010 at 11:41:08AM +0100, Ondrej Zary wrote:
> > > SL11R-IDE 2.6c (at least) reports wrong capacity (one sector more).
> > > Reading that last sector causes the device not to work anymore (and looks
> > > like HAL or something does that automatically after plugging in):
> > > sd 5:0:0:0: [sdc] Device not ready
> > > sd 5:0:0:0: [sdc] Result: hostbyte=0x00 driverbyte=0x08
> > > sd 5:0:0:0: [sdc] Sense Key : 0x2 [current]
> > > sd 5:0:0:0: [sdc] ASC=0x0 ASCQ=0x0
> > > sd 5:0:0:0: [sdc] CDB: cdb[0]=0x28: 28 00 04 a8 b5 70 00 00 01 00
> > >
> > > Add unusual_devs entry to fix the capacity.
> >
> > Why not just adjust the version range of the entry above where you added a
> > new one?
> 
> Because that entry fixes IDENTIFY. My version does not seem to have any 
> problems with IDENTIFY. Or maybe it has but kernel can now handle it 
> automatically:
> usb 1-1: new full speed USB device using uhci_hcd and address 4
> usb 1-1: configuration #1 chosen from 1 choice
> scsi5 : SCSI emulation for USB Mass Storage devices
> usb-storage: device found at 4
> usb-storage: waiting for device to settle before scanning
> scsi scan: INQUIRY result too short (5), using 36
> scsi 5:0:0:0: Direct-Access     WDC WD40 0BB-00AUA1       18.2 PQ: 0 ANSI: 1
> sd 5:0:0:0: Attached scsi generic sg3 type 0
> 
> 
> > If you just expand the upper version of the previous entry, it should cover
> > both devices.
> >
> > Matt
> 
> 
> 
> -- 
> Ondrej Zary
> _______________________________________________
> Usb-storage mailing list
> Usb-storage@lists.one-eyed-alien.net
> https://lists.one-eyed-alien.net/mailman/listinfo/usb-storage

-- 
Matthew Dharm                              Home: mdharm-usb@one-eyed-alien.net 
Maintainer, Linux USB Mass Storage Driver

E:  You run this ship with Windows?!  YOU IDIOT!
L:  Give me a break, it came bundled with the computer!
					-- ESR and Lan Solaris
User Friendly, 12/8/1998

[-- Attachment #2: Type: application/pgp-signature, Size: 189 bytes --]

Re: [PATCH] [resend] unusual_devs.h: Fix capacity for SL11R-IDE 2.6c

[Phil Dibowitz]

[-- Attachment #1: Type: text/plain, Size: 962 bytes --]

On 02/24/2010 11:41 AM, Ondrej Zary wrote:
> SL11R-IDE 2.6c (at least) reports wrong capacity (one sector more).
> Reading that last sector causes the device not to work anymore (and looks
> like HAL or something does that automatically after plugging in):
> sd 5:0:0:0: [sdc] Device not ready
> sd 5:0:0:0: [sdc] Result: hostbyte=0x00 driverbyte=0x08
> sd 5:0:0:0: [sdc] Sense Key : 0x2 [current]
> sd 5:0:0:0: [sdc] ASC=0x0 ASCQ=0x0
> sd 5:0:0:0: [sdc] CDB: cdb[0]=0x28: 28 00 04 a8 b5 70 00 00 01 00

Ondrej,

I got both of your emails, but I was in LA for the SCALE conference. Give me
a day or two to catch up.

-- 
Phil Dibowitz                             phil@ipom.com
Open Source software and tech docs        Insanity Palace of Metallica
http://www.phildev.net/                   http://www.ipom.com/

"Be who you are and say what you feel, because those who mind don't matter
 and those who matter don't mind."
 - Dr. Seuss



[-- Attachment #2: OpenPGP digital signature --]
[-- Type: application/pgp-signature, Size: 261 bytes --]

Re: [usb-storage] [PATCH] [resend] unusual_devs.h: Fix capacity for SL11R-IDE 2.6c

[Phil Dibowitz]

[-- Attachment #1: Type: text/plain, Size: 828 bytes --]

On 02/24/2010 11:47 PM, Matthew Dharm wrote:
> Phil D. --
> 
> I recommend circling around with pwaechtler@loewe-komp.de to see if we
> can't condense these into a single entry.

I'm with Ondrej here, I suspect that entry is completely unneeded now. We've
gotten pretty good at handling bad INQUIRYs.

So unless you object, I'm going to allow Ondrej's patch, and then I'll fire
up a patch to remove the other version and fire it off to Peter W. to see if
his device works without it.

-- 
Phil Dibowitz                             phil@ipom.com
Open Source software and tech docs        Insanity Palace of Metallica
http://www.phildev.net/                   http://www.ipom.com/

"Be who you are and say what you feel, because those who mind don't matter
 and those who matter don't mind."
 - Dr. Seuss



[-- Attachment #2: OpenPGP digital signature --]
[-- Type: application/pgp-signature, Size: 261 bytes --]

Re: [usb-storage] [PATCH] [resend] unusual_devs.h: Fix capacity for SL11R-IDE 2.6c

[Matthew Dharm]

[-- Attachment #1: Type: text/plain, Size: 908 bytes --]

On Sat, Feb 27, 2010 at 08:11:53PM +0100, Phil Dibowitz wrote:
> On 02/24/2010 11:47 PM, Matthew Dharm wrote:
> > Phil D. --
> > 
> > I recommend circling around with pwaechtler@loewe-komp.de to see if we
> > can't condense these into a single entry.
> 
> I'm with Ondrej here, I suspect that entry is completely unneeded now. We've
> gotten pretty good at handling bad INQUIRYs.
> 
> So unless you object, I'm going to allow Ondrej's patch, and then I'll fire
> up a patch to remove the other version and fire it off to Peter W. to see if
> his device works without it.

Seems like a reasonable plan.

Matt


-- 
Matthew Dharm                              Home: mdharm-usb@one-eyed-alien.net 
Maintainer, Linux USB Mass Storage Driver

S:  Another stupid question?
G:  There's no such thing as a stupid question, only stupid people.
					-- Stef and Greg
User Friendly, 7/15/1998

[-- Attachment #2: Type: application/pgp-signature, Size: 189 bytes --]