Return-path: Received: from mx1.redhat.com ([66.187.233.31]:40457 "EHLO mx1.redhat.com" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S1750740AbYG1LUE (ORCPT ); Mon, 28 Jul 2008 07:20:04 -0400 Subject: Re: [PATCH 1/1] toshiba_acpi: Add support for bluetooth toggling through rfkill From: Dan Williams To: Henrique de Moraes Holschuh Cc: Philip Langdale , Ivo van Doorn , LKML , Matthew Garrett , toshiba_acpi@memebeam.org, linux-wireless@vger.kernel.org In-Reply-To: <20080728030437.GC10672@khazad-dum.debian.net> References: <488CBBAB.6010508@overt.org> <200807272259.44364.IvDoorn@gmail.com> <488CF6D8.6000700@overt.org> <20080728030437.GC10672@khazad-dum.debian.net> Content-Type: text/plain Date: Mon, 28 Jul 2008 07:17:43 -0400 Message-Id: <1217243863.24609.6.camel@localhost.localdomain> (sfid-20080728_132008_852164_0400B837) Mime-Version: 1.0 Sender: linux-wireless-owner@vger.kernel.org List-ID: On Mon, 2008-07-28 at 00:04 -0300, Henrique de Moraes Holschuh wrote: > 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 Lots of wireless people (including John) were at OLS this past week, so it's not entirely surprising that patch merging might have been slow. Dan > 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. >