All of lore.kernel.org
 help / color / mirror / Atom feed
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!

  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.