ping! This fixes a compile breakage and IMO should be applied before 2011.12 is out. Can someone, please apply it?
On 11/28/11 09:57, Igor Grinberg wrote: > When ENV_IS_EMBEDDED is not set, but CONFIG_BUILD_ENVCRC is set, > the environment.h file does not get included resulting in unrecognized > env_t type. > Fix this by moving the include directive. > > Reported-by: Mike Frysinger <vap...@gentoo.org> > Signed-off-by: Igor Grinberg <grinb...@compulab.co.il> > --- > tools/envcrc.c | 6 +++--- > 1 files changed, 3 insertions(+), 3 deletions(-) > > diff --git a/tools/envcrc.c b/tools/envcrc.c > index 111d9f6..51e3f54 100644 > --- a/tools/envcrc.c > +++ b/tools/envcrc.c > @@ -61,7 +61,6 @@ > #endif /* CONFIG_ENV_IS_IN_FLASH */ > > #if defined(ENV_IS_EMBEDDED) && !defined(CONFIG_BUILD_ENVCRC) > -# include <environment.h> > # define CONFIG_BUILD_ENVCRC 1 > #endif > > @@ -74,13 +73,14 @@ > #define ENV_SIZE (CONFIG_ENV_SIZE - ENV_HEADER_SIZE) > > > -extern uint32_t crc32 (uint32_t, const unsigned char *, unsigned int); > - > #ifdef CONFIG_BUILD_ENVCRC > +# include <environment.h> > extern unsigned int env_size; > extern env_t environment; > #endif /* CONFIG_BUILD_ENVCRC */ > > +extern uint32_t crc32 (uint32_t, const unsigned char *, unsigned int); > + > int main (int argc, char **argv) > { > #ifdef CONFIG_BUILD_ENVCRC -- Regards, Igor. _______________________________________________ U-Boot mailing list U-Boot@lists.denx.de http://lists.denx.de/mailman/listinfo/u-boot