Re: [RFC PATCH 64/77] dtc: Add support for references by path involving orphan nodes

On Thu, 15 Jan 2026 18:01:39 +1100
David Gibson <[email protected]> wrote:

> On Mon, Jan 12, 2026 at 03:19:54PM +0100, Herve Codina wrote:
> > Referencing a sub-node from an orphan node using a path is needed.
> > 
> > Indeed, using the following snippet:
> > --- 8< ---
> > /addon/;
> > 
> > &node1 {
> > 	subnode {
> > 		foo-phandle = <&foo_label>;
> > 	};
> > };
> > 
> > &node2 {
> > 	foo_label: foo {
> > 		prop = <1>;
> > 	};
> > };
> > --- 8< ---
> > 
> > Even if node2 is an orphan node, foo is a local node. foo-phandle
> > references the foo node using a label.  
> 
> Another option would be to eliminate the idea of local references, and
> require a symbol be attached to things that you want to reference by
> label.

Hum, new kind of references.

We have reference by phandle to local nodes. Reference by symbol for
external nodes (i.e. nodes not present in current dtb).

Now new kind of reference for node available in the current dtb but
in a different tree (orphan tree).

For that we need to:
  - Mark the phandle value in the property as a cross-tree phandle
    reference
  - Add the symbol label in the referenced node.

When the addon is applied, this new kind of reference need to be taken
into account in a new way:
  - The phandle value in the referenced node need to be updated in the
    same way as all other phandle value in nodes to avoid collisions.
  - The cross-tree reference needs to be resolved.

This adds an unneeded complexity.

IMHO, we shouldn't eliminate local references.

We need to reference all possible local nodes by path even if cross-tree
due to orphan tree is involved.

Best regards,
Hervé