Re: [linux-usb-devel] Re: further proposed small API change

2003-01-16 Thread Brad Hards
-BEGIN PGP SIGNED MESSAGE- Hash: SHA1 On Thu, 16 Jan 2003 11:18, Oliver Neukum wrote: > No. Does that mean you are volunteering to get me a USB2.0 storage device? Note that the Beanie fund still has cash. If you don't ask, you don't get Brad - -- http://linux.conf.au. 22-25Jan2003. P

[linux-usb-devel] Re: further proposed small API change

2003-01-15 Thread David Brownell
Oliver Neukum wrote: Am Donnerstag, 16. Januar 2003 01:52 schrieb David Brownell: - 1..N TDs ... dependant on the specific buffer passed later, except (usually) for ISO (where i == n) What prevents you from allocating a reasonable number in the general case and as many as you may need in the

[linux-usb-devel] Re: further proposed small API change

2003-01-15 Thread Oliver Neukum
Am Donnerstag, 16. Januar 2003 01:52 schrieb David Brownell: > >> - 1..N TDs ... dependant on the specific buffer passed later, > >>except (usually) for ISO (where i == n) > > > > What prevents you from allocating a reasonable number in the general case > > and as many as you may need in the i

[linux-usb-devel] Re: further proposed small API change

2003-01-15 Thread Greg KH
On Thu, Jan 16, 2003 at 01:18:06AM +0100, Oliver Neukum wrote: > > No. Does that mean you are volunteering to get me a USB2.0 storage device? Sign up to get one through the USB Slashdot Beanie fund, that's how I got mine. Remember, let's not try to optimize something that hasn't been proven to b

[linux-usb-devel] Re: further proposed small API change

2003-01-15 Thread David Brownell
- 1..N TDs ... dependant on the specific buffer passed later, except (usually) for ISO (where i == n) What prevents you from allocating a reasonable number in the general case and as many as you may need in the iso case? What, and have _two_ code paths to debug/stabilize in the hairy hard

[linux-usb-devel] Re: further proposed small API change

2003-01-15 Thread Oliver Neukum
Am Donnerstag, 16. Januar 2003 01:43 schrieb David Brownell: > Oliver Neukum wrote: > > In addition there are strategic thoughts. Doing this change allows > > (although I think that it's 2.7 stuff) us to handle more of disconnect in > > usbcore. We can under these condition link all URBs of a devic

[linux-usb-devel] Re: further proposed small API change

2003-01-15 Thread Oliver Neukum
Am Donnerstag, 16. Januar 2003 01:30 schrieb David Brownell: > Oliver Neukum wrote: > > usb_alloc_urb (int i, int mem_flags) -> > > usb_alloc_urb (struct usb_device *dev, int i, int mem_flags) > > > > Explanation: > > The HCDs allocate and deallocate data structures during every submission. > > Thi

[linux-usb-devel] Re: further proposed small API change

2003-01-15 Thread David Brownell
Oliver Neukum wrote: In addition there are strategic thoughts. Doing this change allows (although I think that it's 2.7 stuff) us to handle more of disconnect in usbcore. We can under these condition link all URBs of a device and walk the list if we detect a disconnect. If a driver allocates all

[linux-usb-devel] Re: further proposed small API change

2003-01-15 Thread Oliver Neukum
Am Donnerstag, 16. Januar 2003 01:05 schrieb Greg KH: > On Thu, Jan 16, 2003 at 12:34:36AM +0100, Oliver Neukum wrote: > > Hi, > > > > I'd like to propose a small further patch. > > > > I want to bind URBs to usb_devices at allocation. > > usb_alloc_urb (int i, int mem_flags) -> > > usb_alloc_urb (

[linux-usb-devel] Re: further proposed small API change

2003-01-15 Thread David Brownell
Oliver Neukum wrote: usb_alloc_urb (int i, int mem_flags) -> usb_alloc_urb (struct usb_device *dev, int i, int mem_flags) Explanation: The HCDs allocate and deallocate data structures during every submission. This change doesn't get away from that though ... there are two classes of allocation,

[linux-usb-devel] Re: further proposed small API change

2003-01-15 Thread Greg KH
On Thu, Jan 16, 2003 at 12:34:36AM +0100, Oliver Neukum wrote: > Hi, > > I'd like to propose a small further patch. > > I want to bind URBs to usb_devices at allocation. > usb_alloc_urb (int i, int mem_flags) -> > usb_alloc_urb (struct usb_device *dev, int i, int mem_flags) > and usb_fill_* loos