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

[Madhavan Srinivasan]

On Friday 12 May 2017 09:06 AM, Stewart Smith wrote:
> 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.

Thanks.

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

Maddy