From: supriya karanth <supriya.kara...@stericsson.com>

The TXMAXP register is not set correctly for full speed bulk case
when the can_bulk_split() is used. Without this PIO transfers will
not take place correctly

The "mult" factor needs to be updated correctly for the
can_bulk_split() case

The AUTOSET bit in the TXCSR is not being set if the "mult"
factor is greater than 0 for the High Bandwidth ISO case.
But the "mult" factor is also greater than 0 in case of Full speed
bulk transfers with the packet splitting in TXMAXP register

Without the AUTOSET the DMA transfers will not progress in mode1

Signed-off-by: supriya karanth <supriya.kara...@stericsson.com>
Signed-off-by: Praveena NADAHALLY <praveen.nadaha...@stericsson.com>
Acked-by: Linus Walleij <linus.wall...@linaro.org>
---
 drivers/usb/musb/musb_gadget.c |   20 ++++++++++++++++++--
 1 files changed, 18 insertions(+), 2 deletions(-)

diff --git a/drivers/usb/musb/musb_gadget.c b/drivers/usb/musb/musb_gadget.c
index b6b84da..8fb0c1f 100644
--- a/drivers/usb/musb/musb_gadget.c
+++ b/drivers/usb/musb/musb_gadget.c
@@ -408,7 +408,19 @@ static void txstate(struct musb *musb, struct musb_request 
*req)
                                        csr |= (MUSB_TXCSR_DMAENAB
                                                        | MUSB_TXCSR_DMAMODE
                                                        | MUSB_TXCSR_MODE);
-                                       if (!musb_ep->hb_mult)
+                                       /*
+                                        * Enable Autoset according to table
+                                        * below
+                                        * bulk_split hb_mult   Autoset_Enable
+                                        *      0       0       Yes(Normal)
+                                        *      0       >0      No(High BW ISO)
+                                        *      1       0       Yes(HS bulk)
+                                        *      1       >0      Yes(FS bulk)
+                                        */
+                                       if (!musb_ep->hb_mult ||
+                                               (musb_ep->hb_mult &&
+                                                can_bulk_split(musb,
+                                                   musb_ep->type)))
                                                csr |= MUSB_TXCSR_AUTOSET;
                                }
                                csr &= ~MUSB_TXCSR_P_UNDERRUN;
@@ -1113,9 +1125,13 @@ static int musb_gadget_enable(struct usb_ep *ep,
                 */
                if (musb->double_buffer_not_ok)
                        musb_writew(regs, MUSB_TXMAXP, hw_ep->max_packet_sz_tx);
-               else
+               else {
+                       if (can_bulk_split(musb, musb_ep->type))
+                               musb_ep->hb_mult = (hw_ep->max_packet_sz_tx /
+                                                       musb_ep->packet_sz) - 1;
                        musb_writew(regs, MUSB_TXMAXP, musb_ep->packet_sz
                                        | (musb_ep->hb_mult << 11));
+               }
 
                csr = MUSB_TXCSR_MODE | MUSB_TXCSR_CLRDATATOG;
                if (musb_readw(regs, MUSB_TXCSR)
-- 
1.7.1

--
To unsubscribe from this list: send the line "unsubscribe linux-usb" in
the body of a message to majord...@vger.kernel.org
More majordomo info at  http://vger.kernel.org/majordomo-info.html

Reply via email to