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

David Gibson david at gibson.dropbear.id.au
Fri Sep 23 10:14:53 EST 2011


On Thu, Sep 22, 2011 at 11:02:59AM -0700, Anton Staaf wrote:
> On Wed, Sep 21, 2011 at 7:42 PM, David Gibson
> <david at gibson.dropbear.id.au> wrote:
> > On Wed, Sep 21, 2011 at 01:42:11PM -0700, Anton Staaf wrote:
> >> Cells of size 8, 16, 32, and 64 bits are supported.  The new
> >> /size/ 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 = /size/ 16 <0x1234 0x5678 0x0 0xffff>;
> >
> > I'm now thinking that just plain "/size/" is maybe a bit too generic
> > for this keyword.  But I'm not sure what would be better.  Maybe
> > /cellsize/ or /intsize/ or /arraysize/.
> 
> Also /bits/ or /cellbits/ might make good choices because the size is in
> bits and not in bytes.

True; they are better ideas.

>  I'm not fond of /arraysize/ because we are
> specifying the size of each element in the array, not the array
> itself.

Yeah, good point.

-- 
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


More information about the devicetree-discuss mailing list