diff options
author | Pekka Enberg <penberg@cs.helsinki.fi> | 2009-04-08 11:14:00 +0300 |
---|---|---|
committer | Greg Kroah-Hartman <gregkh@suse.de> | 2009-06-19 11:00:37 -0700 |
commit | a39ee671746b6483c6a259fef755a4a3e475e3e4 (patch) | |
tree | 94a2be12816f509ff25ded741cbf36647651eb39 /drivers/staging/winbond/wb35tx.c | |
parent | 00e2e05dd08bc15b09514f5ce35f0b9a3534e9ba (diff) |
Staging: w35und: more simplify hal_init_hardware() error handling
Impact: fix, cleanup
If initialization in hal_init_hardware() fails, we call hal_halt() to
clean up and release resource. However, hal_halt() will attempt to
call del_timer_sync() on ->LEDTimer on "stage 3" although it's not
initialized at that point. Fix that up by simplifying error handling
logic in hal_init_hardware() with gotos.
Acked-by: Pavel Machek <pavel@ucw.cz>
Signed-off-by: Pekka Enberg <penberg@cs.helsinki.fi>
Signed-off-by: Greg Kroah-Hartman <gregkh@suse.de>
Diffstat (limited to 'drivers/staging/winbond/wb35tx.c')
0 files changed, 0 insertions, 0 deletions