From mboxrd@z Thu Jan 1 00:00:00 1970 From: Vinod Koul Subject: Re: [RFC 19/19] ASoC: Intel: mrfld: add the DSP mixers Date: Fri, 4 Jul 2014 10:16:51 +0530 Message-ID: <20140704044651.GT2296@intel.com> References: <1402662848-24534-1-git-send-email-vinod.koul@intel.com> <1402662848-24534-20-git-send-email-vinod.koul@intel.com> <539C6C9D.6020004@metafoo.de> Mime-Version: 1.0 Content-Type: text/plain; charset="us-ascii" Content-Transfer-Encoding: 7bit Return-path: Received: from mga01.intel.com (mga01.intel.com [192.55.52.88]) by alsa0.perex.cz (Postfix) with ESMTP id E11BB265977 for ; Fri, 4 Jul 2014 06:48:16 +0200 (CEST) Content-Disposition: inline In-Reply-To: <539C6C9D.6020004@metafoo.de> List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , Errors-To: alsa-devel-bounces@alsa-project.org Sender: alsa-devel-bounces@alsa-project.org To: Lars-Peter Clausen Cc: alsa-devel@alsa-project.org, broonie@kernel.org, subhransu.s.prusty@intel.com, lgirdwood@gmail.com List-Id: alsa-devel@alsa-project.org 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