diff options
author | Henrique de Moraes Holschuh <hmh@hmh.eng.br> | 2008-10-03 15:58:05 -0400 |
---|---|---|
committer | John W. Linville <linville@tuxdriver.com> | 2008-10-06 18:14:57 -0400 |
commit | 417bd25ac4c6f76c8aafe8a584f3620f4a936b72 (patch) | |
tree | 7b7b55d715d7121dcba815cd2518608c0a6ec010 | |
parent | 5b5d13afeeee959a74114b73c560d3e243f34a11 (diff) |
rfkill: update LEDs for all state changes
The LED state was not being updated by rfkill_force_state(), which
will cause regressions in wireless drivers that had old-style rfkill
support and are updated to use rfkill_force_state().
The LED state was not being updated when a change was detected through
the rfkill->get_state() hook, either.
Move the LED trigger update calls into notify_rfkill_state_change(),
where it should have been in the first place. This takes care of both
issues above.
Signed-off-by: Henrique de Moraes Holschuh <hmh@hmh.eng.br>
Acked-by: Ivo van Doorn <IvDoorn@gmail.com>
Cc: stable@kernel.org
Signed-off-by: John W. Linville <linville@tuxdriver.com>
-rw-r--r-- | net/rfkill/rfkill.c | 5 |
1 files changed, 2 insertions, 3 deletions
diff --git a/net/rfkill/rfkill.c b/net/rfkill/rfkill.c index ea0dc04b3c77..f949a482b007 100644 --- a/net/rfkill/rfkill.c +++ b/net/rfkill/rfkill.c | |||
@@ -125,6 +125,7 @@ static void rfkill_led_trigger_activate(struct led_classdev *led) | |||
125 | 125 | ||
126 | static void notify_rfkill_state_change(struct rfkill *rfkill) | 126 | static void notify_rfkill_state_change(struct rfkill *rfkill) |
127 | { | 127 | { |
128 | rfkill_led_trigger(rfkill, rfkill->state); | ||
128 | blocking_notifier_call_chain(&rfkill_notifier_list, | 129 | blocking_notifier_call_chain(&rfkill_notifier_list, |
129 | RFKILL_STATE_CHANGED, | 130 | RFKILL_STATE_CHANGED, |
130 | rfkill); | 131 | rfkill); |
@@ -217,10 +218,8 @@ static int rfkill_toggle_radio(struct rfkill *rfkill, | |||
217 | rfkill->state = state; | 218 | rfkill->state = state; |
218 | } | 219 | } |
219 | 220 | ||
220 | if (force || rfkill->state != oldstate) { | 221 | if (force || rfkill->state != oldstate) |
221 | rfkill_led_trigger(rfkill, rfkill->state); | ||
222 | notify_rfkill_state_change(rfkill); | 222 | notify_rfkill_state_change(rfkill); |
223 | } | ||
224 | 223 | ||
225 | return retval; | 224 | return retval; |
226 | } | 225 | } |