diff options
author | Cindy H Kao <cindy.h.kao@intel.com> | 2010-03-30 22:19:37 -0400 |
---|---|---|
committer | Inaky Perez-Gonzalez <inaky.perez-gonzalez@intel.com> | 2010-05-11 17:04:54 -0400 |
commit | 49d72df3f6cd63b49528306a2577ae7a6a0e9d6b (patch) | |
tree | 82b6e254c2550f43108c72bea745df9631355fae | |
parent | f22cf689a6353f072bca15d0a26f870e62dfacf8 (diff) |
wimax/i2400m: correct the error path handlers in dev_start()
This fix is to correct order of the handlers in the error path
of dev_start(). When i2400m_firmware_check fails, all the works done
before it should be released or cleared.
Signed-off-by: Cindy H Kao <cindy.h.kao@intel.com>
-rw-r--r-- | drivers/net/wimax/i2400m/driver.c | 2 |
1 files changed, 1 insertions, 1 deletions
diff --git a/drivers/net/wimax/i2400m/driver.c b/drivers/net/wimax/i2400m/driver.c index 94dc83c3969d..3a6c8ddb5353 100644 --- a/drivers/net/wimax/i2400m/driver.c +++ b/drivers/net/wimax/i2400m/driver.c | |||
@@ -403,10 +403,10 @@ retry: | |||
403 | 403 | ||
404 | error_dev_initialize: | 404 | error_dev_initialize: |
405 | error_check_mac_addr: | 405 | error_check_mac_addr: |
406 | error_fw_check: | ||
406 | i2400m->ready = 0; | 407 | i2400m->ready = 0; |
407 | wmb(); /* see i2400m->ready's documentation */ | 408 | wmb(); /* see i2400m->ready's documentation */ |
408 | flush_workqueue(i2400m->work_queue); | 409 | flush_workqueue(i2400m->work_queue); |
409 | error_fw_check: | ||
410 | if (i2400m->bus_dev_stop) | 410 | if (i2400m->bus_dev_stop) |
411 | i2400m->bus_dev_stop(i2400m); | 411 | i2400m->bus_dev_stop(i2400m); |
412 | error_bus_dev_start: | 412 | error_bus_dev_start: |