On Fri, 20 Dec 2013, Greg Kroah-Hartman wrote: > > > /* Check that the pipe's type matches the endpoint's type */ > > > if (usb_pipetype(urb->pipe) != pipetypes[xfertype]) > > > > It looks as though it ought to be possible to make that check: > > if (unlikely(xfertype != urb->pipe->valid_xfertype)) > > You should almost never use unlikely() on your own, in a driver, as the > compiler should make better code without it. Is that really true? If it is, what point is there in having the likely()/unlikely() macros in the first place? Alan Stern -- 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
