[U-Boot-Users] [PATCH] Add include for config.h in command.h.
Wolfgang Denk
wd at denx.de
Fri Jul 25 06:28:16 CEST 2008
In message <1216909345-15346-1-git-send-email-sbabic at denx.de> you wrote:
> Because the cmd_tbl_s structure depends on the configuration file,
> it must be assured that config.h is included before the structure is
> evaluated by the compiler. If this is not certain, it could happen
> that the compiler generates structures of different size, depending
> on the fact if the source file includes <config.h> before or after
> <command.h>.
> The effect is that u-boot crashes when tries to relocate the command
> table (for ppc) or try to access to the command table for other
> architectures.
> The problem can happen on board-depending commands.All general
> commands under /common are unaffected, because they include already
> config.h before command.h.
Agreed in principle, but...
> Signed-off-by: Stefano Babic <sbabic at denx.de>
> ---
> include/command.h | 2 ++
> 1 files changed, 2 insertions(+), 0 deletions(-)
>
> diff --git a/include/command.h b/include/command.h
> index c3ef51d..8815937 100644
> --- a/include/command.h
> +++ b/include/command.h
> @@ -31,6 +31,8 @@
> #define NULL 0
> #endif
>
> +#include <config.h>
> +
...please move the #include *before* the #define.
Best regards,
Wolfgang Denk
--
DENX Software Engineering GmbH, MD: Wolfgang Denk & Detlev Zundel
HRB 165235 Munich, Office: Kirchenstr.5, D-82194 Groebenzell, Germany
Phone: (+49)-8142-66989-10 Fax: (+49)-8142-66989-80 Email: wd at denx.de
A good marriage would be between a blind wife and deaf husband.
-- Michel de Montaigne
More information about the U-Boot
mailing list