Re: [PATCH 0/20] ASoC: rsnd: add Synchronous SRC and U/O error salvage support

[Mark Brown <broonie@kernel.org>]

[-- Attachment #1.1: Type: text/plain, Size: 1442 bytes --]

On Mon, Jan 05, 2015 at 12:42:27AM +0000, Kuninori Morimoto wrote:

> 2nd "dynamic" rate convert which is this patch is used as revision.
> I don't know detail of use case, but, according to HW team,
> sound clock has small accidental error.
> Because of this, if we used 48kHz sound in long term,
> but, the real sound can be 48001 Hz or 47999 Hz or something like that.
> So, if we use it without revision in long term, the sound will has
> small noise. This will be problem for TV or radio (?).
> Thus, sound player wants to revise it by "dynamic" rate convert.
> I guess DPCM is good match for "static" rate convert, but not good
> for "dynamic".

In general it should be fine for anything - the driver for the SoC
shouldn't need to understand why the rates are being converted, it
should let the system integration worry about picking the rates.  That
way we can more easily make the frameworks smarter and get benefits over
all machines.

Though having said that I'm a little confused as to what exactly this
control is intended for - am I right in thinking that what's actually
going on here is that something in userspace is measuring the actual
measured rate and feeding that information back to the DSP for
correction?  If that's the case that does seem reasonable but also like
we ought to have a general control interface for it.

Adding Tim and Pierre since they might have views if my guess is
correct, I could be wrong though.

[-- Attachment #1.2: Digital signature --]
[-- Type: application/pgp-signature, Size: 473 bytes --]

[-- Attachment #2: Type: text/plain, Size: 0 bytes --]