Re: [PATCH v2] ASoC: core: init delayed_work for codec-codec links

From: Mark Brown
Date: Fri Aug 02 2013 - 06:15:26 EST

On Fri, Aug 02, 2013 at 11:01:08AM +0100, Richard Fitzgerald wrote:

> +static void codec2codec_close_delayed_work(struct work_struct *work)
> +{
> + /* No action required.
> + * C2C links only represent a hardware connection and currently
> + * we assume that we don't have to explicitly start or stop them
> + */
> +}

That's not the reason - what's happening with the stream events is that
since these links are not on the edge of the DAPM graph we don't need to
interface with the outside world. Further since we're not dealing with
the application layer there is no need to delay the power down, the
reason that's there is to stop us powering the link down if there's just
a small gap between tracks so we don't get pops, clicks and extra gaps
if the application layer is closing the device at the end of every track
(which is quite a common pattern).

Attachment: signature.asc
Description: Digital signature