[PATCH 2/3] arm/dt: add very basic dts file for babbage board

[Shawn Guo]

Hi Grant,

On Mon, Feb 21, 2011 at 10:10:24AM -0700, Grant Likely wrote:
> On Mon, Feb 21, 2011 at 2:46 AM, Shawn Guo <shawn.guo at linaro.org> wrote:
[...]
> >> +       clocks {

Do we already have a clock binding specification, which is followed
by this example?  Or the properties given here still can be discussed
and open to change?

Also to support dynamically creating and registering 'struct clk' per
dt clock nodes, I need to add a few properties to map stuff like
parent 'struct clk', con_id of 'struct clk_lookup'.  Is there any
specification to follow, or I can draft it per the needs of my work?

> >> +               #address-cells = <1>;
> >> +               #size-cells = <0>;
> >> +
> >
> > Can we use hex value here to keep the consistency throughout the file?
> 
> For #address-cells and #size-cells use decimal integers.
> 
> >
> >> +               uart_clk0: uart at 0 {
> 
> @0 should only be specified if the node has a 'reg = <0>' property.
> In this case it doesn't so either 'reg' should be added, or '@0'
> should be removed.
> 
> >> +                       compatible = "clock";
> >> +                       clock-outputs = "imx-uart.0";
> >> +               };
> >> +
> >> +               uart_clk1: uart at 1{
> >
> > Can we put a space before "{" to keep the consistency throughout the file?
> 
> Yes, please.
> 
> >
> >> +                       compatible = "clock";
> >> +                       clock-outputs = "imx-uart.1";
> >> +               };
> >> +
> >> +               uart_clk2: uart at 2{
> >
> > ditto
> >
> >> +                       compatible = "clock";
> >> +                       clock-outputs = "imx-uart.2";
> >> +               };

-- 
Regards,
Shawn