On Wed, Mar 04, 2009 at 04:36:17PM +0200, Paulius Zaleckas wrote: > > > Sergei Shtylyov wrote: > >> --- a/drivers/usb/host/uhci-q.c > >> +++ b/drivers/usb/host/uhci-q.c > >> @@ -1498,7 +1498,7 @@ __acquires(uhci->lock) > >> * complete successfully. Either it failed or the URB was > >> * unlinked first. Regardless, don't confuse people with a > >> * negative length. */ > >> - urb->actual_length = max(urb->actual_length, 0); > >> + urb->actual_length = max(urb->actual_length, (u32)0); > > > > Why this is needed? > > Greg: > Since actual length now is unsigned it definitely can't be < 0 > This part of code is not needed anymore... Good catch, Alan just posted a follow-on patch to resolve this. thanks, greg k-h -- 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
