On Wed, Feb 09, 2005 at 11:49:42AM -0500, Alan Stern wrote:
> On Wed, 2 Feb 2005, Matthew Dharm wrote:
> 
> > On Wed, Feb 02, 2005 at 11:57:57AM -0500, Alan Stern wrote:
> > > Below is a patch for 2.6.11-rc2 that makes write-protect detection into a 
> > > module parameter.  Matt Dharm has considered doing this in the past; I 
> > > don't remember if he came to a final decision.
> > 
> > Is there a place where we can keep all that useful descriptive text?  I
> > really don't want to throw it away, just to be bombarded by "what's this
> > parameter for" questions...
> 
> Considering that only a small number of devices are unable to handle
> write-protect detection (I've seen reports for no more than two), maybe we
> should just make it into another unusual_devs flag?  That way there's
> nothing to configure either during a build or at runtime.

I think this is valid only if many people are setting
CONFIG_USB_STORAGE_RW_DETECT.  Do you think that's a valid assumption?  I'm
not sure...

> Originally we made it a config option because it caused trouble so often.  
> But now that we always request a 192-byte transfer, it almost never fails.  
> Changing it to a device flag would be the simplest approach.

If most people are already setting the option, then this is probably a good
idea.

Matt

-- 
Matthew Dharm                              Home: [EMAIL PROTECTED] 
Maintainer, Linux USB Mass Storage Driver

You suck Stef.
                                        -- Greg 
User Friendly, 11/29/97

Attachment: pgpANhlAYslI7.pgp
Description: PGP signature

Reply via email to