km/common: remove saveenv from do_checkboardidhwkey
authorHolger Brunck <holger.brunck@keymile.com>
Sun, 5 Jun 2011 22:22:17 +0000 (22:22 +0000)
committerWolfgang Denk <wd@denx.de>
Wed, 27 Jul 2011 21:44:15 +0000 (23:44 +0200)
This is unneeded here because we save the environment when
the board boots the first time. At this time we have set
the values already.

Signed-off-by: Holger Brunck <holger.brunck@keymile.com>
Signed-off-by: Valentin Longchamp <valentin.longchamp@keymile.com>
cc: Heiko Schocher <hs@denx.de>
cc: Wolfgang Denk <wd@denx.de>

board/keymile/common/common.c

index fef62534b46998d6e2109b2e091674a0c6477768..d8fcd39197f2f52cbe2257b8420da6daaab8d274 100644 (file)
@@ -831,11 +831,7 @@ int do_checkboardidhwk(cmd_tbl_t *cmdtp, int flag, int argc,
                                 * Compare the values of the found entry in the
                                 * list with the valid values which are stored
                                 * in the inventory eeprom. If they are equal
-                                * store the values in environment variables
-                                * and save the environment.
-                                * This can only happen once for the lifetime
-                                * of a board, because once saved the function
-                                * will never reach the while loop.
+                                * set the values in environment variables.
                                 */
                                if ((bid == ivmbid) && (hwkey == ivmhwkey)) {
                                        char buf[10];
@@ -847,7 +843,6 @@ int do_checkboardidhwk(cmd_tbl_t *cmdtp, int flag, int argc,
                                        setenv("boardid", buf);
                                        sprintf(buf, "%lx", hwkey);
                                        setenv("hwkey", buf);
-                                       saveenv();
                                }
                        } /* end while( ! found ) */
                }