Re: [U-Boot] [PATCH 14/56] board/esd/pf5200/pf5200.c: Fix GCC 4.6 build warning

[Wolfgang Denk]

Dear Wolfgang Denk,

In message <1320458160-23136-15-git-send-email...@denx.de> you wrote:
> Fix:
> pf5200.c: In function 'do_phypower':
> pf5200.c:330:6: warning: variable 'status' set but not used
> [-Wunused-but-set-variable]
> 
> Signed-off-by: Wolfgang Denk 
> Cc: Reinhard Arlt 
> ---
>  board/esd/pf5200/pf5200.c |   10 --
>  1 files changed, 4 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
"You shouldn't make my toaster angry." - Household security explained
in "Johnny Quest"
___
U-Boot mailing list
U-Boot@lists.denx.de
http://lists.denx.de/mailman/listinfo/u-boot

[U-Boot] [PATCH 14/56] board/esd/pf5200/pf5200.c: Fix GCC 4.6 build warning

[Wolfgang Denk]

Fix:
pf5200.c: In function 'do_phypower':
pf5200.c:330:6: warning: variable 'status' set but not used
[-Wunused-but-set-variable]

Signed-off-by: Wolfgang Denk 
Cc: Reinhard Arlt 
---
 board/esd/pf5200/pf5200.c |   10 --
 1 files changed, 4 insertions(+), 6 deletions(-)

diff --git a/board/esd/pf5200/pf5200.c b/board/esd/pf5200/pf5200.c
index 83dbfcb..2e07ac1 100644
--- a/board/esd/pf5200/pf5200.c
+++ b/board/esd/pf5200/pf5200.c
@@ -327,13 +327,11 @@ int phypower(int flag)
 
 int do_phypower(cmd_tbl_t * cmdtp, int flag, int argc, char * const argv[])
 {
-   int status;
+   if (argv[1][0] == '0')
+   (void)phypower(0);
+   else
+   (void)phypower(1);
 
-   if (argv[1][0] == '0') {
-   status = phypower(0);
-   } else {
-   status = phypower(1);
-   }
return (0);
 }
 
-- 
1.7.6.4

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