From: "John W. Linville" <linville@tuxdriver.com>
To: "Rafael J. Wysocki" <rjw@sisk.pl>
Cc: Johannes Berg <johannes@sipsolutions.net>,
acpi4asus-user@lists.sourceforge.net,
Darren Salt <linux@youmustbejoking.demon.co.uk>,
debian-eeepc-devel@lists.alioth.debian.org,
linux-wireless@vger.kernel.org, linux-kernel@vger.kernel.org,
Thiemo Nagel <thiemo.nagel@ph.tum.de>,
Corentin Chary <corentin.chary@gmail.com>
Subject: Re: [Acpi4asus-user] [PATCH 2.6.31] rfkill: allow toggling soft state in sysfs again
Date: Sat, 25 Jul 2009 18:35:06 -0400 [thread overview]
Message-ID: <20090725223505.GA2729@tuxdriver.com> (raw)
In-Reply-To: <200907252256.44570.rjw@sisk.pl>
On Sat, Jul 25, 2009 at 10:56:43PM +0200, Rafael J. Wysocki wrote:
> On Friday 10 July 2009, Johannes Berg wrote:
> > On Fri, 2009-07-10 at 22:09 +0100, Darren Salt wrote:
> > > I demand that Johannes Berg may or may not have written...
> > >
> > > > Apparently there actually _are_ tools that try to set this in sysfs even
> > > > though it wasn't supposed to be used this way without claiming first.
> > >
> > > Then it should have been documented as such. I don't see anything about this
> > > in Documentation/rfkill.txt (as found in 2.6.30), other than a vague
> > > statement that "Kernel handles events", which isn't exactly helpful :-\
> >
> > Oh, it's not just that rfkill was horrible, the documentation matched :)
> > All the SHOUTING in it about what you must and must not do but nothing
> > actually helpful :)
> >
> > > > *shrug*, I don't like it, but whatever...
> > >
> > > I do. It means that we have a nice simple text-based interface for use in
> > > scripts (for now), and a binary interface which is better suited to the likes
> > > of desktop applications.
> >
> > Indeed, and as long as you expect to only use soft toggle... problem is
> > that you won't know whether it's soft-toggled or not while it's
> > hard-blocked (off)!
> >
> > > > Please test & report.
> > >
> > > With the patch applied, Bluetooth toggling is working again, so you get to
> > > add this:
> > >
> > > Tested-By: Darren Salt <linux@youmustbejoking.demon.co.uk>
> >
> > Ok, John, please pick up the patch.
>
> Is there anything going on with the patch?
>
> Surely it's not in -rc4.
commit f54c142725ad2ba33c3ee627873cb6966bf05447
Author: Johannes Berg <johannes@sipsolutions.net>
Date: Fri Jul 10 21:41:39 2009 +0200
rfkill: allow toggling soft state in sysfs again
Apparently there actually _are_ tools that try to set
this in sysfs even though it wasn't supposed to be used
this way without claiming first. Guess what: now that
I've cleaned it all up it doesn't matter and we can
simply allow setting the soft-block state in sysfs.
Signed-off-by: Johannes Berg <johannes@sipsolutions.net>
Tested-By: Darren Salt <linux@youmustbejoking.demon.co.uk>
Signed-off-by: John W. Linville <linville@tuxdriver.com>
Didn't make -rc4, should be in -rc5.
John
--
John W. Linville Someday the world will need a hero, and you
linville@tuxdriver.com might be all we have. Be ready.
¡Viva Honduras Libre!
next prev parent reply other threads:[~2009-07-25 22:46 UTC|newest]
Thread overview: 10+ messages / expand[flat|nested] mbox.gz Atom feed top
[not found] <4A5716AA.5000903@ph.tum.de>
[not found] ` <71cd59b00907100545v6f440a19xfc3668826eb1e509@mail.gmail.com>
[not found] ` <4A5743AB.6090303@ph.tum.de>
[not found] ` <71cd59b00907100646h5e0283fcyce5874cc4a19106b@mail.gmail.com>
[not found] ` <4A575119.9070505@ph.tum.de>
2009-07-10 18:57 ` [2.6.31-rc2] Writing to /sys/class/rfkill/*/state fails Darren Salt
2009-07-10 19:34 ` Corentin Chary
2009-07-10 19:37 ` Marcel Holtmann
2009-07-10 19:41 ` [PATCH 2.6.31] rfkill: allow toggling soft state in sysfs again Johannes Berg
2009-07-10 21:09 ` Darren Salt
2009-07-10 21:55 ` Johannes Berg
2009-07-10 23:09 ` Darren Salt
2009-07-25 20:56 ` [Acpi4asus-user] " Rafael J. Wysocki
2009-07-25 22:35 ` John W. Linville [this message]
2009-07-26 19:32 ` Rafael J. Wysocki
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=20090725223505.GA2729@tuxdriver.com \
--to=linville@tuxdriver.com \
--cc=acpi4asus-user@lists.sourceforge.net \
--cc=corentin.chary@gmail.com \
--cc=debian-eeepc-devel@lists.alioth.debian.org \
--cc=johannes@sipsolutions.net \
--cc=linux-kernel@vger.kernel.org \
--cc=linux-wireless@vger.kernel.org \
--cc=linux@youmustbejoking.demon.co.uk \
--cc=rjw@sisk.pl \
--cc=thiemo.nagel@ph.tum.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 an external index of several public inboxes,
see mirroring instructions on how to clone and mirror
all data and code used by this external index.