powerpc: Correct build warning introduced by getenv_ulong() patch
authorSimon Glass <sjg@chromium.org>
Sun, 23 Oct 2011 17:41:54 +0000 (17:41 +0000)
committerWolfgang Denk <wd@denx.de>
Mon, 24 Oct 2011 19:13:15 +0000 (21:13 +0200)
Commit 1272592 introduced a warning since the variable 's' is no longer
always used, depending on the CONFIG options.

Signed-off-by: Simon Glass <sjg@chromium.org>
arch/powerpc/lib/board.c

index 6cb0ed63444e55a8c131f2a92b9c3b718f183cb8..508075fa88b89198b49edaa3acf0ee190f8411c6 100644 (file)
@@ -623,7 +623,6 @@ void board_init_f (ulong bootflag)
  */
 void board_init_r (gd_t *id, ulong dest_addr)
 {
-       char *s;
        bd_t *bd;
        ulong malloc_start;
 
@@ -727,6 +726,8 @@ void board_init_r (gd_t *id, ulong dest_addr)
                flash_size = 0;
        } else if ((flash_size = flash_init ()) > 0) {
 # ifdef CONFIG_SYS_FLASH_CHECKSUM
+               char *s;
+
                print_size (flash_size, "");
                /*
                 * Compute and print flash CRC if flashchecksum is set to 'y'
@@ -927,8 +928,11 @@ void board_init_r (gd_t *id, ulong dest_addr)
        /* Initialize from environment */
        load_addr = getenv_ulong("loadaddr", 16, load_addr);
 #if defined(CONFIG_CMD_NET)
-       if ((s = getenv ("bootfile")) != NULL) {
-               copy_filename (BootFile, s, sizeof (BootFile));
+       {
+               char *s = getenv("bootfile");
+
+               if (s != NULL)
+                       copy_filename(BootFile, s, sizeof(BootFile));
        }
 #endif