Re: [usb-storage] drivers/block/ub.c #6

From: Pat LaVarre
Date: Fri Aug 27 2004 - 17:42:59 EST


Pete Z:

> > Date: 30 Jul 2004 04:51:20 -0600
> > ...
> > I wish this could be somehow limited to flash keys,

How committed are we to the "Low Performance" part of the "Low
Performance USB Block driver" title for "config BLK_DEV_UB" found in
drivers/block/Kconfig?

Do we instead hope for ub.ko to simplify all "generic USB Mass Storage",
leaving only the vendor-specific work to usb-storage.ko?

I ask because I haven't yet learned how to reconcile two source files:

--- drivers/usb/storage/usb.c
...
static struct usb_device_id storage_usb_ids [] = {
...
#if !defined(CONFIG_BLK_DEV_UB) && !defined(CONFIG_BLK_DEV_UB_MODULE)
{ USB_INTERFACE_INFO(USB_CLASS_MASS_STORAGE, US_SC_SCSI, US_PR_BULK) },
#endif
...
--- C:/Windows/inf/usbstor.inf:
...
[Generic]
%GenericBulkOnly.DeviceDesc%=USBSTOR_BULK, USB\Class_08&SubClass_02&Prot_50
%GenericBulkOnly.DeviceDesc%=USBSTOR_BULK, USB\Class_08&SubClass_05&Prot_50
%GenericBulkOnly.DeviceDesc%=USBSTOR_BULK, USB\Class_08&SubClass_06&Prot_50
...
---

Looks to me like we have ub.ko taking over just x 08 06 50, rather than
all of the x 08 (06|05|02) 50 = bInterfaceClass ...SubClass ...Protocol
considered generic by MSFT?

http://www.microsoft.com/whdc/device/storage/usbfaq.mspx
agrees Flash should be x 08 06 50 but gives no clear guidance to the
rest of us.

I remember we invented x 08 06 50 to be the one tuple to rule them all,
to move the determination of PDT (peripheral device type) etc. back into
op x12 "INQUIRY" where it belongs, ...

Of course the world may yet contain advocates of connecting HDD/FDD as
bInterfaceSubClass = x05 "SFF 8070" = Compaq LS-120 or connecting DVD/CD
as x02 "SFF 8020" = read-only CD.

Pat LaVarre
http://linux-pel.blog-city.com/read/790489.htm

-
To unsubscribe from this list: send the line "unsubscribe linux-kernel" in
the body of a message to majordomo@xxxxxxxxxxxxxxx
More majordomo info at http://vger.kernel.org/majordomo-info.html
Please read the FAQ at http://www.tux.org/lkml/