[U-Boot] [PATCH 4/7] drivers/net/smc91111.c: fix warning: variable ... set but not used

2011-10-05 Thread Wolfgang Denk 
Fix:
smc9.c: In function 'smc_phy_configure':
smc9.c:1194:6: warning: variable 'failed' set but not used 
[-Wunused-but-set-variable]
smc9.c:1190:7: warning: variable 'phyaddr' set but not used 
[-Wunused-but-set-variable]

Signed-off-by: Wolfgang Denk 
---
 drivers/net/smc9.c |8 
 1 files changed, 0 insertions(+), 8 deletions(-)

diff --git a/drivers/net/smc9.c b/drivers/net/smc9.c
index ba9c67e..9b8236d 100644
--- a/drivers/net/smc9.c
+++ b/drivers/net/smc9.c
@@ -1187,18 +1187,12 @@ static void smc_wait_ms(unsigned int ms)
 static void smc_phy_configure (struct eth_device *dev)
 {
int timeout;
-   byte phyaddr;
word my_phy_caps;   /* My PHY capabilities */
word my_ad_caps;/* My Advertised capabilities */
word status = 0;/*;my status = 0 */
-   int failed = 0;
 
PRINTK3 ("%s: smc_program_phy()\n", SMC_DEV_NAME);
 
-
-   /* Get the detected phy address */
-   phyaddr = SMC_PHY_ADDR;
-
/* Reset the PHY, setting all other bits to zero */
smc_write_phy_register (dev, PHY_CNTL_REG, PHY_CNTL_RST);
 
@@ -1296,13 +1290,11 @@ static void smc_phy_configure (struct eth_device *dev)
 
if (timeout < 1) {
printf ("%s: PHY auto-negotiate timed out\n", SMC_DEV_NAME);
-   failed = 1;
}
 
/* Fail if we detected an auto-negotiate remote fault */
if (status & PHY_STAT_REM_FLT) {
printf ("%s: PHY remote fault detected\n", SMC_DEV_NAME);
-   failed = 1;
}
 
/* Re-Configure the Receive/Phy Control register */
-- 
1.7.6.2

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

Re: [U-Boot] [PATCH 4/7] drivers/net/smc91111.c: fix warning: variable ... set but not used

2011-10-09 Thread Wolfgang Denk 
Dear Wolfgang Denk,

In message <1317849453-8216-4-git-send-email...@denx.de> you wrote:
> Fix:
> smc9.c: In function 'smc_phy_configure':
> smc9.c:1194:6: warning: variable 'failed' set but not used 
> [-Wunused-but-set-variable]
> smc9.c:1190:7: warning: variable 'phyaddr' set but not used 
> [-Wunused-but-set-variable]
> 
> Signed-off-by: Wolfgang Denk 
> ---
>  drivers/net/smc9.c |8 
>  1 files changed, 0 insertions(+), 8 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
The price of curiosity is a terminal experience.
 - Terry Pratchett, _The Dark Side of the Sun_
___
U-Boot mailing list
U-Boot@lists.denx.de
http://lists.denx.de/mailman/listinfo/u-boot