[alsa-devel] [PATCH] ASoC: dapm: Use less aggressive caching to ensure correctness

Mark Brown broonie at kernel.org
Tue Nov 6 18:18:45 CET 2018


On Mon, Nov 05, 2018 at 07:19:12PM +0800, Tzung-Bi Shih wrote:

> +++ b/sound/soc/soc-dapm.c
> @@ -2722,7 +2722,7 @@ static int snd_soc_dapm_add_path(struct snd_soc_dapm_context *dapm,
>  		dapm_mark_dirty(widgets[dir], "Route added");
>  	}
>  
> -	if (dapm->card->instantiated && path->connect)
> +	if (path->connect)
>  		dapm_path_invalidate(path);
>  
>  	return 0;

The whole point with the instantiated check here is that when we
instantiate the card we're supposed to go through and redo all the DAPM
configuration for the card in one fell swoop rather than having to
constantly redo things while we're building up the graph.  If that
invalidation isn't happening then we should fix that.
-------------- next part --------------
A non-text attachment was scrubbed...
Name: signature.asc
Type: application/pgp-signature
Size: 488 bytes
Desc: not available
URL: <http://mailman.alsa-project.org/pipermail/alsa-devel/attachments/20181106/19a72907/attachment.sig>


More information about the Alsa-devel mailing list