[DTC PATCH] libfdt: Add ft_get_next_node(), ft_get_next_prop(), and ft_getprop_offset().

Stephen Rothwell sfr at canb.auug.org.au
Tue Jan 15 09:54:34 EST 2008


On Mon, 14 Jan 2008 16:44:33 -0600 Scott Wood <scottwood at freescale.com> wrote:
>
> Stephen Rothwell wrote:
> >> +++ b/libfdt/libfdt.h
> > 
> >> +#define FDT_ERR_BADDEPTH	8
> > 
> > Wouldn't it have been less intrusive to just use the next error number
> > rather than inserting this here?
> 
> Yes, but then either the order in errtable[] wouldn't match the order in 
> the header file, or the error type grouping would be broken.
> 
> If we want to maintain such a grouping, we should probably leave some 
> number space between the groups.

OK.

> "pointer to a string" could be interpreted as "char *", not "char **".

Yeah, it has always been a bit ambiguous.

> I'll fix the others; thanks for pointing them out.

No worries.
-- 
Cheers,
Stephen Rothwell                    sfr at canb.auug.org.au
http://www.canb.auug.org.au/~sfr/
-------------- next part --------------
A non-text attachment was scrubbed...
Name: not available
Type: application/pgp-signature
Size: 189 bytes
Desc: not available
URL: <http://lists.ozlabs.org/pipermail/linuxppc-dev/attachments/20080115/179c3281/attachment.pgp>


More information about the Linuxppc-dev mailing list