On Thu, Apr 21, 2016 at 7:49 PM, Steve Rae steve.rae@broadcom.com wrote:
On Thu, Apr 21, 2016 at 3:05 AM, Lukasz Majewski l.majewski@samsung.com wrote:
Hi Roger,
Remaining bytes means bytes that are not yet transferred and not the bytes that were transferred in the last transfer.
Reported-by: Lukasz Majewski l.majewski@samsung.com Signed-off-by: Roger Quadros rogerq@ti.com
drivers/usb/gadget/dwc2_udc_otg_xfer_dma.c | 4 ++-- 1 file changed, 2 insertions(+), 2 deletions(-)
diff --git a/drivers/usb/gadget/dwc2_udc_otg_xfer_dma.c b/drivers/usb/gadget/dwc2_udc_otg_xfer_dma.c index a31d875..0594c3d 100644 --- a/drivers/usb/gadget/dwc2_udc_otg_xfer_dma.c +++ b/drivers/usb/gadget/dwc2_udc_otg_xfer_dma.c @@ -235,7 +235,7 @@ static void complete_rx(struct dwc2_udc *dev, u8 ep_num) "%s: RX DMA done : ep = %d, rx bytes = %d/%d, " "is_short = %d, DOEPTSIZ = 0x%x, remained bytes = %d\n", __func__, ep_num, req->req.actual, req->req.length,
is_short, ep_tsr, xfer_size);
is_short, ep_tsr, req->req.length -req->req.actual); if (is_short || req->req.actual == req->req.length) { if (ep_num == EP0_CON && dev->ep0state == DATA_STATE_RECV) { @@ -292,7 +292,7 @@ static void complete_tx(struct dwc2_udc *dev, u8 ep_num) "%s: TX DMA done : ep = %d, tx bytes = %d/%d, " "is_short = %d, DIEPTSIZ = 0x%x, remained bytes = %d\n", __func__, ep_num, req->req.actual, req->req.length,
is_short, ep_tsr, xfer_size);
is_short, ep_tsr, req->req.length - req->req.actual); if (ep_num == 0) { if (dev->ep0state == DATA_STATE_XMIT) {Acked-by: Lukasz Majewski l.majewski@samsung.com
Tested-by: Steve Rae srae@broadcom.com
[Test HW: bcm28155_ap board]
Thanks, Steve
retry because patchworks missed this....
Tested-by: Steve Rae srae@broadcom.com [Test HW: bcm28155_ap board]
-- Best regards,
Lukasz Majewski
Samsung R&D Institute Poland (SRPOL) | Linux Platform Group