<p dir="ltr"><br>
On Jan 15, 2013 2:06 AM, "Tony Lindgren" <<a href="mailto:tony@atomide.com">tony@atomide.com</a>> wrote:<br>
><br>
> * Ezequiel Garcia <<a href="mailto:elezegarcia@gmail.com">elezegarcia@gmail.com</a>> [121223 13:49]:<br>
> > On Fri, Dec 14, 2012 at 7:36 AM, Daniel Mack <<a href="mailto:zonque@gmail.com">zonque@gmail.com</a>> wrote:<br>
> > > +<br>
> > > +Example for an AM33xx board:<br>
> > > +<br>
> > > +       gpmc: gpmc@50000000 {<br>
> > > +               compatible = "ti,am3352-gpmc";<br>
> > > +               ti,hwmods = "gpmc";<br>
> > > +               reg = <0x50000000 0x1000000>;<br>
> > > +               interrupts = <100>;<br>
> > > +               gpmc,num-cs = <8>;<br>
> > > +               gpmc,num-waitpins = <2>;<br>
> > > +               #address-cells = <2>;<br>
> > > +               #size-cells = <1>;<br>
> > > +               ranges = <0 0 0x08000000 0x2000>;       /* CS0: NAND */<br>
> > > +<br>
> > > +               nand@0,0 {<br>
> > > +                       reg = <0 0 0>; /* CS0, offset 0 */<br>
> ><br>
> > I'm a bit confused by this: what are the other two values in "reg"?<br>
> > I see you've only added a binding for CS.<br>
> ><br>
> > I've extended a bit on your work and added a binding to enable OneNAND<br>
> > device on my IGEP board.<br>
> ><br>
> > I might send some patches in case anyone wants to give it a try.<br>
><br>
> Daniel, should this be updated to just pass the CS?</p>
<p dir="ltr">No, as Rob pointed out earlier in a thread about this topic, the 'ranges' feature will help doing the math for the offset calculation eventually, so we need to pass all three values.</p>
<p dir="ltr">Regards,<br>
Daniel<br>
</p>