Borchers, Al (C)(STP) wrote:
David --

I've been using the USB gadget code to develop a driver
for a NetChip 2280 device that runs linux.  It is great--
thanks so much.

People do seem to be using it without any real trouble, which is a good thing! :)


In running the USBCV Chapter 9 compliance test (usb.org)
and found that the net2280.c file has a bug that causes
it to fail the HaltEndpointTest.

This test uses the SET_FEATURE and CLEAR_FEATURE requests
to do ENDPOINT_HALT.  However, the ENDPOINT_HALT selector is
in the wValue field, not in wIndex as the net2280.c code
assumes.

That would prevent it working for ep != 0 (== ENDPOINT_HALT), which matches a similar report I got. Linux has regression tests for ep0, but not (yet) other endpoints.


The attached tiny patch fixes this.  It applies to the net2280.c
in 2.5.70, but was actually generated from a version from
a month or so ago.

Thanks! I hadn't had a chance to track it down that far. Assuming your patch checks out, I'll merge this along.

- Dave






------------------------------------------------------- This SF.NET email is sponsored by: eBay Great deals on office technology -- on eBay now! Click here: http://adfarm.mediaplex.com/ad/ck/711-11697-6916-5 _______________________________________________ [EMAIL PROTECTED] To unsubscribe, use the last form field at: https://lists.sourceforge.net/lists/listinfo/linux-usb-devel

Reply via email to