Re: [PATCH v3 0/2] staging: dwc2: add microframe scheduler

2014-08-06 Thread Nick Hudson
On 08/05/14 20:15, Paul Zimmerman wrote: From: Nick Hudson [mailto:sk...@netbsd.org] Sent: Tuesday, August 05, 2014 1:57 AM On 07/22/14 22:22, Nick Hudson wrote: On 07/22/14 18:46, Paul Zimmerman wrote: The nak_frame handling would have added some benefit, but wouldn't handle all cases. I'm s

RE: [PATCH v3 0/2] staging: dwc2: add microframe scheduler

2014-08-05 Thread Paul Zimmerman
> From: Nick Hudson [mailto:sk...@netbsd.org] > Sent: Tuesday, August 05, 2014 1:57 AM > > On 07/22/14 22:22, Nick Hudson wrote: > > On 07/22/14 18:46, Paul Zimmerman wrote: > >> > >>> The nak_frame handling would have added some benefit, but wouldn't > >>> handle all > >>> cases. > >>> > >>> I'm

Re: [PATCH v3 0/2] staging: dwc2: add microframe scheduler

2014-08-05 Thread Nick Hudson
On 07/22/14 22:22, Nick Hudson wrote: On 07/22/14 18:46, Paul Zimmerman wrote: The nak_frame handling would have added some benefit, but wouldn't handle all cases. I'm seeing problems with devices NAKing to the point of confusing dwc2 completely, so I'd like to investigate a NAK holdoff sch

Re: [PATCH v3 0/2] staging: dwc2: add microframe scheduler

2014-07-22 Thread Nick Hudson
On 07/22/14 18:46, Paul Zimmerman wrote: The nak_frame handling would have added some benefit, but wouldn't handle all cases. I'm seeing problems with devices NAKing to the point of confusing dwc2 completely, so I'd like to investigate a NAK holdoff scheme that's acceptable for dwc2. I suspe

Re: [PATCH v3 0/2] staging: dwc2: add microframe scheduler

2014-07-22 Thread Jonathan Bell
On Tue, 22 Jul 2014 18:46:27 +0100, Paul Zimmerman wrote: From: Nick Hudson [mailto:sk...@netbsd.org] Sent: Tuesday, July 22, 2014 12:19 AM On 09/23/13 22:23, Paul Zimmerman wrote: > Here is the third version of the microframe scheduler patch. This > version removes the NAK holdoff patch fro

RE: [PATCH v3 0/2] staging: dwc2: add microframe scheduler

2014-07-22 Thread Paul Zimmerman
> From: Nick Hudson [mailto:sk...@netbsd.org] > Sent: Tuesday, July 22, 2014 12:19 AM > > On 09/23/13 22:23, Paul Zimmerman wrote: > > Here is the third version of the microframe scheduler patch. This > > version removes the NAK holdoff patch from the series, since it > > was effectively a no-op a

Re: [PATCH v3 0/2] staging: dwc2: add microframe scheduler

2014-07-22 Thread Nick Hudson
On 09/23/13 22:23, Paul Zimmerman wrote: Here is the third version of the microframe scheduler patch. This version removes the NAK holdoff patch from the series, since it was effectively a no-op as pointed out by Matthijs. I think there was a misunderstanding here - Matthijs was pointing out th

Re: [PATCH v3 0/2] staging: dwc2: add microframe scheduler

2013-09-23 Thread Dan Carpenter
On Mon, Sep 23, 2013 at 02:23:32PM -0700, Paul Zimmerman wrote: > There were some style-related comments from Dan Carpenter, but I > would prefer to have those as an additional patch, once we have > this known-good code added to the driver. > Sure. Just add a link to the TODO list so we don't fo

[PATCH v3 0/2] staging: dwc2: add microframe scheduler

2013-09-23 Thread Paul Zimmerman
Here is the third version of the microframe scheduler patch. This version removes the NAK holdoff patch from the series, since it was effectively a no-op as pointed out by Matthijs. It also splits out into a separate patch one hunk which was not part of the downstream patch. It also acknowledges de