linux-kernel - Re: [PATCH 17/18] ARM: ux500: Add all encompassing Sound node to DB8500 Device Tree

lists.openwall.net		lists / announce owl-users owl-dev john-users john-dev passwdqc-users yescrypt popa3d-users / oss-security kernel-hardening musl sabotage tlsify passwords / crypt-dev xvendor / Bugtraq Full-Disclosure linux-kernel linux-netdev linux-ext4 linux-hardening linux-cve-announce PHC
Open Source and information security mailing list archives

Hash Suite: Windows password security audit tool. GUI, reports in PDF.

[<prev] [next>] [<thread-prev] [thread-next>] [day] [month] [year] [list]

Date:	Mon, 30 Jul 2012 15:30:15 +0100
From:	Lee Jones <lee.jones@...aro.org>
To:	Mark Brown <broonie@...nsource.wolfsonmicro.com>
CC:	linux-arm-kernel@...ts.infradead.org, linux-kernel@...r.kernel.org,
	STEricsson_nomadik_linux@...t.st.com, linus.walleij@...ricsson.com,
	arnd@...db.de, olalilja@...oo.se, ola.o.lilja@...ricsson.com,
	alsa-devel@...a-project.org, lrg@...com
Subject: Re: [PATCH 17/18] ARM: ux500: Add all encompassing Sound node to
 DB8500 Device Tree

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
--
To unsubscribe from this list: send the line "unsubscribe linux-kernel" in
the body of a message to majordomo@...r.kernel.org
More majordomo info at  http://vger.kernel.org/majordomo-info.html
Please read the FAQ at  http://www.tux.org/lkml/