On 06.09.2012 16:31, Takashi Iwai wrote: > At Thu, 6 Sep 2012 15:17:58 +0200, > Markus Trippelsdorf wrote: >> >> On 2012.09.06 at 15:08 +0200, Takashi Iwai wrote: >>> At Thu, 6 Sep 2012 11:43:48 +0200, >>> Markus Trippelsdorf wrote: >>>> >>>> On 2012.09.06 at 10:21 +0200, Takashi Iwai wrote: >>>>> At Thu, 06 Sep 2012 09:35:26 +0200, >>>>> Takashi Iwai wrote: >>>>> >>>>> In short, a patch like below may fix the issue (note: completely >>>>> untested!) >>>> >>>> No it doesn't, unfortunately... >>> >>> OK, I start tracking down the problem a bit more deeply now. >>> >>> The issue happens when the first two URBs are passed to >>> retire_playback_urb(). These are URBs filled before start_endpoints() >>> are set, so they contain actually zero size. Even though these are >>> a sort of dummy packets, the driver still tries to check with the >>> queued delay account, and gives bogus errors. >>> >>> So, essentially the messages are harmless and nothing to worry too >>> much, but surely it doesn't look sexy. >>> >>> The patch below should fix the problem. Please give it a try. >> >> Yes, your patch finally fixes the problem. >> Thank you Takashi-san. > > Thanks for your quick test! > > If Daniel has no objection with that patch, I'm going to merge it.
No objections from my side. I also gave it a quick test, and even though I never saw the problem Markus was seeing, I agree to your findings. Many thanks, everyone :) Daniel -- To unsubscribe from this list: send the line "unsubscribe linux-kernel" in the body of a message to majord...@vger.kernel.org More majordomo info at http://vger.kernel.org/majordomo-info.html Please read the FAQ at http://www.tux.org/lkml/
