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
[email protected]
https://lists.sourceforge.net/lists/listinfo/u-boot-users
