Re: [PATCH 2/4] PowerPC: 440GRx Rainier DTS.
From: David Gibson <hidden>
Date: 2007-10-31 02:56:13
On Tue, Oct 30, 2007 at 09:09:17PM -0500, Josh Boyer wrote:
On Tue, 30 Oct 2007 20:56:51 -0500 Olof Johansson [off-list ref] wrote:quoted
On Wed, Oct 31, 2007 at 10:08:05AM +1100, David Gibson wrote:quoted
On Tue, Oct 30, 2007 at 07:56:50PM +0300, Valentine Barshak wrote:quoted
PowerPC 440GRx Rainier DTS.[snip]quoted
+ SDRAM0: sdram { + device_type = "memory-controller";How many times do we need to say it... Don't make up random device_type values. This does not belong here.Maybe there should be something like checkpatch.pl that warns about these kinds of things so people can check for it without getting flamed first. :-)
I'd be gentler; except that I know Valentine has been active on this list recently, so has almost certainly seen similar comments before.
That's actually a decent idea. We could even have this thing that takes DTS files and processes them... oh wait. So why can't we make a whitelist of "allowed" device_types in DTC and make it whine about anything outside of that?
Well, that sort of thing is the idea for dtc to check, when it has checks enabled anyway. Getting it so that it doesn't have too many false positives is the tricky bit, as we've seen with some of the existing checks (where "too many" is "almost any"). Patches welcome... -- David Gibson | I'll have my music baroque, and my code david AT gibson.dropbear.id.au | minimalist, thank you. NOT _the_ _other_ | _way_ _around_! http://www.ozlabs.org/~dgibson