From: Dan Williams <dcbw@redhat.com>
To: Henrique de Moraes Holschuh <hmh@hmh.eng.br>
Cc: Tomas Winkler <tomasw@gmail.com>,
"John W. Linville" <linville@tuxdriver.com>,
Ivo van Doorn <ivdoorn@gmail.com>,
linux-wireless@vger.kernel.org, Dmitry Torokhov <dtor@mail.ru>
Subject: Re: [PATCH 01/12] rfkill: clarify meaning of rfkill states
Date: Fri, 06 Jun 2008 10:27:31 -0400 [thread overview]
Message-ID: <1212762451.28631.3.camel@localhost.localdomain> (raw)
In-Reply-To: <20080606141419.GA3444@khazad-dum.debian.net>
On Fri, 2008-06-06 at 11:14 -0300, Henrique de Moraes Holschuh wrote:
> On Fri, 06 Jun 2008, Dan Williams wrote:
> > Let me explain this a bit more.
> >
> > - My original request was based on my flawed understanding of the
> > current scope of the rfkill system
>
> Ok. I am keeping that in mind.
>
> > - If the rfkill system is _only_ supposed to handle switches that turn
> > the radio off automatically, then a "third state" for the rfkill class
> > really serves _no_ purpose at all, because the radio's already blocked
> > when the switch is thrown
>
> Given these rules (not documented fully yet, I shall fix it when this
> thread quiesces):
>
> 1. Every transmitter shall have ONE, and just ONE (not zero, not two,
> not more) rfkill class attached. For those transmitters lacking support
> in hardware to make it block, the drivers shall quiesce them and avoid
> doing anything to cause transmissions, or even use bus tricks to power
> the device down (i.e. the driver will emulate a switch capable of doing
> the blocking).
>
> Thus, rfkill class will give you the DEVICE (radio) rfkill block/unblock
> state on wireless devices (ignore switches for now, I will explain them
> in another email).
>
> The information of whether a transmitter is currently blocked in a way
> you can request it to be unblocked (soft-kill) or currently blocked in a
> way you cannot request it to be unblocked (hard-kill) DOES belong in
> rfkill class. Do you want/need that information, yes or no?
Yes, I want/need that information.
> > - There are drivers that implement rfkill (laptop-specific BIOS drivers)
> > that have _no_ associated transmitter device; thus the current 'struct
> > rfkill' can only be a representation of the _switch_; thus a "third
> > state" would be useless here as well (as is the toggle_radio() callback)
>
> I haven't finished replying to your other email yet, because it is not a
> fast reply (and I am in a hurry in the next two days) and I need to look
> at hp-wmi itself to understand what the issue that is causing confusion
> is. THAT reply will explain to you how it works with "pure read/write
> switches that are not in a wireless device". THEN we shall be able to
> unravel the confusion, and I will finally understand what you are trying
> to tell me (or you will understand what I am trying to tell you).
Well, it's not just hp-wmi.
On a laptop with _no_ rfkill keys and only pure input keys that get
handled by userspace or whatever, there should still be "transmitter
sw-blocked/hw-blocked/unblocked" functionality available, even though
there is no hardware rfkill key present on the system. That's another
thing I'm confused about. It seems that the rfkill class is not
appropriate for that if it's trying to also handle actual rfkill keys
too.
They are two separate things, because there are examples of hardware
that are both ways. And separating the switch bits from the transmitter
bits is the best way to accommodate both types of hardware.
Dan
> > - It just seems a lot clearer to me to have a separation between the
> > rfkill switch and the radio bits since there's no need to tie them
> > together
>
> And this is part of the confusion. So, for now, please just look at the
> question four paragraphs above, and tell me whether you want that
> information or not. I'd think you would, since you want to gray out the
> "enable radio" button when the radio is blocked by some hardware signal
> you cannot override, but I could be confused about it...
>
next prev parent reply other threads:[~2008-06-06 14:27 UTC|newest]
Thread overview: 43+ messages / expand[flat|nested] mbox.gz Atom feed top
2008-06-04 3:10 [GIT PATCH] rkfill improvements for -next Henrique de Moraes Holschuh
2008-06-04 3:10 ` [PATCH 01/12] rfkill: clarify meaning of rfkill states Henrique de Moraes Holschuh
2008-06-04 3:32 ` Pavel Roskin
2008-06-04 3:39 ` Henrique de Moraes Holschuh
2008-06-04 20:27 ` Dan Williams
2008-06-04 20:32 ` Dan Williams
2008-06-04 23:07 ` Tomas Winkler
2008-06-05 0:38 ` Henrique de Moraes Holschuh
2008-06-05 8:33 ` Tomas Winkler
2008-06-05 12:38 ` Henrique de Moraes Holschuh
2008-06-05 12:12 ` Dan Williams
2008-06-05 13:03 ` Henrique de Moraes Holschuh
2008-06-05 14:46 ` Dan Williams
2008-06-05 20:13 ` Henrique de Moraes Holschuh
2008-06-06 3:26 ` Dan Williams
2008-06-06 13:24 ` Dan Williams
2008-06-06 14:14 ` Henrique de Moraes Holschuh
2008-06-06 14:27 ` Dan Williams [this message]
2008-06-07 12:09 ` Tomas Winkler
2008-06-08 20:16 ` Matthew Garrett
2008-06-10 4:11 ` Henrique de Moraes Holschuh
2008-06-11 17:10 ` Tomas Winkler
2008-06-12 18:03 ` Henrique de Moraes Holschuh
2008-06-12 15:43 ` Dan Williams
2008-06-12 16:31 ` Henrique de Moraes Holschuh
2008-06-05 16:03 ` Ivo van Doorn
2008-06-05 16:36 ` Tomas Winkler
2008-06-05 17:42 ` Henrique de Moraes Holschuh
2008-06-05 17:54 ` Tomas Winkler
2008-06-05 20:16 ` Henrique de Moraes Holschuh
2008-06-05 17:53 ` Henrique de Moraes Holschuh
2008-06-04 3:10 ` [PATCH 02/12] rfkill: fix minor typo in kernel doc Henrique de Moraes Holschuh
2008-06-04 3:10 ` [PATCH 03/12] rfkill: handle SW_RFKILL_ALL events Henrique de Moraes Holschuh
2008-06-04 3:10 ` [PATCH 04/12] rfkill: add parameter to disable radios by default Henrique de Moraes Holschuh
2008-06-04 3:10 ` [PATCH 05/12] rfkill: add read-write rfkill switch support Henrique de Moraes Holschuh
2008-06-04 3:10 ` [PATCH 06/12] rfkill: add the WWAN radio type Henrique de Moraes Holschuh
2008-06-04 3:10 ` [PATCH 07/12] rfkill: rework suspend and resume handlers Henrique de Moraes Holschuh
2008-06-04 3:10 ` [PATCH 08/12] rfkill: add notifier chains support Henrique de Moraes Holschuh
2008-06-04 3:10 ` [PATCH 09/12] rfkill: add type string helper Henrique de Moraes Holschuh
2008-06-04 3:10 ` [PATCH 10/12] rfkill: add uevent notifications Henrique de Moraes Holschuh
2008-06-04 3:10 ` [PATCH 11/12] rfkill: do not allow userspace to override ALL RADIOS OFF Henrique de Moraes Holschuh
2008-06-04 3:10 ` [PATCH 12/12] rfkill: document rw rfkill switches and clarify input subsystem interactions Henrique de Moraes Holschuh
-- strict thread matches above, loose matches on Subject: below --
2008-06-22 15:38 [GIT PATCH] rfkill rework for 2.6.27 (v2) Henrique de Moraes Holschuh
2008-06-22 15:38 ` [PATCH 01/12] rfkill: clarify meaning of rfkill states Henrique de Moraes Holschuh
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=1212762451.28631.3.camel@localhost.localdomain \
--to=dcbw@redhat.com \
--cc=dtor@mail.ru \
--cc=hmh@hmh.eng.br \
--cc=ivdoorn@gmail.com \
--cc=linux-wireless@vger.kernel.org \
--cc=linville@tuxdriver.com \
--cc=tomasw@gmail.com \
/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).