From mboxrd@z Thu Jan 1 00:00:00 1970 From: Takashi Iwai Subject: Re: [PATCH 2/2] ALSA: Integrate control based jack reporting with core jack reporting Date: Wed, 22 Feb 2012 18:34:49 +0100 Message-ID: References: <4F3516E4.2080706@canonical.com> <20120210155003.GA11701@sirena.org.uk> <4F35413F.9000701@canonical.com> <20120210163946.GG6472@opensource.wolfsonmicro.com> <20120213154458.GB3494@opensource.wolfsonmicro.com> <20120213192309.GG3494@opensource.wolfsonmicro.com> <20120222171843.GA3265@opensource.wolfsonmicro.com> Mime-Version: 1.0 (generated by SEMI 1.14.6 - "Maruoka") Content-Type: text/plain; charset="us-ascii" Content-Transfer-Encoding: 7bit Return-path: Received: from mx2.suse.de (cantor2.suse.de [195.135.220.15]) by alsa0.perex.cz (Postfix) with ESMTP id 43BD6103B87 for ; Wed, 22 Feb 2012 18:34:52 +0100 (CET) In-Reply-To: <20120222171843.GA3265@opensource.wolfsonmicro.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: Mark Brown Cc: alsa-devel@alsa-project.org, patches@opensource.wolfsonmicro.com, David Henningsson List-Id: alsa-devel@alsa-project.org At Wed, 22 Feb 2012 17:18:44 +0000, Mark Brown wrote: > > On Wed, Feb 22, 2012 at 05:52:17PM +0100, Takashi Iwai wrote: > > Mark Brown wrote: > > > > Now that the kctl jacks are there I'm getting people asking me about it > > > often enough so I'd like to see it merged. > > > Yeah, if things were easy, I'd be happy to merge. > > But, judging from the situation, I see no big reason to hurry too > > much. > > I'm not sure what problems you see here - all the issues that are being > discussed here are about the kctl interface to applications, there's no > issues I can see with the in-kernel interfaces. First of all, we don't agree yet which naming rule to be applied. Since your patch assumes the case B, i.e. constant names corresponding only to the key type (HEADPHONE, etc), it's incompatible with the current implementation in HD-audio. That is, once when the patch is merged, the kctl expression will be forcibly to case B but without TLV yet, because kctls will be created automatically when the jack instance is created. In other words, if I merge your patch now, the only solution for HD-audio side for the time being is to disable CONFIG_SND_HDA_INPUT_JACK. That's why I hesitate to merge it now. And, it's why I prefer defining the naming rule at first, thus refining the implementation not to conflict with the existing one. Takashi