[U-Boot-Users] [PATCH] Add include for config.h in command.h.

Wolfgang Denk wd at denx.de
Wed Jul 30 00:44:01 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.
> 
> Signed-off-by: Stefano Babic <sbabic at denx.de>
> ---
>  include/command.h |    2 ++
>  1 files changed, 2 insertions(+), 0 deletions(-)

Applied, thanks.

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
The greatest threat towards future is indifference.




More information about the U-Boot mailing list