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@xxxxxxxxxxxxxxxx > http://mailman.alsa-project.org/mailman/listinfo/alsa-devel _______________________________________________ Alsa-devel mailing list Alsa-devel@xxxxxxxxxxxxxxxx http://mailman.alsa-project.org/mailman/listinfo/alsa-devel
