On Sun, Oct 08, 2017 at 11:29:45PM -0700, anish kumar wrote: > On Fri, Sep 29, 2017 at 2:09 AM, Charles Keepax > <ckeepax@xxxxxxxxxxxxxxxxxxxxx> wrote: > > On Thu, Sep 28, 2017 at 09:52:39PM -0700, yesanishhere@xxxxxxxxx wrote: > >> From: anish kumar <yesanishhere@xxxxxxxxx> > >> > >> 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. > >> > >> Signed-off-by: anish kumar <yesanishhere@xxxxxxxxx> > >> --- > > > > Reviewed-by: Charles Keepax <ckeepax@xxxxxxxxxxxxxxxxxxxxx> > > Tested-by: Charles Keepax <ckeepax@xxxxxxxxxxxxxxxxxxxxx> > > Charles, do you know if there is anything pending for this series > to be picked up for merge in mark's tree? > > Its been picked up that's what that email from Mark is letting you know. However, you do need to do an incremental patch on top of this one for the issues the Intel build bot spotted with the patch. Thanks, Charles _______________________________________________ Alsa-devel mailing list Alsa-devel@xxxxxxxxxxxxxxxx http://mailman.alsa-project.org/mailman/listinfo/alsa-devel
