Re:

From: Markus Trippelsdorf
Date: Thu Sep 06 2012 - 02:53:35 EST


On 2012.09.06 at 08:48 +0200, Takashi Iwai wrote:
> At Thu, 06 Sep 2012 08:33:30 +0200,
> Daniel Mack wrote:
> >
> > On 06.09.2012 08:02, Markus Trippelsdorf wrote:
> > > On 2012.09.04 at 16:40 +0200, Takashi Iwai wrote:
> > >> ----------------------------------------------------------------
> > >> Sound fixes for 3.6-rc5
> > >>
> > >> There are nothing scaring, contains only small fixes for HD-audio and
> > >> USB-audio:
> > >> - EPSS regression fix and GPIO fix for HD-audio IDT codecs
> > >> - A series of USB-audio regression fixes that are found since 3.5 kernel
> > >>
> > >> ----------------------------------------------------------------
> > >> Daniel Mack (4):
> > >> ALSA: snd-usb: Fix URB cancellation at stream start
> > >> ALSA: snd-usb: restore delay information
> > > ^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^
> > > The commit fbcfbf5f above causes the following lines to be printed
> > > whenever I start a new song:
> >
> > Copied Pierre-Louis Bossart - he wrote the code in 294c4fb8 which this
> > patch (fbcfbf5f) brings back now.
> >
> > > delay: estimated 0, actual 352
> > > delay: estimated 353, actual 705
> > >
> > > (44.1 * 8 = 352.8)
> > >
> > > This happens with an USB-DAC that identifies itself as "C-Media USB
> > > Headphone Set".
> >
> > And you didn't you see these lines with 3.4?
>
> Maybe the difference of start condition?
>
> Markus, does the patch below fix anything?

Unfortunately no.
However reverting the following fixes the problem:

commit 245baf983cc39524cce39c24d01b276e6e653c9e
Author: Daniel Mack <zonque@xxxxxxxxx>
Date: Thu Aug 30 18:52:30 2012 +0200

ALSA: snd-usb: fix calls to next_packet_size

--
Markus
--
To unsubscribe from this list: send the line "unsubscribe linux-kernel" in
the body of a message to majordomo@xxxxxxxxxxxxxxx
More majordomo info at http://vger.kernel.org/majordomo-info.html
Please read the FAQ at http://www.tux.org/lkml/