[PATCH] Move the fdt*_t type definitions to fdt.h

David Gibson david at gibson.dropbear.id.au
Mon Jan 28 15:52:33 EST 2013


On Sun, Jan 27, 2013 at 09:44:37PM -0500, Gerald Van Baren wrote:
> Arguably they belong there since they are fundamental to the fdt.
> As a side effect, I had to flip the order of #includes of fdt.h and
> libfdt_env.h in libfdt.h since libfdt_env.h needs the fdt*_t definitions.
> 
> Signed-off-by: Gerald Van Baren <gvb at unssw.com>
> ---
> 
> I'm not sure I like this, but submit it for your consideration.  If, for
> some reason, libfdt_env.h needed to (re)define something in fdt.h, it
> would be unable to do so. :-/

Sorry, nack on this one.  fdt.h is supposed to be as independent as we
can make it from the environment in which its being compiled.  That
means I don't want it to include *any* system headers, even stdint.h.

-- 
David Gibson			| I'll have my music baroque, and my code
david AT gibson.dropbear.id.au	| minimalist, thank you.  NOT _the_ _other_
				| _way_ _around_!
http://www.ozlabs.org/~dgibson
-------------- next part --------------
A non-text attachment was scrubbed...
Name: signature.asc
Type: application/pgp-signature
Size: 198 bytes
Desc: Digital signature
URL: <http://lists.ozlabs.org/pipermail/devicetree-discuss/attachments/20130128/5afee6e1/attachment.sig>


More information about the devicetree-discuss mailing list