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

The "mult" factor is not updated properly for the can_bulk_split()
case.

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

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_host.c |   20 ++++++++++++++++----
 1 files changed, 16 insertions(+), 4 deletions(-)

diff --git a/drivers/usb/musb/musb_host.c b/drivers/usb/musb/musb_host.c
index 3df6a76..b3e663c 100644
--- a/drivers/usb/musb/musb_host.c
+++ b/drivers/usb/musb/musb_host.c
@@ -634,7 +634,17 @@ static bool musb_tx_dma_program(struct dma_controller *dma,
                mode = 1;
                csr |= MUSB_TXCSR_DMAMODE | MUSB_TXCSR_DMAENAB;
                /* autoset shouldn't be set in high bandwidth */
-               if (qh->hb_mult == 1)
+               /*
+                * Enable Autoset according to table
+                * below
+                * bulk_split hb_mult   Autoset_Enable
+                *      0       1       Yes(Normal)
+                *      0       >1      No(High BW ISO)
+                *      1       1       Yes(HS bulk)
+                *      1       >1      Yes(FS bulk)
+                */
+               if (qh->hb_mult == 1 || (qh->hb_mult > 1 &&
+                                       can_bulk_split(hw_ep->musb, qh->type)))
                        csr |= MUSB_TXCSR_AUTOSET;
        } else {
                mode = 0;
@@ -794,10 +804,12 @@ static void musb_ep_program(struct musb *musb, u8 epnum,
                        if (musb->double_buffer_not_ok)
                                musb_writew(epio, MUSB_TXMAXP,
                                                hw_ep->max_packet_sz_tx);
-                       else if (can_bulk_split(musb, qh->type))
+                       else if (can_bulk_split(musb, qh->type)) {
+                               qh->hb_mult = hw_ep->max_packet_sz_tx
+                                               / packet_sz;
                                musb_writew(epio, MUSB_TXMAXP, packet_sz
-                                       | ((hw_ep->max_packet_sz_tx /
-                                               packet_sz) - 1) << 11);
+                                       | ((qh->hb_mult) - 1) << 11);
+                       }
                        else
                                musb_writew(epio, MUSB_TXMAXP,
                                                qh->maxpacket |
-- 
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