In message <[EMAIL PROTECTED]> you wrote:
> Fix a warning type in lib_i386/board.c
Um... you not only fix a warning; you change the code pretty heavily.
> --- a/lib_i386/board.c
> +++ b/lib_i386/board.c
> @@ -38,6 +38,9 @@
> #include <ide.h>
> #include <asm/u-boot-i386.h>
>
> +gd_t gd_data;
> +gd_t *gd;
> +
Please do not name this 'gd_data'; stick with the 'global_data' name.
Also be aware that this is dangerous - this way it ends up in the BSS
segment which most probably is uninitialized when you acess this data,
and gets overwritten with zeroes some time later.
Is this really working code?
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]
Inside every old person is a young person wondering what happened.
- Terry Pratchett, _Moving Pictures_
-------------------------------------------------------------------------
This SF.net email is sponsored by: Microsoft
Defy all challenges. Microsoft(R) Visual Studio 2008.
http://clk.atdmt.com/MRT/go/vse0120000070mrt/direct/01/
_______________________________________________
U-Boot-Users mailing list
U-Boot-Users@lists.sourceforge.net
https://lists.sourceforge.net/lists/listinfo/u-boot-users
