Return-path: Received: from ist.d-labs.de ([213.239.218.44]:53593 "EHLO mx01.d-labs.de" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S1756545Ab0BTWIK (ORCPT ); Sat, 20 Feb 2010 17:08:10 -0500 From: florian@mickler.org To: linux-wireless@vger.kernel.org Cc: Randy Dunlap , Johannes Berg , "John W. Linville" , Alan Jenkins , linux-kernel@vger.kernel.org, linux-doc@vger.kernel.org, Greg Kroah-Hartman , Florian Mickler Subject: [PATCH 2/2] enhance /sys/class/rfkill//state interface Date: Sat, 20 Feb 2010 22:40:45 +0100 Message-Id: <1266702045-6062-3-git-send-email-florian@mickler.org> In-Reply-To: <1266702045-6062-1-git-send-email-florian@mickler.org> References: <1266702045-6062-1-git-send-email-florian@mickler.org> Sender: linux-wireless-owner@vger.kernel.org List-ID: Introduce a new state-value RFKILL_STATE_SOFT_AND_HARD_BLOCKED which is returned only through the sysfs state file. The other interfaces are designed so that they don't need this extra state. This allows the sysfs to represent all possible states an rfkill driver can have. Signed-off-by: Florian Mickler --- After stumbling over this arbitrary limitation of sys/class/rfkill/*/state I wondered what would hinder this patch? Documentation/ABI/stable/sysfs-class-rfkill | 7 ++++--- include/linux/rfkill.h | 3 +++ net/rfkill/core.c | 9 ++++++++- 3 files changed, 15 insertions(+), 4 deletions(-) diff --git a/Documentation/ABI/stable/sysfs-class-rfkill b/Documentation/ABI/stable/sysfs-class-rfkill index 53a4287..0cda9b3 100644 --- a/Documentation/ABI/stable/sysfs-class-rfkill +++ b/Documentation/ABI/stable/sysfs-class-rfkill @@ -42,9 +42,7 @@ What: /sys/class/rfkill/rfkill[0-9]+/state Date: 09-Jul-2007 KernelVersion v2.6.22 Contact: linux-wireless@vger.kernel.org -Description: Current state of the transmitter. This file is deprecated - because it can only properly show three of the four possible - states, soft-and-hard-blocked is missing. +Description: Current state of the transmitter. Values: A numeric value. 0: RFKILL_STATE_SOFT_BLOCKED transmitter is turned off by software @@ -53,6 +51,9 @@ Values: A numeric value. 2: RFKILL_STATE_HARD_BLOCKED transmitter is forced off by something outside of the driver's control. + 3: RFKILL_STATE_SOFT_AND_HARD_BLOCKED + transmitter is blocked by something outside of + the driver's control as well as turned off by software What: /sys/class/rfkill/rfkill[0-9]+/claim diff --git a/include/linux/rfkill.h b/include/linux/rfkill.h index 97059d0..b09a655 100644 --- a/include/linux/rfkill.h +++ b/include/linux/rfkill.h @@ -25,6 +25,8 @@ #define RFKILL_STATE_SOFT_BLOCKED 0 #define RFKILL_STATE_UNBLOCKED 1 #define RFKILL_STATE_HARD_BLOCKED 2 +/* only used for sysfs */ +#define RFKILL_STATE_SOFT_AND_HARD_BLOCKED 3 /** * enum rfkill_type - type of rfkill switch. @@ -109,6 +111,7 @@ enum rfkill_user_states { RFKILL_USER_STATE_SOFT_BLOCKED = RFKILL_STATE_SOFT_BLOCKED, RFKILL_USER_STATE_UNBLOCKED = RFKILL_STATE_UNBLOCKED, RFKILL_USER_STATE_HARD_BLOCKED = RFKILL_STATE_HARD_BLOCKED, + RFKILL_USER_STATE_SOFT_AND_HARD_BLOCKED = RFKILL_STATE_SOFT_AND_HARD_BLOCKED, }; #undef RFKILL_STATE_SOFT_BLOCKED #undef RFKILL_STATE_UNBLOCKED diff --git a/net/rfkill/core.c b/net/rfkill/core.c index c218e07..6d40297 100644 --- a/net/rfkill/core.c +++ b/net/rfkill/core.c @@ -645,12 +645,19 @@ static ssize_t rfkill_state_show(struct device *dev, struct rfkill *rfkill = to_rfkill(dev); unsigned long flags; u32 state; + int user_state; spin_lock_irqsave(&rfkill->lock, flags); state = rfkill->state; spin_unlock_irqrestore(&rfkill->lock, flags); - return sprintf(buf, "%d\n", user_state_from_blocked(state)); + if (( state & RFKILL_BLOCK_HW ) && ( state & RFKILL_BLOCK_SW )) + user_state = RFKILL_USER_STATE_SOFT_AND_HARD_BLOCKED; + else + user_state = user_state_from_blocked(state); + + return sprintf(buf, "%d\n", user_state); + } static ssize_t rfkill_state_store(struct device *dev, -- 1.6.6.1