On Sat, 2003-06-21 at 23:24, Matthew Dharm wrote: > On Sat, Jun 21, 2003 at 09:54:46PM -0500, James Bottomley wrote: > > } > > - n = buffer[3] + 4; > > + n = rc; > > cd->cdi.speed = ((buffer[n + 8] << 8) + buffer[n + 9]) / 176; > > This bit isn't right. n is supposed to point to the start of the page > data, not the page header. The header is a different size if the command > is 6-byte or 10-byte.
Yes it is. That's why I eliminated the dbd bit. Your patch was calculating the offsets past the dbd headers. However, if you tell the mode sense not to send any dbd headers (which are pointless, because the routine isn't interested in them), then you don't have to skip past them. and the mode data begins just past the mode header. James ------------------------------------------------------- This SF.Net email is sponsored by: INetU Attention Web Developers & Consultants: Become An INetU Hosting Partner. Refer Dedicated Servers. We Manage Them. You Get 10% Monthly Commission! INetU Dedicated Managed Hosting http://www.inetu.net/partner/index.php _______________________________________________ [EMAIL PROTECTED] To unsubscribe, use the last form field at: https://lists.sourceforge.net/lists/listinfo/linux-usb-devel
