lkml.org 
[lkml]   [2018]   [Nov]   [5]   [last100]   RSS Feed
Views: [wrap][no wrap]   [headers]  [forward] 
 
Messages in this thread
/
From
SubjectRe: [PATCH] usb: dwc2: gadget: fix ISOC frame overflow handling
Date
Hi John,

On 10/23/2018 5:43 PM, John Keeping wrote:
> By clearing the overrun flag as soon as the target frame is next
> incremented, we can end up incrementing the target frame more than
> expected in dwc2_gadget_handle_ep_disabled() when the endpoint's
> interval is greater than 1. This happens if the target frame has just
> wrapped at the point when the endpoint is disabled and the frame number
> has not yet done so.
>
> Instead, wait until the frame number also wraps and then clear the
> overrun flag.
>
> Signed-off-by: John Keeping <john@metanate.com>
> ---
> drivers/usb/dwc2/gadget.c | 2 +-
> 1 file changed, 1 insertion(+), 1 deletion(-)
>
> diff --git a/drivers/usb/dwc2/gadget.c b/drivers/usb/dwc2/gadget.c
> index 2d6d2c8244de..8da2c052dfa1 100644
> --- a/drivers/usb/dwc2/gadget.c
> +++ b/drivers/usb/dwc2/gadget.c
> @@ -117,7 +117,7 @@ static inline void dwc2_gadget_incr_frame_num(struct dwc2_hsotg_ep *hs_ep)
> if (hs_ep->target_frame > DSTS_SOFFN_LIMIT) {
> hs_ep->frame_overrun = true;
> hs_ep->target_frame &= DSTS_SOFFN_LIMIT;
> - } else {
> + } else if (hs_ep->parent->frame_number < hs_ep->target_frame) {
> hs_ep->frame_overrun = false;
> }
> }
>
Did you tested mentioned by you scenario? If you see issue can you
provide debug log and point the issue line in the log.

Thanks,
Minas


\
 
 \ /
  Last update: 2018-11-05 09:28    [W:3.324 / U:0.016 seconds]
©2003-2020 Jasper Spaans|hosted at Digital Ocean and TransIP|Read the blog|Advertise on this site