]> git.karo-electronics.de Git - karo-tx-linux.git/commit
ALSA: usb-audio: Fix bogus error messages for delay accounting
authorTakashi Iwai <tiwai@suse.de>
Thu, 6 Sep 2012 12:58:00 +0000 (14:58 +0200)
committerGreg Kroah-Hartman <gregkh@linuxfoundation.org>
Tue, 2 Oct 2012 17:39:00 +0000 (10:39 -0700)
commitea026e7f8bde9b0906137b06a70b22d706d3c0c5
tree60d9132c5f9683c662688b47c228d0d3ec836a94
parentffe018be7b6f3b423654d64980b1b1119bfc3fda
ALSA: usb-audio: Fix bogus error messages for delay accounting

commit 1213a205f9ed27d97de3d5bed28fb085ef4853e2 upstream.

The recent fix for the missing fine delayed time adjustment gives
strange error messages at each start of the playback stream, such as
  delay: estimated 0, actual 352
  delay: estimated 353, actual 705

These come from the sanity check in retire_playback_urb().  Before the
stream is activated via start_endpoints(), a few silent packets have
been already sent.  And at this point the delay account is still in
the state as if the new packets are just queued, so the driver gets
confused and spews the bogus error messages.

For fixing the issue, we just need to check whether the received
packet is valid, whether it's zero sized or not.

Reported-by: Markus Trippelsdorf <markus@trippelsdorf.de>
Signed-off-by: Takashi Iwai <tiwai@suse.de>
Signed-off-by: Greg Kroah-Hartman <gregkh@linuxfoundation.org>
sound/usb/pcm.c