Re: [PATCH v3] ASOC: dapm: add code to configure dai link parameters

2014-08-18 Thread Mark Brown
On Wed, Aug 06, 2014 at 05:16:26PM +0100, Nikesh Oswal wrote: > +/* create new dapm dai link control */ > +static int dapm_new_dai_link(struct snd_soc_dapm_widget *w) > +{ > + int i, ret; > + struct snd_kcontrol *kcontrol; > + struct snd_soc_dapm_context *dapm = w->dapm; > + struct

Re: [alsa-devel] [PATCH v3] ASOC: dapm: add code to configure dai link parameters

2014-08-12 Thread Mark Brown
On Tue, Aug 12, 2014 at 08:40:14PM +0530, Vinod Koul wrote: > On Mon, Aug 11, 2014 at 01:19:12PM +0100, Mark Brown wrote: > > That doesn't mean it's the best way to go in the end though - long term > > I expect to see us move away from that to something more data driven and > > DAPM integrated. R

Re: [alsa-devel] [PATCH v3] ASOC: dapm: add code to configure dai link parameters

2014-08-12 Thread Vinod Koul
On Mon, Aug 11, 2014 at 01:19:12PM +0100, Mark Brown wrote: > On Mon, Aug 11, 2014 at 11:54:29AM +0530, Vinod Koul wrote: > > On Thu, Aug 07, 2014 at 06:13:24PM +0100, Mark Brown wrote: > > > On Thu, Aug 07, 2014 at 07:46:05PM +0530, Vinod Koul wrote: > > > > > I was thinking about this as well a

Re: [alsa-devel] [PATCH v3] ASOC: dapm: add code to configure dai link parameters

2014-08-11 Thread Mark Brown
On Mon, Aug 11, 2014 at 11:54:29AM +0530, Vinod Koul wrote: > On Thu, Aug 07, 2014 at 06:13:24PM +0100, Mark Brown wrote: > > On Thu, Aug 07, 2014 at 07:46:05PM +0530, Vinod Koul wrote: > > > I was thinking about this as well a while back, wont it make sense to let > > > machine driver expose mean

Re: [alsa-devel] [PATCH v3] ASOC: dapm: add code to configure dai link parameters

2014-08-10 Thread Vinod Koul
On Thu, Aug 07, 2014 at 06:13:24PM +0100, Mark Brown wrote: > On Thu, Aug 07, 2014 at 07:46:05PM +0530, Vinod Koul wrote: > > > I was thinking about this as well a while back, wont it make sense to let > > machine driver expose meaningful controls for this. Just like we have fixup > > for back-end

Re: [alsa-devel] [PATCH v3] ASOC: dapm: add code to configure dai link parameters

2014-08-07 Thread Mark Brown
On Thu, Aug 07, 2014 at 07:46:05PM +0530, Vinod Koul wrote: > I was thinking about this as well a while back, wont it make sense to let > machine driver expose meaningful controls for this. Just like we have fixup > for back-ends, we can add fixups for codec-codec links and let machine tell > you

Re: [alsa-devel] [PATCH v3] ASOC: dapm: add code to configure dai link parameters

2014-08-07 Thread Vinod Koul
On Wed, Aug 06, 2014 at 05:16:26PM +0100, Nikesh Oswal wrote: > dai-link params for codec-codec links were fixed. The fixed > link between codec and another chip which may be another codec, > baseband, bluetooth codec etc may require run time configuaration > changes. This change provides an option

[PATCH v3] ASOC: dapm: add code to configure dai link parameters

2014-08-06 Thread Nikesh Oswal
dai-link params for codec-codec links were fixed. The fixed link between codec and another chip which may be another codec, baseband, bluetooth codec etc may require run time configuaration changes. This change provides an optional alsa control to select one of the params from a list of params. Si