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 w...@denx.de
 ---
  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

[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 w...@denx.de
---
 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