Re: [PATCH] rfkill: always init poll delayed work

[Larry Finger <Larry.Finger@lwfinger.net>]

Johannes Berg wrote:
> The rfkill core didn't initialise the poll delayed work
> because it assumed that polling was always done by specifying
> the poll function. cfg80211, however, would like to start
> polling only later, which is a valid use case and easy to
> support, so change rfkill to always initialise the poll
> delayed work and thus allow starting polling by calling the
> rfkill_resume_polling() function after registration.
> 
> Signed-off-by: Johannes Berg <johannes@sipsolutions.net>
> ---

This patch fixed to kernel BUG_ON, but there are still problems. This
has been tested with b43.

1. A minor one - the radio LED switch never comes on. Module led-class
is loaded.

2. Much more serious - When the radio kill switch is turned off, the
radio is killed just as expected, but it is not restored when the
switch is turned on. The only way to restore the radio is to
rmmod/insmod b43. Similarly, if the module is loaded with the switch
off, it is not possible to turn the radio on. An unload/load resquence
is then needed.

Larry