From: Alan Stern Date: Wed, 29 May 2013 17:21:01 +0000 (-0400) Subject: USB: MUSB: upgrade the isochronous API X-Git-Tag: upstream/snapshot3+hdmi~4849^2~87 X-Git-Url: http://review.tizen.org/git/?a=commitdiff_plain;h=8a1ea51f87a6149c3263a63e9c60d852bedbecd7;p=platform%2Fadaptation%2Frenesas_rcar%2Frenesas_kernel.git USB: MUSB: upgrade the isochronous API This patch attempts to fix the isochonour API in the musb host driver. In particular, the urb->start_frame field should always be set by the driver; it isn't an input parameter. The simplest way to accomplish this is to treat all URBs as though the URB_ISO_ASAP flag was set. This won't give the right behavior when an underrun occurs, but I don't know enough about the musb driver to handle that case. Unfortunately, I have no way to test this change. Signed-off-by: Alan Stern Signed-off-by: Greg Kroah-Hartman --- diff --git a/drivers/usb/musb/musb_host.c b/drivers/usb/musb/musb_host.c index 8914dec..85ed11e 100644 --- a/drivers/usb/musb/musb_host.c +++ b/drivers/usb/musb/musb_host.c @@ -269,8 +269,7 @@ musb_start_urb(struct musb *musb, int is_in, struct musb_qh *qh) /* FIXME this doesn't implement that scheduling policy ... * or handle framecounter wrapping */ - if ((urb->transfer_flags & URB_ISO_ASAP) - || (frame >= urb->start_frame)) { + if (1) { /* Always assume URB_ISO_ASAP */ /* REVISIT the SOF irq handler shouldn't duplicate * this code; and we don't init urb->start_frame... */