On Mon, Jul 30, 2012 at 10:44:29AM -0500, Rob Herring wrote: > On 07/27/2012 07:05 AM, Alexandre Courbot wrote: > > + power-on-sequence { > > + regulator@0 { > > + id = "power"; > > + enable; > What do this mean? Isn't this implied for a regulator? I assume you might have some sequences which need some things to be turned off for some reason; it at least seems to be something you'd want to design for. > This looks like you designed the platform_data structs first and then > came up with device nodes to mirror the struct. Judging by most of the DT I review this seems idiomatic :) -- To unsubscribe from this list: send the line "unsubscribe linux-tegra" in the body of a message to majordomo@xxxxxxxxxxxxxxx More majordomo info at http://vger.kernel.org/majordomo-info.html