diff options
| author | Stephen Hemminger | 2007-02-26 23:51:56 +0100 |
|---|---|---|
| committer | Jeff Garzik | 2007-02-27 10:38:33 +0100 |
| commit | f8dfdd5cab482a2ce4a8e2375e1512aa4829c653 (patch) | |
| tree | d2c409de5c9cf830aa22793f162beb5612371df2 /drivers/net | |
| parent | natsemi: Fix detection of vanilla natsemi cards (diff) | |
| download | kernel-qcow2-linux-f8dfdd5cab482a2ce4a8e2375e1512aa4829c653.tar.gz kernel-qcow2-linux-f8dfdd5cab482a2ce4a8e2375e1512aa4829c653.tar.xz kernel-qcow2-linux-f8dfdd5cab482a2ce4a8e2375e1512aa4829c653.zip | |
netxen: do_rom_fast_write error handling
Compiler warning spots real error!
The function do_rom_fast_read called in do_rom_fast_write can fail
and leave data1 unset. This causes a compile warning.
The correct thing is to propagate the error out.
Signed-off-by: Stephen Hemminger <shemminger@linux-foundation.org>
Signed-off-by: Jeff Garzik <jeff@garzik.org>
Diffstat (limited to 'drivers/net')
| -rw-r--r-- | drivers/net/netxen/netxen_nic_init.c | 5 |
1 files changed, 4 insertions, 1 deletions
diff --git a/drivers/net/netxen/netxen_nic_init.c b/drivers/net/netxen/netxen_nic_init.c index 2f324366784d..2f965701a95b 100644 --- a/drivers/net/netxen/netxen_nic_init.c +++ b/drivers/net/netxen/netxen_nic_init.c @@ -499,7 +499,10 @@ static inline int do_rom_fast_write_words(struct netxen_adapter *adapter, while(1) { int data1; - do_rom_fast_read(adapter, addridx, &data1); + ret = do_rom_fast_read(adapter, addridx, &data1); + if (ret < 0) + return ret; + if (data1 == data) break; |