Date: Mon, 28 Jul 2008 00:04:37 -0300
From: Henrique de Moraes Holschuh <hmh@hmh.eng.br>
To: Philip Langdale <philipl@overt.org>
Cc: Ivo van Doorn <ivdoorn@gmail.com>, LKML <linux-kernel@vger.kernel.org>,
       Matthew Garrett <mjg59@srcf.ucam.org>, toshiba_acpi@memebeam.org,
       linux-wireless@vger.kernel.org
Subject: Re: [PATCH 1/1] toshiba_acpi: Add support for bluetooth toggling
	through rfkill
Message-ID: <20080728030437.GC10672@khazad-dum.debian.net>
References: <488CBBAB.6010508@overt.org> <200807272259.44364.IvDoorn@gmail.com> <488CF6D8.6000700@overt.org>
MIME-Version: 1.0
Content-Type: text/plain; charset=us-ascii
Content-Disposition: inline
In-Reply-To: <488CF6D8.6000700@overt.org>
User-Agent: Mutt/1.5.18 (2008-05-17)
Sender: linux-kernel-owner@vger.kernel.org
Content-Length: 1441
Lines: 30

On Sun, 27 Jul 2008, Philip Langdale wrote:
> Ivo van Doorn wrote:
>>
>> You don't seem to be using rfkill_force_state() which is required to inform the rfkill
>> layer about the state changes.
>
> Hmm? According to rfkill.txt, one can either use force_state() or implement the
> get_state() hook, and I have done the later. If this is not the correct method,
> can you please explain when I should be using force_state?

There is a bunch of rfkill bug fix patches that was not merged in
wireless-testing yet (which is a pity, it would be really good if they could
go into 2.6.27).  One of those patches fixes the docs to make it clear that
rfkill_force_state() is the way to go if you have events of any sort.

The big difference from get_state() is that rfkill_force_state() propagates
state changes as soon as your driver notices them and sends them to rfkill
through rfkill_force_state()... while get_state() can only do that when
someone asks rfkill about the current state.

-- 
  "One disk to rule them all, One disk to find them. One disk to bring
  them all and in the darkness grind them. In the Land of Redmond
  where the shadows lie." -- The Silicon Valley Tarot
  Henrique Holschuh
--
To unsubscribe from this list: send the line "unsubscribe linux-kernel" in
the body of a message to majordomo@vger.kernel.org
More majordomo info at  http://vger.kernel.org/majordomo-info.html
Please read the FAQ at  http://www.tux.org/lkml/