Re: [PATCH] ASoC: pcm: allow changing the playback/capture rates for symmetric links

From	Mark Brown <broonie@kernel.org>
Newsgroups	linux.kernel
Subject	Re: [PATCH] ASoC: pcm: allow changing the playback/capture rates for symmetric links
Date	2016-05-04 18:50 +0200
Message-ID	<rv89P-7As-1@gated-at.bofh.it> (permalink)
References	<rstDQ-779-3@gated-at.bofh.it> <rsAlY-41b-15@gated-at.bofh.it> <rsEzf-7F4-3@gated-at.bofh.it> <rsRwv-1zl-55@gated-at.bofh.it> <rugM9-7NK-1@gated-at.bofh.it>
Organization	linux.* mail to news gateway

Show all headers | View raw

[Multipart message — attachments visible in raw view] - view raw

On Mon, May 02, 2016 at 09:43:53AM +0200, Peter Rosin wrote:

> The problem is in the drain/close part. If I leave out snd_pcm_drain
> I naturally lose the tail of the output, but if I have it I get what
> I guess is a repeat of the content in some circular buffer. I.e. the
> very last part of the played sound is played twice(ice).

> If I instead write the generated samples to a file and play them
> with aplay, all is fine, so I appear to be misunderstanding the
> alsa api?

> Anything obviously wrong in the above code?

Look at how aplay handles this...  it's a natural consequence of the
hardware being free running.  You need to either make the teardown
non-blocking or play some silence at the end.

Thread

Re: [PATCH] ASoC: pcm: allow changing the playback/capture rates for  symmetric links Mark Brown <broonie@kernel.org> - 2016-04-28 12:40 +0200
  Re: [PATCH] ASoC: pcm: allow changing the playback/capture rates for symmetric links Takashi Iwai <tiwai@suse.de> - 2016-04-28 13:10 +0200
    Re: [PATCH] ASoC: pcm: allow changing the playback/capture rates for  symmetric links Mark Brown <broonie@kernel.org> - 2016-04-28 13:20 +0200
      Re: [PATCH] ASoC: pcm: allow changing the playback/capture rates for symmetric links Takashi Iwai <tiwai@suse.de> - 2016-04-28 13:30 +0200
        Re: [PATCH] ASoC: pcm: allow changing the playback/capture rates for  symmetric links Mark Brown <broonie@kernel.org> - 2016-04-28 13:50 +0200
          Re: [PATCH] ASoC: pcm: allow changing the playback/capture rates for symmetric links Takashi Iwai <tiwai@suse.de> - 2016-04-28 14:10 +0200
            Re: [PATCH] ASoC: pcm: allow changing the playback/capture rates for  symmetric links Mark Brown <broonie@kernel.org> - 2016-04-28 19:20 +0200
  Re: [PATCH] ASoC: pcm: allow changing the playback/capture rates for  symmetric links Peter Rosin <peda@axentia.se> - 2016-05-02 09:50 +0200
    Re: [PATCH] ASoC: pcm: allow changing the playback/capture rates for  symmetric links Mark Brown <broonie@kernel.org> - 2016-05-04 18:50 +0200
      Re: [alsa-devel] [PATCH] ASoC: pcm: allow changing the playback/capture rates for symmetric links Takashi Iwai <tiwai@suse.de> - 2016-05-09 14:20 +0200

csiph-web