On Thursday 03 August 2006 7:11 am, Luiz Fernando N. Capitulino wrote:
> 

> __usb_endpoint_is_in()
> __usb_endpoint_is_out()
> __usb_endpoint_is_bulk()
> __usb_endpoint_is_int()

Why the double underscores?  Those make perfectly reasonable
functions in their own right.  Ditto for "is_iso", 'is_interrupt".


> usb_endpoint_is_bulk_in()
> usb_endpoint_is_bulk_out()
> usb_endpoint_is_int_in()
> usb_endpoint_is_int_out()
> 
>  Which is flexible, is still good, but looks too much macros to me.

The "is_in" and "is_out" tests are more common than the "is_bulk_in" etc.,
in my experience ...


>  Oh, BTW, I didn't add a macro to check isochronous endpoints because
> they're not so common in USB driver's code. But if we're going to make
> the API complete, I think we should add it.

Yes.

- Dave

-------------------------------------------------------------------------
Take Surveys. Earn Cash. Influence the Future of IT
Join SourceForge.net's Techsay panel and you'll get the chance to share your
opinions on IT & business topics through brief surveys -- and earn cash
http://www.techsay.com/default.php?page=join.php&p=sourceforge&CID=DEVDEV
_______________________________________________
linux-usb-devel@lists.sourceforge.net
To unsubscribe, use the last form field at:
https://lists.sourceforge.net/lists/listinfo/linux-usb-devel

Reply via email to