[alsa-devel] [RFC 19/19] ASoC: Intel: mrfld: add the DSP mixers

[Vinod Koul]

On Sat, Jun 14, 2014 at 05:39:09PM +0200, Lars-Peter Clausen wrote:
> On 06/13/2014 02:34 PM, Vinod Koul wrote:
> >The is RFC patch for adding the platform mixer controls. This requires the
> >dapm_set-get to be exported. Or changes required after component series is
> >merged
> 
> You are not using dapm_kcontrol_get_value() in this patch and
> without that calling dapm_kcontrol_set_value() is pretty pointless
> as the value is never read again. But it is still a good idea to use
> the generic controls rather than having a custom copy&pasted
> version.

Hey Lars,

Circling back on this one..

We tried implementing based on the feedback. One question though remains.

We were able to remove the get/put widget handlers. Now only the callback is
present. With this when widget is powered up for a mixer, we need to know the value
of the mixer so that we can inform DSP using IPC.

Now potentially looks like we should be able to remove the register file as well,
but only problem would be how to get the mixer control value (how many and which
inputs are on/off and tell DSP). This would need me to export the dapm_get
function, is that approach fine, or do we have any other way to get the value of
this in driver.

-- 
~Vinod