Am Montag, 18. Juni 2012, 18:25:15 schrieb Huajun Li: > On Mon, Jun 18, 2012 at 3:23 PM, Oliver Neukum <oneukum@xxxxxxx> wrote: > > Am Montag, 18. Juni 2012, 01:30:17 schrieb David Miller: > >> From: Huajun Li <huajun.li.lee@xxxxxxxxx> > >> Date: Wed, 13 Jun 2012 20:50:31 +0800 > >> > >> > intr_complete() submits URB even the interrupt endpoint stalls. > >> > This patch will try to activate the endpoint once the exception > >> > occurs, and then re-submit the URB if the endpoint works again. > >> > > >> > Signed-off-by: Huajun Li <huajun.li.lee@xxxxxxxxx> > >> > >> Review from USB experts would be appreciated. > > > > The code implements a minimum error handler correctly. > > Did you observe a stall in actual hardware or is this a just > > in case patch? > > > > This one is just a patch, thanks for your comments. Then I am inclined to say that this is not fully thought through If the endpoint stalls, the device has detected an error. We have no idea how to generically handle this error. We might come into a vicious circle. Could you add a sanity limit for the amount of halts we clear? Regards Oliver -- To unsubscribe from this list: send the line "unsubscribe linux-usb" in the body of a message to majordomo@xxxxxxxxxxxxxxx More majordomo info at http://vger.kernel.org/majordomo-info.html
