Re: anchors and their use

Alan Stern <[email protected]>
Newsgroups gmane.linux.usb.devel
Message-ID <[email protected]>
On Wed, 16 May 2007, Oliver Neukum wrote:

> Hi Alan, list,
> 
> here's the patch to introduce anchors for use with "fire-and-forget"
> techniques and a modification for the skeleton driver to use them.
> This should make implementing pre/post_reset and suspend/resume
> much easier.

This won't build because you forgot to export usb_unanchor_urb().

You could make usb_init_anchor() non-inline.  I'm not sure where 
tradeoff lies, but you might be over the limit.

I don't like the way you combined the error handling in usb-skeleton 
with the anchor stuff.  Better to keep the two things in separate 
patches.

There's no need for the version of skel_draw_down() which acquires the
mutex.  You use it only in skel_flush(), so you might as well acquire
the mutex in the caller.

Alan Stern


-------------------------------------------------------------------------
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/
_______________________________________________
[email protected]
To unsubscribe, use the last form field at:
https://lists.sourceforge.net/lists/listinfo/linux-usb-devel
lmpx.com only provides a reader for public news (NNTP) servers. It is not affiliated with the servers or forums shown here and is not responsible for the content of articles, which is written by their respective authors.