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