In message <[EMAIL PROTECTED]> 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 <[EMAIL PROTECTED]>
> ---
>  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: [EMAIL PROTECTED]
A good marriage would be between a blind wife and deaf husband.
                                               -- Michel de Montaigne

-------------------------------------------------------------------------
This SF.Net email is sponsored by the Moblin Your Move Developer's challenge
Build the coolest Linux based applications with Moblin SDK & win great prizes
Grand prize is a trip for two to an Open Source event anywhere in the world
http://moblin-contest.org/redirect.php?banner_id=100&url=/
_______________________________________________
U-Boot-Users mailing list
U-Boot-Users@lists.sourceforge.net
https://lists.sourceforge.net/lists/listinfo/u-boot-users

Reply via email to