[U-Boot] [PATCH 3/3] usb: s3c-otg: Fix remaining bytes in debug messages

[Steve Rae]

On Thu, Apr 21, 2016 at 3:05 AM, Lukasz Majewski <l.majewski at 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 at samsung.com>
> > Signed-off-by: Roger Quadros <rogerq at 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 at samsung.com>
>
> Tested-by: Steve Rae <srae at broadcom.com>
[Test HW: bcm28155_ap board]

Thanks, Steve


> --
> Best regards,
>
> Lukasz Majewski
>
> Samsung R&D Institute Poland (SRPOL) | Linux Platform Group
>