[<prev] [next>] [<thread-prev] [thread-next>] [day] [month] [year] [list]
Message-ID: <ec4b7edcda0a62cd4e36e501f5ad56f9@sipsolutions.net>
Date: Wed, 10 Feb 2016 18:12:41 +0100
From: Johannes Berg <johannes@...solutions.net>
To: Dan Williams <dcbw@...hat.com>
Cc: João Paulo Rechi Vita <jprvita@...il.com>,
"David S. Miller" <davem@...emloft.net>,
Darren Hart <dvhart@...radead.org>,
linux-wireless@...r.kernel.org, netdev@...r.kernel.org,
platform-driver-x86@...r.kernel.org, linux-api@...r.kernel.org,
linux-doc@...r.kernel.org, linux-kernel@...r.kernel.org,
linux@...lessm.com,
João Paulo Rechi Vita <jprvita@...lessm.com>
Subject: Re: [PATCH 8/9] rfkill: Userspace control for airplane mode
On 2016-02-10 17:53, Dan Williams wrote:
> Yeah, I get that now. It's just that to me, something called
> "AIRPLANE_MODE_CHANGE" seems like it should actually change airplane
> mode on/off, which implies killing radios. I wouldn't have had the
> problem if it was named AIRPLANE_MODE_INDICATOR_CHANGE, which makes it
> clear this is simply an indicator and has no actual effect on anything
> other than a LED.
Yeah, I agree. I'm not even sure that it's a good idea to subsume this
into the regular operations in the API, although that's obviously the
easiest extension. I'll need to think about that a bit more with the
code at hand though.
johannes
Powered by blists - more mailing lists