[U-Boot] [PATCH 02/10] Add some standard headers external code might need

Simon Glass sjg at chromium.org
Tue Oct 28 17:29:03 CET 2014


Hi Masahiro,

On 28 October 2014 10:25, Masahiro YAMADA <yamada.m at jp.panasonic.com> wrote:
>
> Hi Gabe, Simon,
>
>
> 2014-10-15 19:38 GMT+09:00 Simon Glass <sjg at chromium.org>:
> > From: Gabe Black <gabeblack at chromium.org>
> >
> > inttypes.h defines format specifiers for printf which work with data types of
> > particular sizes. stdlib.h is currently just a passthrough to malloc.h which
> > has declarations of the various *alloc functions.
> >
> > Add the required #define to common.h so that these printf format specifiers
> > will be made available.
> >
> > Signed-off-by: Gabe Black <gabeblack at google.com>
> > Reviewed-by: Gabe Black <gabeblack at chromium.org>
> > Tested-by: Gabe Black <gabeblack at chromium.org>
> > Reviewed-by: Bill Richardson <wfrichar at google.com>
> > Signed-off-by: Simon Glass <sjg at chromium.org>
> > (Replaced with a GPL version from glibc)
> >
> [snip]
> > diff --git a/include/stdlib.h b/include/stdlib.h
> > new file mode 100644
> > index 0000000..6bc7fbb
> > --- /dev/null
> > +++ b/include/stdlib.h
> > @@ -0,0 +1,12 @@
> > +/*
> > + *  Copyright (C) 2013 Google Inc.
> > + *
> > + * SPDX-License-Identifier:    GPL-2.0+
> > + */
> > +
> > +#ifndef __STDLIB_H_
> > +#define __STDLIB_H_
> > +
> > +#include <malloc.h>
> > +
> > +#endif /* __STDLIB_H_ */
> > --
> > 2.1.0.rc2.206.gedb03e5
>
>
> This patch is not clear to me.
>
> Why do we need include/stdlib.h ?

This makes the U-Boot environment more similar to that used by other
software, so we can more easily build it without lots of glue files.
Normally stdlib.h defines malloc() and friends.

Regards,
Simon


More information about the U-Boot mailing list