On Wednesday 04 February 2009, Sergei Shtylyov wrote:
> From: Dmitry Krivoschekov <dkrivosche...@ru.mvista.com>
> 
> Input queue should be used for Tx if an endpoint's FIFO is shared.

(Because there's only the one queue in such cases...)

> 
> Signed-off-by: Dmitry Krivoschekov <dkrivosche...@ru.mvista.com>
> Signed-off-by: Sergei Shtylyov <sshtyl...@ru.mvista.com>

Acked-by: David Brownell <dbrown...@users.sourceforge.net>

Greg, this bugfix looks appropriate for 2.6.29-rc.

I'm not going to suggest -stable backports for any of
these fixes, since the MUSB codes hasn't really been
usable previously (sigh).


Also:  note that I'm just reviewing these, not trying
to test them on half a dozen varieties of hardware
using this driver.


> 
> ---
> The patch is against the recent Linus' kernel...
> 
>  drivers/usb/musb/musb_host.c |    3 ++-
>  1 files changed, 2 insertions(+), 1 deletion(-)
> 
> Index: mainline/drivers/usb/musb/musb_host.c
> ===================================================================
> --- mainline.orig/drivers/usb/musb/musb_host.c
> +++ mainline/drivers/usb/musb/musb_host.c
> @@ -1155,7 +1155,8 @@ void musb_host_tx(struct musb *musb, u8 
>       struct urb              *urb;
>       struct musb_hw_ep       *hw_ep = musb->endpoints + epnum;
>       void __iomem            *epio = hw_ep->regs;
> -     struct musb_qh          *qh = hw_ep->out_qh;
> +     struct musb_qh          *qh = hw_ep->is_shared_fifo ? hw_ep->in_qh
> +                                                         : hw_ep->out_qh;
>       u32                     status = 0;
>       void __iomem            *mbase = musb->mregs;
>       struct dma_channel      *dma;
> 
> 



_______________________________________________
Davinci-linux-open-source mailing list
Davinci-linux-open-source@linux.davincidsp.com
http://linux.davincidsp.com/mailman/listinfo/davinci-linux-open-source

Reply via email to