From mboxrd@z Thu Jan 1 00:00:00 1970 From: Lee Jones Subject: Re: [PATCH 17/18] ARM: ux500: Add all encompassing Sound node to DB8500 Device Tree Date: Mon, 30 Jul 2012 15:30:15 +0100 Message-ID: <50169A77.3050402@linaro.org> References: <1343393162-11938-1-git-send-email-lee.jones@linaro.org> <1343393162-11938-18-git-send-email-lee.jones@linaro.org> <20120729205227.GL4384@opensource.wolfsonmicro.com> <50163936.1050901@linaro.org> <20120730134220.GD4468@opensource.wolfsonmicro.com> Mime-Version: 1.0 Content-Type: text/plain; charset="utf-8"; Format="flowed" Content-Transfer-Encoding: base64 Return-path: Received: from mail-ey0-f179.google.com (mail-ey0-f179.google.com [209.85.215.179]) by alsa0.perex.cz (Postfix) with ESMTP id 34F25265322 for ; Mon, 30 Jul 2012 16:30:16 +0200 (CEST) Received: by eaaa13 with SMTP id a13so838194eaa.38 for ; Mon, 30 Jul 2012 07:30:18 -0700 (PDT) In-Reply-To: <20120730134220.GD4468@opensource.wolfsonmicro.com> 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: Mark Brown Cc: ola.o.lilja@stericsson.com, alsa-devel@alsa-project.org, linus.walleij@stericsson.com, arnd@arndb.de, olalilja@yahoo.se, linux-kernel@vger.kernel.org, STEricsson_nomadik_linux@list.st.com, lrg@ti.com, linux-arm-kernel@lists.infradead.org List-Id: alsa-devel@alsa-project.org T24gMzAvMDcvMTIgMTQ6NDIsIE1hcmsgQnJvd24gd3JvdGU6Cj4gT24gTW9uLCBKdWwgMzAsIDIw MTIgYXQgMDg6MzU6MThBTSArMDEwMCwgTGVlIEpvbmVzIHdyb3RlOgo+PiBPbiAyOS8wNy8xMiAy MTo1MiwgTWFyayBCcm93biB3cm90ZToKPgo+Pj4gLi4udGhpcyBpcyBjbGVhcmx5IHRoZSBib2Fy ZC1zcGVjaWZpYyBzZXR1cCBmb3IgdGhlIHN5c3RlbSwgc2hvd2luZyBob3cKPj4+IHRoZSBDUFUg YW5kIENPREVDIGFyZSBob29rZWQgdXAuICBUaGlzIHNob3VsZCB0aGVyZWZvcmUgYmUgaW4gdGhl IERUIGZvcgo+Pj4gd2hhdGV2ZXIgYm9hcmQgdGhpcyBpcyB5b3UncmUgd29ya2luZyBvbi4KPgo+ PiBJIGRvbid0IHRoaW5rIGl0J3MgYm9hcmQgc3BlY2lmaWMuIEkgYmVsaWV2ZSB0aGlzIGlzIHRo ZSBzZXR1cCBvbgo+PiBhbGwgZGI4NTAwIHBsYXRmb3Jtcy4gSSdtIGhhcHB5IHRvIGJlIHRvbGQg SSdtIHdyb25nIGhlcmUsIGluIHdoaWNoCj4+IGNhc2UgSSBjYW4gbW92ZSBpdCB0byB0aGUgYm9h cmQgc3BlY2lmaWMgZmlsZXMuCj4KPiBJIHRoaW5rIHRoYXQncyBtb3JlIGEgc2lnbiB0aGF0IGFs bCB0aGUgcGxhdGZvcm1zIHlvdSd2ZSBzZWVuIGFyZSBqdXN0Cj4gY3V0J24ncGFzdGVkIGZyb20g dGhlIHJlZmVyZW5jZSBkZXNpZ24gdGhhbiBhbnl0aGluZyBlbHNlLiAgSWYgdGhlIHRoaW5nCj4g eW91J3JlIGRlc2NyaWJpbmcgaXMgb24gdGhlIFBDQiBpdCdzIGdlbmVyYWxseSBib2FyZCBzcGVj aWZpYy4uLgoKSSdsbCBzcGxpdCBpdCBvdXQgaW50byB0aGUgYm9hcmQgZmlsZShzKSBpZiBpdCBt YWtlcyB5b3UgaGFwcHkuIEknZCAKcmF0aGVyIGp1c3QgZ2V0IGl0IGluIGFuZCBtb3ZlIG9uIHRv IHNvbWV0aGluZyBtb3JlIHByb2R1Y3RpdmUuIFdlIGNhbiAKY2hhbmdlIGl0IHJldHJvc3BlY3Rp dmVseSBpZiBJJ20gdG9sZCBieSBzb21lb25lIG1vcmUga25vd2xlZGdlYWJsZSB0aGF0IAppdCdz IDEwMCUgY29tbW9uIGFjcm9zcyBhbGwgcGxhdGZvcm1zLgoKLS0gCkxlZSBKb25lcwpMaW5hcm8g U1QtRXJpY3Nzb24gTGFuZGluZyBUZWFtIExlYWQKTGluYXJvLm9yZyDilIIgT3BlbiBzb3VyY2Ug c29mdHdhcmUgZm9yIEFSTSBTb0NzCkZvbGxvdyBMaW5hcm86IEZhY2Vib29rIHwgVHdpdHRlciB8 IEJsb2cKX19fX19fX19fX19fX19fX19fX19fX19fX19fX19fX19fX19fX19fX19fX19fX18KQWxz YS1kZXZlbCBtYWlsaW5nIGxpc3QKQWxzYS1kZXZlbEBhbHNhLXByb2plY3Qub3JnCmh0dHA6Ly9t YWlsbWFuLmFsc2EtcHJvamVjdC5vcmcvbWFpbG1hbi9saXN0aW5mby9hbHNhLWRldmVsCg== From mboxrd@z Thu Jan 1 00:00:00 1970 From: lee.jones@linaro.org (Lee Jones) Date: Mon, 30 Jul 2012 15:30:15 +0100 Subject: [PATCH 17/18] ARM: ux500: Add all encompassing Sound node to DB8500 Device Tree In-Reply-To: <20120730134220.GD4468@opensource.wolfsonmicro.com> References: <1343393162-11938-1-git-send-email-lee.jones@linaro.org> <1343393162-11938-18-git-send-email-lee.jones@linaro.org> <20120729205227.GL4384@opensource.wolfsonmicro.com> <50163936.1050901@linaro.org> <20120730134220.GD4468@opensource.wolfsonmicro.com> Message-ID: <50169A77.3050402@linaro.org> To: linux-arm-kernel@lists.infradead.org List-Id: linux-arm-kernel.lists.infradead.org On 30/07/12 14:42, Mark Brown wrote: > On Mon, Jul 30, 2012 at 08:35:18AM +0100, Lee Jones wrote: >> On 29/07/12 21:52, Mark Brown wrote: > >>> ...this is clearly the board-specific setup for the system, showing how >>> the CPU and CODEC are hooked up. This should therefore be in the DT for >>> whatever board this is you're working on. > >> I don't think it's board specific. I believe this is the setup on >> all db8500 platforms. I'm happy to be told I'm wrong here, in which >> case I can move it to the board specific files. > > I think that's more a sign that all the platforms you've seen are just > cut'n'pasted from the reference design than anything else. If the thing > you're describing is on the PCB it's generally board specific... I'll split it out into the board file(s) if it makes you happy. I'd rather just get it in and move on to something more productive. We can change it retrospectively if I'm told by someone more knowledgeable that it's 100% common across all platforms. -- Lee Jones Linaro ST-Ericsson Landing Team Lead Linaro.org ? Open source software for ARM SoCs Follow Linaro: Facebook | Twitter | Blog From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S1754657Ab2G3OaU (ORCPT ); Mon, 30 Jul 2012 10:30:20 -0400 Received: from mail-ey0-f174.google.com ([209.85.215.174]:54927 "EHLO mail-ey0-f174.google.com" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S1754217Ab2G3OaT (ORCPT ); Mon, 30 Jul 2012 10:30:19 -0400 Message-ID: <50169A77.3050402@linaro.org> Date: Mon, 30 Jul 2012 15:30:15 +0100 From: Lee Jones User-Agent: Mozilla/5.0 (X11; Linux x86_64; rv:14.0) Gecko/20120714 Thunderbird/14.0 MIME-Version: 1.0 To: Mark Brown CC: linux-arm-kernel@lists.infradead.org, linux-kernel@vger.kernel.org, STEricsson_nomadik_linux@list.st.com, linus.walleij@stericsson.com, arnd@arndb.de, olalilja@yahoo.se, ola.o.lilja@stericsson.com, alsa-devel@alsa-project.org, lrg@ti.com Subject: Re: [PATCH 17/18] ARM: ux500: Add all encompassing Sound node to DB8500 Device Tree References: <1343393162-11938-1-git-send-email-lee.jones@linaro.org> <1343393162-11938-18-git-send-email-lee.jones@linaro.org> <20120729205227.GL4384@opensource.wolfsonmicro.com> <50163936.1050901@linaro.org> <20120730134220.GD4468@opensource.wolfsonmicro.com> In-Reply-To: <20120730134220.GD4468@opensource.wolfsonmicro.com> Content-Type: text/plain; charset=UTF-8; format=flowed Content-Transfer-Encoding: 8bit Sender: linux-kernel-owner@vger.kernel.org List-ID: X-Mailing-List: linux-kernel@vger.kernel.org On 30/07/12 14:42, Mark Brown wrote: > On Mon, Jul 30, 2012 at 08:35:18AM +0100, Lee Jones wrote: >> On 29/07/12 21:52, Mark Brown wrote: > >>> ...this is clearly the board-specific setup for the system, showing how >>> the CPU and CODEC are hooked up. This should therefore be in the DT for >>> whatever board this is you're working on. > >> I don't think it's board specific. I believe this is the setup on >> all db8500 platforms. I'm happy to be told I'm wrong here, in which >> case I can move it to the board specific files. > > I think that's more a sign that all the platforms you've seen are just > cut'n'pasted from the reference design than anything else. If the thing > you're describing is on the PCB it's generally board specific... I'll split it out into the board file(s) if it makes you happy. I'd rather just get it in and move on to something more productive. We can change it retrospectively if I'm told by someone more knowledgeable that it's 100% common across all platforms. -- Lee Jones Linaro ST-Ericsson Landing Team Lead Linaro.org │ Open source software for ARM SoCs Follow Linaro: Facebook | Twitter | Blog