[alsa-devel] [PATCH] ASoC: dapm: Add new widget type for constructing DAPM graphs on DSPs.

[Liam Girdwood]

On Wed, 2017-06-07 at 20:23 +0100, Mark Brown wrote:
> On Tue, Jun 06, 2017 at 03:44:42PM +0100, Liam Girdwood wrote:
> 
> > +	snd_soc_dapm_buffer,		/* DSP/CODEC internal buffer */
> > +	snd_soc_dapm_pipeline,		/* DSP/CODEC internal pipeline */
> > +	snd_soc_dapm_effect,		/* DSP/CODEC effect component */
> 
> What's the difference between a buffer and a pipeline, or between a
> pipeline and an effect?  We should have clear documentation, if they're
> different we're probably going to end up using them differently in the
> core at some point and if people use the wrong widget type it might end
> up breaking things.

Yes, that's a good point. I'll do a V2 that includes some documentation.

> 
> > +	snd_soc_dapm_codec,		/* FW/SW coder/decoder component */
> 
> Can we call this a fmt_conv or something?  CODEC is unfortunately a bit
> overloaded.

True, what about snd_soc_dapm_media_codec ? or _sw_codec ? or
_media_decoder ?

Liam

> _______________________________________________
> Alsa-devel mailing list
> Alsa-devel at alsa-project.org
> http://mailman.alsa-project.org/mailman/listinfo/alsa-devel