Re: [PATCH v3 3/3] dtc: Add support for variable sized cells

2011-09-27 Thread Anton Staaf
On Mon, Sep 26, 2011 at 5:55 PM, David Gibson wrote: > On Mon, Sep 26, 2011 at 10:15:48AM -0700, Anton Staaf wrote: >> On Sun, Sep 25, 2011 at 4:04 AM, David Gibson >> wrote: >> > On Fri, Sep 23, 2011 at 11:02:54AM -0700, Anton Staaf wrote: > [snip] >> > Uh, so, this is actually worse than what y

Re: [PATCH v3 3/3] dtc: Add support for variable sized cells

2011-09-27 Thread Anton Staaf
On Mon, Sep 26, 2011 at 5:58 PM, David Gibson wrote: > On Mon, Sep 26, 2011 at 01:48:31PM -0700, Anton Staaf wrote: >> On Sun, Sep 25, 2011 at 4:04 AM, David Gibson >> wrote: >> > On Fri, Sep 23, 2011 at 11:02:54AM -0700, Anton Staaf wrote: >> >        1) Exactly what to call the keyword.  /bits/

Re: [PATCH v3 3/3] dtc: Add support for variable sized cells

2011-09-26 Thread David Gibson
On Mon, Sep 26, 2011 at 01:48:31PM -0700, Anton Staaf wrote: > On Sun, Sep 25, 2011 at 4:04 AM, David Gibson > wrote: > > On Fri, Sep 23, 2011 at 11:02:54AM -0700, Anton Staaf wrote: > >        1) Exactly what to call the keyword.  /bits/ is better than > > /size/, but I'd still like to stew a bit

Re: [PATCH v3 3/3] dtc: Add support for variable sized cells

2011-09-26 Thread David Gibson
On Mon, Sep 26, 2011 at 10:15:48AM -0700, Anton Staaf wrote: > On Sun, Sep 25, 2011 at 4:04 AM, David Gibson > wrote: > > On Fri, Sep 23, 2011 at 11:02:54AM -0700, Anton Staaf wrote: [snip] > > Uh, so, this is actually worse than what you had before.  Remember > > print_error() just prints an erro

Re: [PATCH v3 3/3] dtc: Add support for variable sized cells

2011-09-26 Thread Anton Staaf
On Sun, Sep 25, 2011 at 4:04 AM, David Gibson wrote: > On Fri, Sep 23, 2011 at 11:02:54AM -0700, Anton Staaf wrote: >> Cells of size 8, 16, 32, and 64 bits are supported.  The new >> /bits/ syntax was selected so as to not pollute the reserved >> keyword space with uint8/uint16/... type names. >>

Re: [PATCH v3 3/3] dtc: Add support for variable sized cells

2011-09-26 Thread Anton Staaf
On Sun, Sep 25, 2011 at 4:04 AM, David Gibson wrote: > On Fri, Sep 23, 2011 at 11:02:54AM -0700, Anton Staaf wrote: >> Cells of size 8, 16, 32, and 64 bits are supported.  The new >> /bits/ syntax was selected so as to not pollute the reserved >> keyword space with uint8/uint16/... type names. >>

Re: [PATCH v3 3/3] dtc: Add support for variable sized cells

2011-09-25 Thread David Gibson
On Fri, Sep 23, 2011 at 11:02:54AM -0700, Anton Staaf wrote: > Cells of size 8, 16, 32, and 64 bits are supported. The new > /bits/ syntax was selected so as to not pollute the reserved > keyword space with uint8/uint16/... type names. > > With this patch the following property assignment: > >

[PATCH v3 3/3] dtc: Add support for variable sized cells

2011-09-23 Thread Anton Staaf
Cells of size 8, 16, 32, and 64 bits are supported. The new /bits/ syntax was selected so as to not pollute the reserved keyword space with uint8/uint16/... type names. With this patch the following property assignment: property = /bits/ 16 <0x1234 0x5678 0x0 0x>; is equivalent to: