Skip to content

Commit

Permalink
netxen: fix file firmware leak
Browse files Browse the repository at this point in the history
Release file firmware when no firmware reset is required.

Signed-off-by: Dhananjay Phadke <dhananjay@netxen.com>
Signed-off-by: David S. Miller <davem@davemloft.net>
  • Loading branch information
Dhananjay Phadke authored and David S. Miller committed Sep 11, 2009
1 parent ea6828b commit ec5c50c
Showing 1 changed file with 3 additions and 2 deletions.
5 changes: 3 additions & 2 deletions drivers/net/netxen/netxen_nic_main.c
Original file line number Diff line number Diff line change
Expand Up @@ -817,7 +817,7 @@ netxen_start_firmware(struct netxen_adapter *adapter)
if (err < 0)
goto err_out;
if (err == 0)
goto wait_init;
goto ready;

if (first_boot != 0x55555555) {
NXWR32(adapter, CRB_CMDPEG_STATE, 0);
Expand Down Expand Up @@ -860,6 +860,7 @@ netxen_start_firmware(struct netxen_adapter *adapter)
| (_NETXEN_NIC_LINUX_SUBVERSION);
NXWR32(adapter, CRB_DRIVER_VERSION, val);

ready:
NXWR32(adapter, NX_CRB_DEV_STATE, NX_DEV_READY);

wait_init:
Expand All @@ -874,7 +875,7 @@ netxen_start_firmware(struct netxen_adapter *adapter)

netxen_check_options(adapter);

return 0;
/* fall through and release firmware */

err_out:
netxen_release_firmware(adapter);
Expand Down

0 comments on commit ec5c50c

Please sign in to comment.