Well, I haven't tested it with 2.5... I'm sure someone has, tho.

It is necessary to set the configuration to 1 on some devices.

I haven't seen a device that falls into this category with multiple
interfaces.

Matt

On Sun, Mar 30, 2003 at 02:53:59AM +0100, Oliver Neukum wrote:
> Hi,
> 
> is the following code from storage_probe() necessary:
> 
> #ifdef CONFIG_USB_STORAGE_SDDR09
>       if (protocol == US_PR_EUSB_SDDR09 || protocol == US_PR_DPCM_USB) {
>               /* set the configuration -- STALL is an acceptable response here */
>               result = usb_set_configuration(dev, 1);
> 
>               US_DEBUGP("Result from usb_set_configuration is %d\n", result);
>               if (result == -EPIPE) {
>                       US_DEBUGP("-- stall on control interface\n");
>               } else if (result != 0) {
>                       /* it's not a stall, but another error -- time to bail */
>                       US_DEBUGP("-- Unknown error.  Rejecting device\n");
>                       return -EIO;
>               }
>       }
> #endif
> 
> Can we assume that there are no further interfaces in these devices?
> Has this been tested with 2.5?
> 
>       Regards
>               Oliver

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

What, are you one of those Microsoft-bashing Linux freaks?
                                        -- Customer to Greg
User Friendly, 2/10/1999

Attachment: pgp00000.pgp
Description: PGP signature

Reply via email to