diff options
| author |   Steve Glendinning <steve.glendinning@smsc.com> | 2009-03-04 07:33:25 +0000 |
|---|---|---|
| committer |   David S. Miller <davem@davemloft.net> | 2009-03-13 12:14:11 -0700 |
| commit | f7efb6ccc2113911e4e064f78bcd0343c4673038 (patch) | |
| tree | 3856f2a8ffa3ac0efc6da507b194f93183b2b0d6 /drivers/net/smsc911x.c | |
| parent | smsc911x: check for FFWD success before checking for timeout (diff) | |
| download | linux-dev-f7efb6ccc2113911e4e064f78bcd0343c4673038.tar.xz linux-dev-f7efb6ccc2113911e4e064f78bcd0343c4673038.zip | |
smsc911x: improve EEPROM loading timeout logic in open
This patch from Juha Leppanen suppresses a false warning if the eeprom
load succeeds on the very last attempt.
Juha> In function smsc911x_open smsc911x_reg_read+udelay can be run 50
Juha> times with timeout reaching -1, and the following if statetement
Juha> does not catch the timeout and no warning is issued. Also if the
Juha> 50th smsc911x_reg_read is GOOD, loop is exited with timeout as 0
Juha> and bogus warning issued. Replace testing order and --timeout
Juha> instead of timeout-- and now max 50 smsc911x_reg_read's are done,
Juha> with max 49 udelays.
Signed-off-by: Steve Glendinning <steve.glendinning@smsc.com>
Signed-off-by: David S. Miller <davem@davemloft.net>
Diffstat (limited to 'drivers/net/smsc911x.c')
| -rw-r--r-- | drivers/net/smsc911x.c | 4 |
1 files changed, 2 insertions, 2 deletions
diff --git a/drivers/net/smsc911x.c b/drivers/net/smsc911x.c index 9a795105cc7c..ad15aab2137d 100644 --- a/drivers/net/smsc911x.c +++ b/drivers/net/smsc911x.c @@ -1162,8 +1162,8 @@ static int smsc911x_open(struct net_device *dev) /* Make sure EEPROM has finished loading before setting GPIO_CFG */ timeout = 50; - while ((timeout--) && - (smsc911x_reg_read(pdata, E2P_CMD) & E2P_CMD_EPC_BUSY_)) { + while ((smsc911x_reg_read(pdata, E2P_CMD) & E2P_CMD_EPC_BUSY_) && + --timeout) { udelay(10); } |