On Do, 2018-11-15 at 18:14 +0100, Nicolas Saenz Julienne wrote:
Hi,
what Alan said, in addition you need to stop the error handling
when the device is suspended or reset.
> @@ -713,8 +751,10 @@ static void hub_irq(struct urb *urb)
> return;
>
> status = usb_submit_urb(hub-
On Fri, 16 Nov 2018, Nicolas Saenz Julienne wrote:
> Hi Alan,
> thanks for the review.
>
> On Thu, 2018-11-15 at 14:24 -0500, Alan Stern wrote:
> > On Thu, 15 Nov 2018, Nicolas Saenz Julienne wrote:
> >
> > > An URB submission error in the HUB's endpoint completion function
> > > renders the who
Hi Alan,
thanks for the review.
On Thu, 2018-11-15 at 14:24 -0500, Alan Stern wrote:
> On Thu, 15 Nov 2018, Nicolas Saenz Julienne wrote:
>
> > An URB submission error in the HUB's endpoint completion function
> > renders the whole HUB device unresponsive. This patch introduces a
> > routine that
On Thu, 15 Nov 2018, Nicolas Saenz Julienne wrote:
> An URB submission error in the HUB's endpoint completion function
> renders the whole HUB device unresponsive. This patch introduces a
> routine that retries the submission for 1s to then, as a last resort,
> reset the whole device.
>
> The imp
An URB submission error in the HUB's endpoint completion function
renders the whole HUB device unresponsive. This patch introduces a
routine that retries the submission for 1s to then, as a last resort,
reset the whole device.
The implementation is based on usbhid/hid_core.c's, which implements th
