On Sep 1, 2009, at 11:59 AM, Anton Vorontsov wrote:
> The warning is bogus, so silence it with uninitialized_var().
>
> Signed-off-by: Anton Vorontsov <avoront...@ru.mvista.com>
> ---
> board/freescale/common/sys_eeprom.c | 4 +++-
> 1 files changed, 3 insertions(+), 1 deletions(-)
>
> diff --git a/board/freescale/common/sys_eeprom.c b/board/freescale/
> common/sys_eeprom.c
> index c0fff68..a765b39 100644
> --- a/board/freescale/common/sys_eeprom.c
> +++ b/board/freescale/common/sys_eeprom.c
> @@ -24,6 +24,7 @@
> */
>
> #include <common.h>
> +#include <compiler.h>
> #include <command.h>
> #include <i2c.h>
> #include <linux/ctype.h>
> @@ -204,7 +205,8 @@ static void update_crc(void)
> */
> static int prog_eeprom(void)
> {
> - int ret, i;
> + int uninitialized_var(ret);
> + int i;
why don't we init ret = 0; seems like we should be doing that since
we might not enter the for loop
- k
_______________________________________________
U-Boot mailing list
U-Boot@lists.denx.de
http://lists.denx.de/mailman/listinfo/u-boot
