On Saturday 30 June 2007, Oliver Neukum wrote:
> So do we have consensus for this prototype:
>
> struct urb *usb_alloc_urb(struct usb_host_endpoint *ep,
> unsigned int max_length, unsigned int iso_fields,
> gfp_t mem_flags);
Calling conventions that require line-wrapping are almost by
definition unwieldy ... :(
That might make sense for an HCD method; size_t for length,
and rename it so it doesn't suggest it's a "maximum" so much
as an educated guess.
But otherwise, I thought one of the goals was to get rid of
such annoying "iso fields" (should be packets, not fields!)
parameters, which were always being passed as zero ... ?
I'd expect drivers would just pass {ep, length, gfp} except
for the rare ISO ones, which would pass the number of packets.
That suggests two separate signatures at that level.
- Dave
-------------------------------------------------------------------------
This SF.net email is sponsored by DB2 Express
Download DB2 Express C - the FREE version of DB2 express and take
control of your XML. No limits. Just data. Click to get it now.
http://sourceforge.net/powerbar/db2/
_______________________________________________
linux-usb-devel@lists.sourceforge.net
To unsubscribe, use the last form field at:
https://lists.sourceforge.net/lists/listinfo/linux-usb-devel
