From mboxrd@z Thu Jan 1 00:00:00 1970 From: Liam Girdwood Subject: Re: [PATCH] ASoC: Fix dapm_is_shared_kcontrol so everything isn't shared Date: Fri, 27 May 2011 10:19:13 +0100 Message-ID: <4DDF6C91.1010807@ti.com> References: <1306425453-20753-1-git-send-email-swarren@nvidia.com> Mime-Version: 1.0 Content-Type: text/plain; charset="us-ascii" Content-Transfer-Encoding: 7bit Return-path: Received: from bear.ext.ti.com (bear.ext.ti.com [192.94.94.41]) by alsa0.perex.cz (Postfix) with ESMTP id 5F490243BD for ; Fri, 27 May 2011 11:19:16 +0200 (CEST) In-Reply-To: <1306425453-20753-1-git-send-email-swarren@nvidia.com> List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , Sender: alsa-devel-bounces@alsa-project.org Errors-To: alsa-devel-bounces@alsa-project.org To: Stephen Warren Cc: "alsa-devel@alsa-project.org" , "broonie@opensource.wolfsonmicro.com" List-Id: alsa-devel@alsa-project.org On 26/05/11 16:57, Stephen Warren wrote: > Commit af46800 ("ASoC: Implement mux control sharing") introduced > function dapm_is_shared_kcontrol. > > When this function returns true, the naming of DAPM controls is derived > from the kcontrol_new. Otherwise, the name comes from the widget (and > possibly a widget's naming prefix). > > A bug in the implementation of dapm_is_shared_kcontrol made it return 1 > in all cases. Hence, that commit caused a change in control naming for > all controls instead of just shared controls. > > Specifically, a control is always considered shared because it is always > compared against itself. Solve this by never comparing against the widget > containing the control being created. > > Equally, controls should never be shared between DAPM contexts; when the > same codec is instantiated multiple times, the same kcontrol_new will be > used. However, the control should no be shared between the multiple > instances. > > I tested that with the Tegra WM8903 driver: > * Shared is now mostly 0 as expected, and sometimes 1. > * The expected controls are still generated after this change. > > However, I don't have any systems that have a widget/control naming > prefix, so I can't test that aspect. > > Thanks for Jarkko Nikula for pointing out how to fix this. > > v2: > * Don't share across DAPM contexts. > * Changed to Tested-by tag below for Jarkko. > > Reported-by: Liam Girdwood > Tested-by: Jarkko Nikula > Signed-off-by: Stephen Warren > --- Acked-by: Liam Girdwood