diff options
author | Dan Carpenter <error27@gmail.com> | 2010-08-22 00:54:30 -0700 |
---|---|---|
committer | David S. Miller <davem@davemloft.net> | 2010-08-22 00:54:30 -0700 |
commit | 773bc97397ea2cfc2ff3c075f712aaf376ed7cb0 (patch) | |
tree | ecb848e8e2d0978f73e903a49298527859531fa3 /drivers/net/tokenring/tms380tr.c | |
parent | 739a91ef0625e0e4a40b835f4f891313c47915df (diff) |
tokenring: remove unneeded NULL checks
"fw_entry" is always non-NULL at this point and anyway
release_firmware() handles NULL parameters.
Signed-off-by: Dan Carpenter <error27@gmail.com>
Signed-off-by: David S. Miller <davem@davemloft.net>
Diffstat (limited to 'drivers/net/tokenring/tms380tr.c')
-rw-r--r-- | drivers/net/tokenring/tms380tr.c | 6 |
1 files changed, 2 insertions, 4 deletions
diff --git a/drivers/net/tokenring/tms380tr.c b/drivers/net/tokenring/tms380tr.c index 435ef7d5470..08182fde3dc 100644 --- a/drivers/net/tokenring/tms380tr.c +++ b/drivers/net/tokenring/tms380tr.c @@ -1321,14 +1321,12 @@ static int tms380tr_reset_adapter(struct net_device *dev) /* Clear CPHALT and start BUD */ SIFWRITEW(c, SIFACL); - if (fw_entry) - release_firmware(fw_entry); + release_firmware(fw_entry); return (1); } } while(count == 0); - if (fw_entry) - release_firmware(fw_entry); + release_firmware(fw_entry); printk(KERN_INFO "%s: Adapter Download Failed\n", dev->name); return (-1); } |