[Skiboot] [PATCH v11 05/11] dt: Add phandle fixup helpers

Stewart Smith stewart at linux.vnet.ibm.com
Fri May 12 13:36:23 AEST 2017


Madhavan Srinivasan <maddy at linux.vnet.ibm.com> writes:
> New subtree needs would have "phandle"
> already assigned to some of it's nodes.
> If this new substree is attached to main dt,
> without fixing the phandles, would lead to
> phandle duplicate errors.
>
> Also, new subtree could have "properties"
> refering to the phandle. So these "properties"
> should also get updated accordingly when fixing
> the "phandle".
>
> New helper dt_adjust_subtree_phandle() added to
> core/device.c to do both. It does phandle fixup
> with a single pass through the subtree.

Great, this looks good.

Please add tests to core/test though, as there's no reason we can't
verify this code in unit tests.

-- 
Stewart Smith
OPAL Architect, IBM.



More information about the Skiboot mailing list