From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S1754466Ab1BGRty (ORCPT ); Mon, 7 Feb 2011 12:49:54 -0500 Received: from opensource.wolfsonmicro.com ([80.75.67.52]:35713 "EHLO opensource2.wolfsonmicro.com" rhost-flags-OK-OK-OK-FAIL) by vger.kernel.org with ESMTP id S1754310Ab1BGRtx (ORCPT ); Mon, 7 Feb 2011 12:49:53 -0500 Date: Mon, 7 Feb 2011 17:49:51 +0000 From: Mark Brown To: Lars-Peter Clausen Cc: Liam Girdwood , alsa-devel@alsa-project.org, linux-kernel@vger.kernel.org Subject: Re: [PATCH 6/7] ASoC: Samsung: neo1973_gta02: Fix bluetooth DAI registration Message-ID: <20110207174950.GQ10564@opensource.wolfsonmicro.com> References: <1297033463-14577-1-git-send-email-lars@metafoo.de> <1297033463-14577-6-git-send-email-lars@metafoo.de> <20110207115906.GD10564@opensource.wolfsonmicro.com> <4D5023A4.3020209@metafoo.de> <20110207170213.GP10564@opensource.wolfsonmicro.com> <4D502DBF.3050205@metafoo.de> MIME-Version: 1.0 Content-Type: text/plain; charset=us-ascii Content-Disposition: inline In-Reply-To: <4D502DBF.3050205@metafoo.de> X-Cookie: You are fairminded, just and loving. User-Agent: Mutt/1.5.20 (2009-06-14) Sender: linux-kernel-owner@vger.kernel.org List-ID: X-Mailing-List: linux-kernel@vger.kernel.org As I said when replying to your previous mail and I'm sure some earlier ones too you need to fix your MUA to word wrap at less than 80 columns. I've yet again reflowed your text so that it's legible. On Mon, Feb 07, 2011 at 06:37:03PM +0100, Lars-Peter Clausen wrote: > On 02/07/2011 06:02 PM, Mark Brown wrote: > > If you think the core isn't behaving helpfully the core should be > > changed. This is part of how APIs evolve to be maximally useful. > As I see it the problem is that we have a deviceless dai and there is > not really a way to register a dai without a device. But I have no > idea right now how to change the core to make it "behave helpfully". You don't like the names the core is coming up with. Make better ones. > And in a sense snd_soc_register_dais seems to be the right thing to > use for now, because the sound card as a whole has multiple dais they > just not all registered at the same time. The card is only registering one DAI, all the other DAIs are attached to other devices in the system. > > To be honest it's not massively obvious that we shouldn't just be taking > > the name of the device here, either using a device to represent the > > modem > Seriously? I don't see how adding a dummy device wouldn't be "bodging > around the core". Especially if using snd_soc_register_dais is. The bluetooth chip is an actual device which I can point to on the board and schematic, having a struct device to represent a device that's actually present doesn't seem like a great leap. > > or registering the card using snd_soc_register_machine() and using > > a more meaningful name for the card seems like a sensible approach here. > Well, if were using snd_soc_register_machine to give the card a > different name the bluetooth-dai would still be named after the card, > wouldn't it? So there is no improvement here as to giving the dai a > meaningful name. It does mean it's named after the board. From mboxrd@z Thu Jan 1 00:00:00 1970 From: Mark Brown Subject: Re: [PATCH 6/7] ASoC: Samsung: neo1973_gta02: Fix bluetooth DAI registration Date: Mon, 7 Feb 2011 17:49:51 +0000 Message-ID: <20110207174950.GQ10564@opensource.wolfsonmicro.com> References: <1297033463-14577-1-git-send-email-lars@metafoo.de> <1297033463-14577-6-git-send-email-lars@metafoo.de> <20110207115906.GD10564@opensource.wolfsonmicro.com> <4D5023A4.3020209@metafoo.de> <20110207170213.GP10564@opensource.wolfsonmicro.com> <4D502DBF.3050205@metafoo.de> Mime-Version: 1.0 Content-Type: text/plain; charset="us-ascii" Content-Transfer-Encoding: 7bit Return-path: Received: from opensource2.wolfsonmicro.com (opensource.wolfsonmicro.com [80.75.67.52]) by alsa0.perex.cz (Postfix) with ESMTP id 9D943244C7 for ; Mon, 7 Feb 2011 18:49:53 +0100 (CET) Content-Disposition: inline In-Reply-To: <4D502DBF.3050205@metafoo.de> 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: Lars-Peter Clausen Cc: alsa-devel@alsa-project.org, linux-kernel@vger.kernel.org, Liam Girdwood List-Id: alsa-devel@alsa-project.org As I said when replying to your previous mail and I'm sure some earlier ones too you need to fix your MUA to word wrap at less than 80 columns. I've yet again reflowed your text so that it's legible. On Mon, Feb 07, 2011 at 06:37:03PM +0100, Lars-Peter Clausen wrote: > On 02/07/2011 06:02 PM, Mark Brown wrote: > > If you think the core isn't behaving helpfully the core should be > > changed. This is part of how APIs evolve to be maximally useful. > As I see it the problem is that we have a deviceless dai and there is > not really a way to register a dai without a device. But I have no > idea right now how to change the core to make it "behave helpfully". You don't like the names the core is coming up with. Make better ones. > And in a sense snd_soc_register_dais seems to be the right thing to > use for now, because the sound card as a whole has multiple dais they > just not all registered at the same time. The card is only registering one DAI, all the other DAIs are attached to other devices in the system. > > To be honest it's not massively obvious that we shouldn't just be taking > > the name of the device here, either using a device to represent the > > modem > Seriously? I don't see how adding a dummy device wouldn't be "bodging > around the core". Especially if using snd_soc_register_dais is. The bluetooth chip is an actual device which I can point to on the board and schematic, having a struct device to represent a device that's actually present doesn't seem like a great leap. > > or registering the card using snd_soc_register_machine() and using > > a more meaningful name for the card seems like a sensible approach here. > Well, if were using snd_soc_register_machine to give the card a > different name the bluetooth-dai would still be named after the card, > wouldn't it? So there is no improvement here as to giving the dai a > meaningful name. It does mean it's named after the board.