Re: [U-Boot] [PATCH 07/56] board/mcc200/auto_update.c: fixx GCC 4.6 build warnings

2011-11-07 Thread Wolfgang Denk
Dear Wolfgang Denk,

In message <1320458160-23136-8-git-send-email...@denx.de> you wrote:
> Fix:
> auto_update.c: In function 'do_auto_update':
> auto_update.c:344:48: warning: variable 'got_ctrlc' set but not used
> [-Wunused-but-set-variable]
> auto_update.c:344:18: warning: variable 'bitmap_first' set but not
> used [-Wunused-but-set-variable]
> 
> Signed-off-by: Wolfgang Denk 
> ---
>  board/mcc200/auto_update.c |7 +--
>  1 files changed, 1 insertions(+), 6 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: w...@denx.de
A list is only as strong as its weakest link.-- Don Knuth
___
U-Boot mailing list
U-Boot@lists.denx.de
http://lists.denx.de/mailman/listinfo/u-boot


[U-Boot] [PATCH 07/56] board/mcc200/auto_update.c: fixx GCC 4.6 build warnings

2011-11-04 Thread Wolfgang Denk
Fix:
auto_update.c: In function 'do_auto_update':
auto_update.c:344:48: warning: variable 'got_ctrlc' set but not used
[-Wunused-but-set-variable]
auto_update.c:344:18: warning: variable 'bitmap_first' set but not
used [-Wunused-but-set-variable]

Signed-off-by: Wolfgang Denk 
---
 board/mcc200/auto_update.c |7 +--
 1 files changed, 1 insertions(+), 6 deletions(-)

diff --git a/board/mcc200/auto_update.c b/board/mcc200/auto_update.c
index 49213d0..4152873 100644
--- a/board/mcc200/auto_update.c
+++ b/board/mcc200/auto_update.c
@@ -341,7 +341,7 @@ int do_auto_update(void)
 {
block_dev_desc_t *stor_dev;
long sz;
-   int i, res = 0, bitmap_first, cnt, old_ctrlc, got_ctrlc;
+   int i, res = 0, cnt, old_ctrlc;
char *env;
long start, end;
 
@@ -450,8 +450,6 @@ int do_auto_update(void)
/* make sure that we see CTRL-C and save the old state */
old_ctrlc = disable_ctrlc(0);
 
-   bitmap_first = 0;
-
/* validate the images first */
for (i = 0; i < AU_MAXFILES; i++) {
ulong imsize;
@@ -506,14 +504,11 @@ int do_auto_update(void)
/* this is really not a good idea, but it's what the */
/* customer wants. */
cnt = 0;
-   got_ctrlc = 0;
do {
res = au_do_update(i, sz);
/* let the user break out of the loop */
if (ctrlc() || had_ctrlc()) {
clear_ctrlc();
-   if (res < 0)
-   got_ctrlc = 1;
break;
}
cnt++;
-- 
1.7.6.4

___
U-Boot mailing list
U-Boot@lists.denx.de
http://lists.denx.de/mailman/listinfo/u-boot