Ping....... On Sat, Aug 26, 2017 at 1:29 AM, anish kumar <yesanishhere@xxxxxxxxx> wrote: > Currently in codec to codec dai link if there are multiple > params defined then dapm can use created kcontrol to > decide which param to apply at runtime. > > However, in case there is only single param configuration > then there is no point in creating the kcontrol and also there > is no point in allocating memory for kcontrol. > > In the snd_soc_dapm_new_pcm function, there is memory > allocation happening for kcontrol which is later used > or not used based on num_param. It is better to not > allocate memory when there is only a single configuration. > This change is to remedy that anomaly. > > This code change doesn't do much except just refactor > snd_soc_dapm_new_pcm function. Now the part of allocating > memory for kcontrol is allocated to a new function and it > is called only in the case it is needed. > > anish kumar (4): > ASoC: dapm: fix error path in snd_soc_dapm_new_pcm Below 3 patches are still not reviewed. > ASoC: dapm: Refactor the code in snd_soc_dapm_new_pcm > ASoC: dapm: Avoid creating kcontrol for params > ASoC: dapm: Remove the redundant check > > sound/soc/soc-dapm.c | 123 ++++++++++++++++++++++++++++----------------------- > 1 file changed, 67 insertions(+), 56 deletions(-) > > -- > 2.10.1 (Apple Git-78) > _______________________________________________ Alsa-devel mailing list Alsa-devel@xxxxxxxxxxxxxxxx http://mailman.alsa-project.org/mailman/listinfo/alsa-devel