[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