On Fri, 15 Dec 2017, Mathias Nyman wrote: > Clear Feature Endpoint Halt should reset the data toggle even if the > endpoint isn't halted. Host should manage to clear the host side data > toggle to keep in sync with the device. > > Test by sending a "3 data packet URB" before and after clearing the halt. > this should create a toggle sequence with two consecutive DATA0 packets. > > A successful test sequence looks like this > ClearFeature(ENDPOINT_HALT) - initial toggle clear > DATA0 (max packet sized) > DATA1 (max packet sized) > DATA0 (zero length packet) > ClearFeature(ENDPOINT_HALT) - resets toggle > DATA0 (max packet sized), if clear halt fails then toggle is DATA1 > DATA1 (max packet sized) > DATA0 (zero length packet) > > Signed-off-by: Mathias Nyman <mathias.nyman@xxxxxxxxxxxxxxx> > > --- > v2: - use 2 * get_maxpacket() to set urb length > - don't check dev->in_pipe for this out only test > - tidy up the for loop a bit Acked-by: Alan Stern <stern@xxxxxxxxxxxxxxxxxxx> -- To unsubscribe from this list: send the line "unsubscribe linux-usb" in the body of a message to majordomo@xxxxxxxxxxxxxxx More majordomo info at http://vger.kernel.org/majordomo-info.html