From: Larry Finger <Larry.Finger@lwfinger.net>
To: Michael Buesch <mb@bu3sch.de>
Cc: Johannes Berg <johannes@sipsolutions.net>,
linux-wireless@vger.kernel.org
Subject: Re: [RFC V2] b43/legacy: port to cfg80211 rfkill
Date: Fri, 05 Jun 2009 16:20:55 -0500 [thread overview]
Message-ID: <4A298C37.9030503@lwfinger.net> (raw)
In-Reply-To: <200906052101.47168.mb@bu3sch.de>
Michael Buesch wrote:
>
> This is pretty silly behavior IMO. Just to bring it to the point:
> We initialize a huge wireless MAC, PHY and Radio that consume several watts of power
> just to poll a silly RF-kill bit.
That is what the driver already does.
> We can't we just accept that the RF-kill status is unknown while the device is down?
The problem is that while the interface is down the switch status
cannot be interrogated. If you try, you get a fatal SSB error. Thus
the only way to bring it back up is to flip the switch, then
rmmod/insmod the driver. If you want hardware rfkill to be one-way,
then take Johannes's patch. We would save a little power by calling
b43_wireless_exit() if we brought it up to test the switch, and the
switch was still off. That would leave everything off most of the time.
> I really do hate all that rfkill crap and I'm still refusing to sign off on anything that's
> related to rfkill (like I did for the past year or so). If people want this merged,
> somebody else maintain and sign it off, please.
I'm sick of rfkill as well and really detest the endless discussions
that have taken place; however, I do want the stuff to work.
As I see it, we have several options (presented in my order of
preference):
1. We switch to the cfg80211 rfkill and use this patch modified to
turn the interface back off if the switch is still off.
2. We continue to use the old rfkill mechanism. It works just fine,
but this method runs the risk of the old method being deprecated and
eliminated.
3. We get new callbacks that will only power down/up the radio when it
is blocked. That saves a little power.
Larry
next prev parent reply other threads:[~2009-06-05 21:20 UTC|newest]
Thread overview: 16+ messages / expand[flat|nested] mbox.gz Atom feed top
2009-06-05 18:19 [RFC V2] b43/legacy: port to cfg80211 rfkill Larry Finger
2009-06-05 19:01 ` Michael Buesch
2009-06-05 21:20 ` Larry Finger [this message]
2009-06-05 21:38 ` Michael Buesch
2009-06-05 22:32 ` Johannes Berg
2009-06-06 9:34 ` Michael Buesch
2009-06-06 17:38 ` Johannes Berg
2009-06-05 22:38 ` Larry Finger
2009-06-06 9:38 ` Michael Buesch
2009-06-06 17:38 ` Johannes Berg
2009-06-06 17:37 ` Johannes Berg
2009-06-06 17:44 ` Johannes Berg
2009-06-06 18:00 ` Michael Buesch
2009-06-07 12:55 ` Larry Finger
2009-06-06 17:42 ` Johannes Berg
2009-06-06 17:49 ` Michael Buesch
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=4A298C37.9030503@lwfinger.net \
--to=larry.finger@lwfinger.net \
--cc=johannes@sipsolutions.net \
--cc=linux-wireless@vger.kernel.org \
--cc=mb@bu3sch.de \
/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;
as well as URLs for NNTP newsgroup(s).