Linux wireless drivers development
 help / color / mirror / Atom feed
From: Larry Finger <Larry.Finger@lwfinger.net>
To: Johannes Berg <johannes@sipsolutions.net>
Cc: wireless <linux-wireless@vger.kernel.org>
Subject: b43 rfkill LED broken in 2.6.30-rc1 from wireless-testing
Date: Thu, 09 Apr 2009 16:38:06 -0500	[thread overview]
Message-ID: <49DE6ABE.30207@lwfinger.net> (raw)

Johannes,

Boy is the rfkill subsystem fragile.

After all our debugging of the control of the b43 LED with the rfkill switch, it
is broken again with 2.6.30-rc1.

I put a test print at the start of b43_led_brightness_set(). It gets called lots
of times for the TX/RX LED (which I do not have), but only gets called for the
radio LED when the driver is loaded or unloaded. Changing the rfkill switch
causes the "radio hardware status" messages to be output, but nothing happens
with the LED.

Now, led_trigger_event() once again is only called indirectly through
rfkill_led_trigger() from rfkill_led_trigger_activate(), but
rfkill_led_trigger() is never called. I have no idea who is calling
b43_led_brightness at start-up/shut-down. It must be some part of the LED system.

Some things are working. The contents of /sys/class/rfkill/rfkill3/state change
from 0 to 1 when the switch is turned on; however, I have not been able to
locate what code is doing the change. Routine rfkill_uevent() is never entered.

Larry

             reply	other threads:[~2009-04-09 21:38 UTC|newest]

Thread overview: 5+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2009-04-09 21:38 Larry Finger [this message]
2009-04-10  3:08 ` b43 rfkill LED broken in 2.6.30-rc1 from wireless-testing Larry Finger
2009-04-10  7:32 ` Johannes Berg
2009-04-11  6:19   ` Henrique de Moraes Holschuh
2009-04-11 10:43     ` Johannes Berg

Reply instructions:

You may reply publicly to this message via plain-text email
using any one of the following methods:

* Save the following mbox file, import it into your mail client,
  and reply-to-all from there: mbox

  Avoid top-posting and favor interleaved quoting:
  https://en.wikipedia.org/wiki/Posting_style#Interleaved_style

* Reply using the --to, --cc, and --in-reply-to
  switches of git-send-email(1):

  git send-email \
    --in-reply-to=49DE6ABE.30207@lwfinger.net \
    --to=larry.finger@lwfinger.net \
    --cc=johannes@sipsolutions.net \
    --cc=linux-wireless@vger.kernel.org \
    /path/to/YOUR_REPLY

  https://kernel.org/pub/software/scm/git/docs/git-send-email.html

* If your mail client supports setting the In-Reply-To header
  via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line before the message body.
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox