From mboxrd@z Thu Jan 1 00:00:00 1970 From: Jean-Francois Moine Subject: Re: [PATCH v3 3/5] ASoC: tda998x: add DT documentation of the tda998x CODEC Date: Tue, 4 Feb 2014 20:02:39 +0100 Message-ID: <20140204200239.1baf309b@armhf> References: <8e4231b7a55802f58a14dd07ac5cd8b0babb1dce.1391274628.git.moinejf@free.fr> <20140204181213.GO22609@sirena.org.uk> Mime-Version: 1.0 Content-Type: text/plain; charset="utf-8" Content-Transfer-Encoding: base64 Return-path: In-Reply-To: <20140204181213.GO22609@sirena.org.uk> List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , Sender: dri-devel-bounces@lists.freedesktop.org Errors-To: dri-devel-bounces@lists.freedesktop.org To: Mark Brown Cc: devicetree@vger.kernel.org, alsa-devel@alsa-project.org, Russell King - ARM Linux , linux-kernel@vger.kernel.org, dri-devel@lists.freedesktop.org, linux-arm-kernel@lists.infradead.org List-Id: alsa-devel@alsa-project.org T24gVHVlLCA0IEZlYiAyMDE0IDE4OjEyOjEzICswMDAwCk1hcmsgQnJvd24gPGJyb29uaWVAa2Vy bmVsLm9yZz4gd3JvdGU6Cgo+IE9uIFNhdCwgRmViIDAxLCAyMDE0IGF0IDA1OjQ4OjQ5UE0gKzAx MDAsIEplYW4tRnJhbmNvaXMgTW9pbmUgd3JvdGU6Cj4gCj4gPiArCS0gY29tcGF0aWJsZTogbXVz dCBiZSAibnhwLHRkYTk5OHgtY29kZWMiLgo+IAo+IEl0J3Mgbm90IGNsZWFyIHRvIG1lIHdoeSB0 aGVyZSdzIGEgc2VwYXJhdGUgY29tcGF0aWJsZSBoZXJlIC0gYXMgZmFyIGFzCj4gSSBjYW4gc2Vl IHRoaXMgY2FuIG9ubHkgYXBwZWFyIGFzIHBhcnQgb2Ygb25lIG9mIHRoZXNlIGRldmljZXMgYW5k Cj4gdGhlcmUncyBubyBhZGRyZXNzaW5nIG9yIG90aGVyIGluZm9ybWF0aW9uIHRoYXQnZCBhY2Nv dW50IGZvciBjaGlwCj4gdmFyaWF0aW9uIHNvIEknZCBub3QgZXhwZWN0IHRvIG5lZWQgdG8gYmlu ZCB0aGlzIGluZGVwZW5kZW50bHkgb2YgdGhlCj4gcGFyZW50LgoKSWYgdGhlcmUgaXMgbm8gJ2Nv bXBhdGlibGUnLCB0aGUgQ09ERUMgbW9kdWxlIGlzIG5vdCBsb2FkZWQsIGFuZCwgd2hlbgp0aGUg bW9kdWxlIGlzIGluIHRoZSBjb3JlLCBubyBDT0RFQyBkZXZpY2UgY2FuIGJlIGNyZWF0ZWQgZnJv bSB0aGUgRFQuCgotLSAKS2VuIGFyIGMnaGVudGHDsQl8CSAgICAgICoqIEJyZWl6aCBoYSBMaW51 eCBhdGF2ISAqKgpKZWYJCXwJCWh0dHA6Ly9tb2luZWpmLmZyZWUuZnIvCl9fX19fX19fX19fX19f X19fX19fX19fX19fX19fX19fX19fX19fX19fX19fX19fCmRyaS1kZXZlbCBtYWlsaW5nIGxpc3QK ZHJpLWRldmVsQGxpc3RzLmZyZWVkZXNrdG9wLm9yZwpodHRwOi8vbGlzdHMuZnJlZWRlc2t0b3Au b3JnL21haWxtYW4vbGlzdGluZm8vZHJpLWRldmVsCg== From mboxrd@z Thu Jan 1 00:00:00 1970 From: moinejf@free.fr (Jean-Francois Moine) Date: Tue, 4 Feb 2014 20:02:39 +0100 Subject: [PATCH v3 3/5] ASoC: tda998x: add DT documentation of the tda998x CODEC In-Reply-To: <20140204181213.GO22609@sirena.org.uk> References: <8e4231b7a55802f58a14dd07ac5cd8b0babb1dce.1391274628.git.moinejf@free.fr> <20140204181213.GO22609@sirena.org.uk> Message-ID: <20140204200239.1baf309b@armhf> To: linux-arm-kernel@lists.infradead.org List-Id: linux-arm-kernel.lists.infradead.org On Tue, 4 Feb 2014 18:12:13 +0000 Mark Brown wrote: > On Sat, Feb 01, 2014 at 05:48:49PM +0100, Jean-Francois Moine wrote: > > > + - compatible: must be "nxp,tda998x-codec". > > It's not clear to me why there's a separate compatible here - as far as > I can see this can only appear as part of one of these devices and > there's no addressing or other information that'd account for chip > variation so I'd not expect to need to bind this independently of the > parent. If there is no 'compatible', the CODEC module is not loaded, and, when the module is in the core, no CODEC device can be created from the DT. -- Ken ar c'henta? | ** Breizh ha Linux atav! ** Jef | http://moinejf.free.fr/ From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S932288AbaBDTC1 (ORCPT ); Tue, 4 Feb 2014 14:02:27 -0500 Received: from smtp3-g21.free.fr ([212.27.42.3]:44914 "EHLO smtp3-g21.free.fr" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S932188AbaBDTCZ convert rfc822-to-8bit (ORCPT ); Tue, 4 Feb 2014 14:02:25 -0500 Date: Tue, 4 Feb 2014 20:02:39 +0100 From: Jean-Francois Moine To: Mark Brown Cc: alsa-devel@alsa-project.org, devicetree@vger.kernel.org, Dave Airlie , dri-devel@lists.freedesktop.org, linux-arm-kernel@lists.infradead.org, linux-kernel@vger.kernel.org, Rob Clark , Russell King - ARM Linux Subject: Re: [PATCH v3 3/5] ASoC: tda998x: add DT documentation of the tda998x CODEC Message-ID: <20140204200239.1baf309b@armhf> In-Reply-To: <20140204181213.GO22609@sirena.org.uk> References: <8e4231b7a55802f58a14dd07ac5cd8b0babb1dce.1391274628.git.moinejf@free.fr> <20140204181213.GO22609@sirena.org.uk> X-Mailer: Claws Mail 3.9.3 (GTK+ 2.24.22; arm-unknown-linux-gnueabihf) MIME-Version: 1.0 Content-Type: text/plain; charset=UTF-8 Content-Transfer-Encoding: 8BIT Sender: linux-kernel-owner@vger.kernel.org List-ID: X-Mailing-List: linux-kernel@vger.kernel.org On Tue, 4 Feb 2014 18:12:13 +0000 Mark Brown wrote: > On Sat, Feb 01, 2014 at 05:48:49PM +0100, Jean-Francois Moine wrote: > > > + - compatible: must be "nxp,tda998x-codec". > > It's not clear to me why there's a separate compatible here - as far as > I can see this can only appear as part of one of these devices and > there's no addressing or other information that'd account for chip > variation so I'd not expect to need to bind this independently of the > parent. If there is no 'compatible', the CODEC module is not loaded, and, when the module is in the core, no CODEC device can be created from the DT. -- Ken ar c'hentañ | ** Breizh ha Linux atav! ** Jef | http://moinejf.free.fr/