[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

Re: [Xen-devel] [PATCH 2/8] xen/arm: Retrieve the correct number of cells when building dom0 DT



On Mon, 2015-09-28 at 15:59 +0100, Julien Grall wrote:
> Hi Ian,
> 
> On 25/09/15 17:01, Ian Campbell wrote:
> > On Tue, 2015-09-22 at 18:47 +0100, Julien Grall wrote:
> > > The function dt_n_*_cells will retrieve the number of cells for a
> > > given
> > > node. Those numbers may not be correct to use for the child "reg"
> > > property if the parent is passed.
> > 
> > I think a clearer way to express this is that the functions return the
> > number of cells to use for a reg property in the given node, and so
> > they
> > will naturally not necessarily return the right number if you give them
> > some other node (whether that's the parent or some other node
> > entirely).
> 
> I will rework this part into:
> 
> "The function dt_n_*_cells return the number of cells to use for a reg
> property in a given node. So those numbers won't be correct if the
> parent of a give node is passed."

                  ^n

> 
> > 
> > > This is fine today because the parent is always the root node which
> > > means there is no upper parent.
> > > 
> > > Introduce new helpers dt_child_n_*_cells to retrieve the number of
> > > cells for the address and size that can be used to create the "reg"
> > > property of a child of a given parent.
> > 
> > "of the immediate child of a given"
> > 
> > maybe?
> 
> I'm not sure to see why "immediate" is necessary here. A child is always
> immediate, otherwise it would be grandchild or else.

Sadly it's not as unambiguous as that in this kind of context, child could
be taken to mean "somewhere below".


Ian.

_______________________________________________
Xen-devel mailing list
Xen-devel@xxxxxxxxxxxxx
http://lists.xen.org/xen-devel


 


Rackspace

Lists.xenproject.org is hosted with RackSpace, monitoring our
servers 24x7x365 and backed by RackSpace's Fanatical Support®.