diff options
author | Inaky Perez-Gonzalez <inaky@linux.intel.com> | 2009-06-11 14:13:41 -0400 |
---|---|---|
committer | Inaky Perez-Gonzalez <inaky@linux.intel.com> | 2009-06-11 14:47:39 -0400 |
commit | 98eb0f53e2fc66482e2ea8033c58b20a079e5260 (patch) | |
tree | d7bfd46772eb1da57cdb2f40e3e227186242bef6 /drivers/net/wimax/i2400m/op-rfkill.c | |
parent | d2f4c10544231c3681f937a1e511f4780b4b39af (diff) |
wimax: fix gcc warnings in sh4 when calling BUG()
SH4's BUG() seems to confuse the compiler as it is considered to
return; thus, some functions would trigger usage of uninitialized
variables or non-void functions returning void.
Work around by initializing/returning.
Signed-off-by: Inaky Perez-Gonzalez <inaky@linux.intel.com>
Diffstat (limited to 'drivers/net/wimax/i2400m/op-rfkill.c')
-rw-r--r-- | drivers/net/wimax/i2400m/op-rfkill.c | 4 |
1 files changed, 3 insertions, 1 deletions
diff --git a/drivers/net/wimax/i2400m/op-rfkill.c b/drivers/net/wimax/i2400m/op-rfkill.c index 487ec58cea46..43927b5d7ad6 100644 --- a/drivers/net/wimax/i2400m/op-rfkill.c +++ b/drivers/net/wimax/i2400m/op-rfkill.c | |||
@@ -54,8 +54,10 @@ int i2400m_radio_is(struct i2400m *i2400m, enum wimax_rf_state state) | |||
54 | /* state == WIMAX_RF_ON */ | 54 | /* state == WIMAX_RF_ON */ |
55 | return i2400m->state != I2400M_SS_RF_OFF | 55 | return i2400m->state != I2400M_SS_RF_OFF |
56 | && i2400m->state != I2400M_SS_RF_SHUTDOWN; | 56 | && i2400m->state != I2400M_SS_RF_SHUTDOWN; |
57 | else | 57 | else { |
58 | BUG(); | 58 | BUG(); |
59 | return -EINVAL; /* shut gcc warnings on certain arches */ | ||
60 | } | ||
59 | } | 61 | } |
60 | 62 | ||
61 | 63 | ||