David Gibson wrote:
> And here's a revised version. This now also handles recursive
> iteration and iteration across nodes without respect to depth. I've
> removed the for_each() macros for the time being, because they were
> making my brain hurt, but I'm still contemplating bringing them back.
> Several libfdt functions are now implemented using the new iterator,
> so this ends up as a code-size-reducing patch.
>
> I'm pretty happy with the basic outline of this now, although the
> names and details might want a bit of polish still.
Can we get this merged?
> +int _fdt_next_node(const void *fdt, int offset, int *depth)
> +{
This is a public function; why the underscore?
-Scott
_______________________________________________
Linuxppc-dev mailing list
Linuxppc-dev@ozlabs.org
https://ozlabs.org/mailman/listinfo/linuxppc-dev
